Skip to contentSkip to conversation

staging: comedi: update comedi pci drivers

PATCH In review H Hartley Sweeten 2 participants 3 messages Aug 17, 2012

lore
Share & tools
lore
#0 lore raw

cover staging: comedi: update comedi pci drivers

Remove the manual legacy 'attach' callback in the comedi pci drivers that have it stubbed out. The 'attach' callback is not optional in the comedi core.

Cleanup and convert some of the other comedi pci drivers to use the 'attach_pci' callback and remove the legacy 'attach'.

H Hartley Sweeten (35):
staging: comedi: adl_pci6208: remove manual legacy attach
staging: comedi: adl_pci7296: remove manual legacy attach
staging: comedi: adl_pci7x3x: remove manual legacy attach
staging: comedi: adl_pci8164: use attach_pci callback
staging: comedi: cb_pcidas: use attach_pci callback
staging: comedi: cb_pcidio: remove thisboard macro
staging: comedi: cb_pcidio: use attach_pci callback
staging: comedi: cb_pcidda: remove thisboard and devpriv macros
staging: comedi: cb_pcidda: remove forward declarations
staging: comedi: cb_pcidda: use attach_pci callback
staging: comedi: cb_pcimdas: remove thisboard and devpriv macros
staging: comedi: cb_pcimdas: remove forward declarations
staging: comedi: cb_pcimdas: use attach_pci callback
staging: comedi: cb_pcimdda: remove REG_SZ and REGS_BARINDEX macros
staging: comedi: cb_pcimdda: remove thisboard and devpriv macros
staging: comedi: cb_pcimdda: remove forward declarations
staging: comedi: cb_pcimdda: add namespace to the driver
staging: comedi: cb_pcimdda: cleanup pci probe
staging: comedi: cb_pcimdda: use dev->iobase
staging: comedi: cb_pcimdda: store the pci_dev in the comedi_device
staging: comedi: cb_pcimdda: cleanup the 8255 subdevice init
staging: comedi: cb_pcimdda: remove the DIO_METHODS
staging: comedi: cb_pcimdda: remove dio_chans from the private data
staging: comedi: cb_pcimdda: define the register map
staging: comedi: cb_pcimdda: remove dio_offset from the boardinfo
staging: comedi: cb_pcimdda: remove regs_badrindex and reg_sz from
boardinfo
staging: comedi: cb_pcimdda: remove ao_chans and ao_bits from
boardinfo
staging: comedi: cb_pcimdda: remove boardinfo
staging: comedi: cb_pcimdda: remove attached_to_8255 from private
data
staging: comedi: cb_pcimdda: fix bug in call to subdev_8255_cleanup
staging: comedi: cb_pcimdda: minor cleanup of the private data
staging: comedi: cb_pcimdda: cleanup the analog out read/write
staging: comedi: cb_pcimdda: use attach_pci callback
staging: comedi: dyna_pci10xx: remove manual legacy attach
staging: comedi: ke_counter: use attach_pci callback

 drivers/staging/comedi/drivers/adl_pci6208.c  |  10 -
 drivers/staging/comedi/drivers/adl_pci7296.c  |  10 -
 drivers/staging/comedi/drivers/adl_pci7x3x.c  |  10 -
 drivers/staging/comedi/drivers/adl_pci8164.c  |  65 +----
 drivers/staging/comedi/drivers/cb_pcidas.c    |  84 ++----
 drivers/staging/comedi/drivers/cb_pcidda.c    | 382 +++++++++++--------------
 drivers/staging/comedi/drivers/cb_pcidio.c    | 104 +++----
 drivers/staging/comedi/drivers/cb_pcimdas.c   | 279 ++++++++-----------
 drivers/staging/comedi/drivers/cb_pcimdda.c   | 387 ++++++--------------------
 drivers/staging/comedi/drivers/dyna_pci10xx.c |  10 -
 drivers/staging/comedi/drivers/ke_counter.c   |  83 ++----
 11 files changed, 475 insertions(+), 949 deletions(-)
Signature · 1 line
1.7.11
#1 lore raw
Quoted 3 lines · H Hartley Sweeten

Remove the manual legacy 'attach' callback in the comedi pci drivers that have it stubbed out. The 'attach' callback is not optional in the comedi core.

Just to avoid confusion, I think Hartley meant "now optional" in the above sentence!

Signature · 2 lines
-=( Ian Abbott @ MEV Ltd.    E-mail: <[email protected]>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
#2 lore raw
Quoted 7 lines · Ian Abbott
Quoted 3 lines · H Hartley Sweeten

Remove the manual legacy 'attach' callback in the comedi pci drivers that have it stubbed out. The 'attach' callback is not optional in the comedi core.

Just to avoid confusion, I think Hartley meant "now optional" in the above sentence!

Doh... Fat fingered that. Thanks!

Hartley