Re: [PATCH] staging: comedi: Convert struct addi_board initialization to C99 format

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

 



On 2012-05-17 22:12, H Hartley Sweeten wrote:
On Thursday, May 17, 2012 11:29 AM, Ian Abbott wrote:
On 17/05/2012 17:38, H Hartley Sweeten wrote:
BTW, what actual hardware do you have available to test with?

Not a lot.  Just the Amplicon cards (amplc_* drivers).

Speaking of the amplc_* drivers... What's up with them?

The dio200, pc236, and pc263 drivers appear to be some kind of
hybrid comedi driver/comedi pci driver.

That's right. The ISA devices and PCI devices are selected separately in the configuration, but they are similar enough to share the same code.

If the driver is simply
registered as a comedi pci driver will the isa drivers still work?

Yes if both the ISA devices and PCI devices are enabled in the configuration. But you might have one or the other.

In amplc_dio200.c (for example, but similarly for the others), you can assume that at least one of the CONFIG_COMEDI_AMPLC_DIO200_ISA and CONFIG_COMEDI_AMPLC_DIO200_PCI macros is #define'd, so you could do (with the existing variable names, that you'd probably change):

#ifdef CONFIG_COMEDI_AMPLC_DIO200_PCI
module_comedi_pci_driver(driver_amplc_dio200, driver_amplc_dio200_pci_driver);
#else
module_comedi_driver(driver_amplc_dio200);
#endif

(I guess you'd change driver_amplc_dio200 to amplc_dio200_driver and driver_amplc_dio200_pci_driver to amplc_dio200_pci_driver.)

If so we can get rid of a bunch of #ifdef stuff.

Some of them are there to separately configure the ISA and PCI devices and some are just there to trim out some code that is unreachable when one or the other isn't defined.

I tend not to include several functions within the same #ifdef, so it should be easy to move things around to avoid forward declarations.

--
-=( 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