On Tue, Feb 05, 2013 at 06:15:20PM -0700, H Hartley Sweeten wrote: > - ret = pcmcia_enable_device(link); > - if (ret) > + link->config_flags |= CONF_AUTO_SET_IO; > + ret = comedi_pcmcia_enable(dev); > return ret; > dev->iobase = link->resource[0]->start; There is an if (ret) statement missing? The last line is unreachable. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel