[PATCH 1/5] staging: comedi: ni_daq_dio24: use comedi_pcmcia_{enable, disable}

H Hartley Sweeten hartleys at visionengravers.com
Thu Feb 7 17:04:18 UTC 2013


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




More information about the devel mailing list