* [PATCH 00/31] staging: comedi: ni_daq_700: cleanup the driver
@ 2012-06-22 23:12 H Hartley Sweeten
0 siblings, 0 replies; only message in thread
From: H Hartley Sweeten @ 2012-06-22 23:12 UTC (permalink / raw)
To: Linux Kernel; +Cc: devel, abbotti, fmhess, gregkh
This is a full cleanup of the ni_daq_700 comedi driver.
This driver was originally based on the 8255 driver which exports
a subdevice interface used by other comedi drivers. This makes
the ni_daq_700 driver overly complex. Also, the exported symbols
in it are not referenced by any other code or declared in any
headers.
The pcmcia support code in this driver is also overly complex. All
the driver needs from the pcmcia support is for the device to be
enabled and to get the iobase address of the card. Everything else
appears to be cut-and-paste from other drivers.
H Hartley Sweeten (31):
staging: comedi: ni_daq_700: remove exported symbols
staging: comedi: ni_daq_700: move the MODULE_* stuff to the EOF
staging: comedi: ni_daq_700: add the module_{init,exit} declarations
staging: comedi: ni_daq_700: consolidate and init and exit functions
staging: comedi: ni_daq_700: change driver registration order
staging: comedi: ni_daq_700: cleanup to the pcmcia_driver declaration
staging: comedi: ni_daq_700: remove some useless comments
staging: comedi: ni_daq_700: refactor the pcmcia support code
staging: comedi: ni_daq_700: Move the comedi_driver variable
staging: comedi: ni_daq_700: Remove the unused private data struct
staging: comedi: ni_daq_700: Remove a "do nothing" function stub
staging: comedi: ni_daq_700: Remove unused variables from the
boardinfo
staging: comedi: ni_daq_700: Remove unneeded variables from the
boardinfo
staging: comedi: ni_daq_700: Remove bustype variable from the
boardinfo
staging: comedi: ni_daq_700: move the boardinfo variable
staging: comedi: ni_daq_700: remove thisboard macro
staging: comedi: ni_daq_700: remove unused private data variable
staging: comedi: ni_daq_700: remove the CALLBACK_* code
staging: comedi: ni_daq_700: remove a couple unused defines
staging: comedi: ni_daq_700: remove the subdev_700_init function
staging: comedi: ni_daq_700: remove the #ifdef'ed out irq code
staging: comedi: ni_daq_700: change to 'attach' printk's to
dev_printk's
staging: comedi: ni_daq_700: remove local variable in the 'attach'
function
staging: comedi: ni_daq_700: remove some dangling ';'
staging: comedi: ni_daq_700: use a local var to fix a > 80 char line
issue
staging: comedi: ni_daq_700: fix return for insn_config function
staging: comedi: ni_daq_700: cleanup pcmcia debug output messages
staging: comedi: ni_daq_700: remove unused pcmcia 'stop' logic
staging: comedi: ni_daq_700: remove unneeded pcmcia private data
staging: comedi: ni_daq_700: remove the pcmcia suspend/resume
staging: comedi: ni_daq_700: refactor the pcmcia attach/detach
drivers/staging/comedi/drivers/ni_daq_700.c | 508 +++++-----------------------
1 file changed, 81 insertions(+), 427 deletions(-)
--
1.7.11
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2012-06-22 23:12 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-22 23:12 [PATCH 00/31] staging: comedi: ni_daq_700: cleanup the driver H Hartley Sweeten
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®