[PATCH 0/5] staging: comedi: initialize dev->board_name in the core
H Hartley Sweeten
hartleys at visionengravers.com
Mon Apr 8 18:38:58 UTC 2013
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 at mev.co.uk>
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 at mev.co.uk> )=-
-=( Tel: +44 (0)161 477 1898 FAX: +44 (0)161 718 3587 )=-
More information about the devel
mailing list