On Thursday, February 07, 2013 12:43 AM, Dan Carpenter wrote:
> 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.

Thanks for noticing this. Removing the if (ret) was not intended.

Greg has already merged this into his staging tree. I will post a follow up
patch to fix it shortly.

Regards,
Hartley

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to