RE: [PATCH] staging: comedi: Add helper macro for comedi pci driverboilerplate
From: H Hartley Sweeten
Date: Fri May 11 2012 - 13:34:15 EST
On Friday, May 11, 2012 10:32 AM, Ian Abbott wrote:
> On 2012-05-11 18:07, H Hartley Sweeten wrote:
>> +int comedi_pci_driver_register(struct comedi_driver *comedi_driver,
>> + struct pci_driver *pci_driver)
>> +{
>> + int ret;
>> +
>> + ret = comedi_driver_register(comedi_driver);
>> + if (ret< 0)
>> + return ret;
>> +
>> + pci_driver->name = comedi_driver->driver_name;
>
> I think it would be better if the driver modules that call this function
> already had pci_driver->name initialized statically.
I agree, but this is how all the pci drivers do it now.
To change this will require auditing all the comedi pci drivers and adding
the static name.
Regards,
Hartley
èº{.nÇ+·®+%Ëlzwm
ébëæìr¸zX§»®w¥{ayºÊÚë,j¢f£¢·hàz¹®w¥¢¸¢·¦j:+v¨wèjØm¶ÿ¾«êçzZ+ùÝj"ú!¶iOæ¬z·vØ^¶m§ÿðÃnÆàþY&