RE: [PATCH 0/5] staging: comedi: initialize dev->board_name in the core

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday, April 08, 2013 11:19 AM, Ian Abbott wrote:
> On 2013-04-08 18:54, H Hartley Sweeten wrote:
>> Initialize the dev->board_name in the comedi core before calling the (*attach)
>> or (*auto_attach) function.
>>
>> Remove all the iitialization of the dev->board_name in the comedi drivers.
>>
>> H Hartley Sweeten (5):
>>    staging: comedi: drivers: rename 'comedi_dev' in comedi_auto_config()
>>    staging: comedi: drivers: set dev->board_name before attaching
>>    staging: comedi: drivers: dev->board_name is always valid
>>    staging: comedi: comedi_pci: dev->board_name is always valid
>>    staging: comedi: remove unnecessary dev->board_name initialization
>>
>
> They all look fine to me.
>
> Reviewed-by: Ian Abbott <abbotti@xxxxxxxxx>

Thanks.

There are a couple more drivers that still need the dev->board_name
init removed. They are all legacy drivers that have use the board_ptr. 
I'm still looking at them to make sure they don't do any additional probing.

I'll post those after this series is accepted.

> (P.S. Hartley, I don't know if you plan to resubmit those other changes 
> you had that passed a "main" PCI BAR index to comedi_pci_enable(), but I 
> don't think they're as useful now as they were when you initially 
> submitted them, so I wouldn't bother if I were you.)

No. Now that you added the 'ioenabled' flag it makes more sense to
leave the iobase initialization in the driver.

Regards,
Hartley


-- 
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@xxxxxxxxx>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux