staging: comedi: ni_mio_common: tidy up the calibration subdevice init
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 14 Jul 2014 19:24:01 +0000 (12:24 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 16 Jul 2014 20:24:43 +0000 (13:24 -0700)
For aesthetics, add some whitespace to the subdevice init.

Refactor the code so that the common parts of the subdevice are initialized
in one place.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/ni_mio_common.c

index e60b9e6..49c22d6 100644 (file)
@@ -5642,26 +5642,26 @@ static int ni_E_init(struct comedi_device *dev,
        s = &dev->subdevices[NI_UNUSED_SUBDEV];
        s->type = COMEDI_SUBD_UNUSED;
 
-       /* calibration subdevice -- ai and ao */
+       /* Calibration subdevice */
        s = &dev->subdevices[NI_CALIBRATION_SUBDEV];
-       s->type = COMEDI_SUBD_CALIB;
+       s->type         = COMEDI_SUBD_CALIB;
+       s->subdev_flags = SDF_INTERNAL;
+       s->n_chan       = 1;
+       s->maxdata      = 0;
        if (devpriv->is_m_series) {
-               /*  internal PWM analog output used for AI nonlinearity calibration */
-               s->subdev_flags = SDF_INTERNAL;
-               s->insn_config = &ni_m_series_pwm_config;
-               s->n_chan = 1;
-               s->maxdata = 0;
+               /* internal PWM output used for AI nonlinearity calibration */
+               s->insn_config  = ni_m_series_pwm_config;
+
                ni_writel(dev, 0x0, M_Offset_Cal_PWM);
        } else if (devpriv->is_6143) {
-               /*  internal PWM analog output used for AI nonlinearity calibration */
-               s->subdev_flags = SDF_INTERNAL;
-               s->insn_config = &ni_6143_pwm_config;
-               s->n_chan = 1;
-               s->maxdata = 0;
+               /* internal PWM output used for AI nonlinearity calibration */
+               s->insn_config  = ni_6143_pwm_config;
        } else {
-               s->subdev_flags = SDF_WRITABLE | SDF_INTERNAL;
-               s->insn_read = &ni_calib_insn_read;
-               s->insn_write = &ni_calib_insn_write;
+               s->subdev_flags |= SDF_WRITABLE;
+               s->insn_read    = ni_calib_insn_read;
+               s->insn_write   = ni_calib_insn_write;
+
+               /* setup the caldacs and find the real n_chan and maxdata */
                caldac_setup(dev, s);
        }