[PATCH 00/46] staging: comedi: cleanup step 3 of do_cmdtest functions

H Hartley Sweeten hartleys at visionengravers.com
Wed Nov 14 00:47:38 UTC 2012


Step 3 of the do_cmdtest functions validates the arguments for the
command to be executed. Most of these are simple tests to see if the
argument "is" a value, a "min" value, or a "max" value. Each of these
tests then clamps the argument to the value if it fails the test.

Introduce three new helper functions in comedi_fc.h to handle these
tests.

Use the new helpers to remove the boilerplate code from the drivers.

H Hartley Sweeten (46):
  staging: comedi: comedi_fc.h: introduce new helpers for do_cmdtest step 3
  staging: comedi: 8255: use cfc_check_trigger_arg_*() helpers
  staging: comedi: comedi_parport: use cfc_check_trigger_arg_*() helpers
  staging: comedi: addi_apci_1032: use cfc_check_trigger_arg_*() helpers
  staging: comedi: amplc_dio200: use cfc_check_trigger_arg_*() helpers
  staging: comedi: amplc_pc236: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_6527: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_65xx: use cfc_check_trigger_arg_*() helpers
  staging: comedi: pcm_common: use cfc_check_trigger_arg_*() helpers
  staging: comedi: das800: use cfc_check_trigger_arg_*() helpers
  staging: comedi: comedi_test: use cfc_check_trigger_arg_*() helpers
  staging: comedi: hwdrv_apci3120: use cfc_check_trigger_arg_*() helpers
  staging: comedi: adl_pci9111: use cfc_check_trigger_arg_*() helpers
  staging: comedi: adl_pci9118: use cfc_check_trigger_arg_*() helpers
  staging: comedi: adv_pci1710: use cfc_check_trigger_arg_*() helpers
  staging: comedi: amplc_pci224: use cfc_check_trigger_arg_*() helpers
  staging: comedi: amplc_pci230: use cfc_check_trigger_arg_*() helpers
  staging: comedi: cb_das16_cs: use cfc_check_trigger_arg_*() helpers
  staging: comedi: cb_pcidas: use cfc_check_trigger_arg_*() helpers
  staging: comedi: cb_pcidas64: use cfc_check_trigger_arg_*() helpers
  staging: comedi: das16: use cfc_check_trigger_arg_*() helpers
  staging: comedi: das16m1: use cfc_check_trigger_arg_*() helpers
  staging: comedi: das1800: use cfc_check_trigger_arg_*() helpers
  staging: comedi: dmm32at: use cfc_check_trigger_arg_*() helpers
  staging: comedi: dt2814: use cfc_check_trigger_arg_*() helpers
  staging: comedi: dt282x: use cfc_check_trigger_arg_*() helpers
  staging: comedi: dt3000: use cfc_check_trigger_arg_*() helpers
  staging: comedi: gsc_hpdi: use cfc_check_trigger_arg_*() helpers
  staging: comedi: gsc_hpdi: cleanup step 3 of me4000_ai_do_cmd_test()
  staging: comedi: ni_at_a2150: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_atmio16d: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_labpc: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_pcidio: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_tiocmd: use cfc_check_trigger_arg_*() helpers
  staging: comedi: pcl711: use cfc_check_trigger_arg_*() helpers
  staging: comedi: pcl812: use cfc_check_trigger_arg_*() helpers
  staging: comedi: pcl816: use cfc_check_trigger_arg_*() helpers
  staging: comedi: pcl818: use cfc_check_trigger_arg_*() helpers
  staging: comedi: quatech_daqp_cs: use cfc_check_trigger_arg_*() helpers
  staging: comedi: rtd520: use cfc_check_trigger_arg_*() helpers
  staging: comedi: s626: use cfc_check_trigger_arg_*() helpers
  staging: comedi: skel: use cfc_check_trigger_arg_*() helpers
  staging: comedi: usbdux: use cfc_check_trigger_arg_*() helpers
  staging: comedi: usbduxfast: use cfc_check_trigger_arg_*() helpers
  staging: comedi: usbduxsigma: use cfc_check_trigger_arg_*() helpers
  staging: comedi: ni_mio_common: use cfc_check_trigger_arg_*() helpers

 drivers/staging/comedi/drivers/8255.c              |  27 +--
 .../comedi/drivers/addi-data/hwdrv_apci3120.c      |  56 ++-----
 drivers/staging/comedi/drivers/addi_apci_1032.c    |  27 +--
 drivers/staging/comedi/drivers/adl_pci9111.c       |  59 ++-----
 drivers/staging/comedi/drivers/adl_pci9118.c       |  72 +++-----
 drivers/staging/comedi/drivers/adv_pci1710.c       |  48 ++----
 drivers/staging/comedi/drivers/amplc_dio200.c      |  34 +---
 drivers/staging/comedi/drivers/amplc_pc236.c       |  27 +--
 drivers/staging/comedi/drivers/amplc_pci224.c      |  53 ++----
 drivers/staging/comedi/drivers/amplc_pci230.c      | 120 +++++--------
 drivers/staging/comedi/drivers/cb_das16_cs.c       |  64 +++----
 drivers/staging/comedi/drivers/cb_pcidas.c         |  80 +++------
 drivers/staging/comedi/drivers/cb_pcidas64.c       |  76 +++------
 drivers/staging/comedi/drivers/comedi_fc.h         |  44 +++++
 drivers/staging/comedi/drivers/comedi_parport.c    |  27 +--
 drivers/staging/comedi/drivers/comedi_test.c       |  63 ++-----
 drivers/staging/comedi/drivers/das16.c             |  55 ++----
 drivers/staging/comedi/drivers/das16m1.c           |  35 +---
 drivers/staging/comedi/drivers/das1800.c           |  38 ++---
 drivers/staging/comedi/drivers/das800.c            |  44 ++---
 drivers/staging/comedi/drivers/dmm32at.c           |  53 ++----
 drivers/staging/comedi/drivers/dt2814.c            |  42 ++---
 drivers/staging/comedi/drivers/dt282x.c            |  82 +++------
 drivers/staging/comedi/drivers/dt3000.c            |  53 ++----
 drivers/staging/comedi/drivers/gsc_hpdi.c          |  19 +--
 drivers/staging/comedi/drivers/me4000.c            |  17 +-
 drivers/staging/comedi/drivers/ni_6527.c           |  28 +---
 drivers/staging/comedi/drivers/ni_65xx.c           |  28 +---
 drivers/staging/comedi/drivers/ni_at_a2150.c       |  44 ++---
 drivers/staging/comedi/drivers/ni_atmio16d.c       |  43 ++---
 drivers/staging/comedi/drivers/ni_labpc.c          |  54 ++----
 drivers/staging/comedi/drivers/ni_mio_common.c     | 186 ++++++---------------
 drivers/staging/comedi/drivers/ni_pcidio.c         |  36 ++--
 drivers/staging/comedi/drivers/ni_tiocmd.c         |  40 ++---
 drivers/staging/comedi/drivers/pcl711.c            |  36 ++--
 drivers/staging/comedi/drivers/pcl812.c            |  58 ++-----
 drivers/staging/comedi/drivers/pcl816.c            |  50 ++----
 drivers/staging/comedi/drivers/pcl818.c            |  48 ++----
 drivers/staging/comedi/drivers/pcm_common.c        |  34 +---
 drivers/staging/comedi/drivers/quatech_daqp_cs.c   |  46 ++---
 drivers/staging/comedi/drivers/rtd520.c            |  72 ++++----
 drivers/staging/comedi/drivers/s626.c              |  78 +++------
 drivers/staging/comedi/drivers/skel.c              |  63 ++-----
 drivers/staging/comedi/drivers/usbdux.c            | 108 +++---------
 drivers/staging/comedi/drivers/usbduxfast.c        |  38 +----
 drivers/staging/comedi/drivers/usbduxsigma.c       | 105 +++---------
 46 files changed, 746 insertions(+), 1764 deletions(-)

-- 
1.7.11




More information about the devel mailing list