staging: comedi: ni_labpc: remove some unnecessary defines
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 22 Mar 2016 18:21:35 +0000 (11:21 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
The EEPROM_SIZE and NUM_AO_CHAN defines are only used once and they
don't add any significant clarity to the driver. They are also pretty
generic symbol names. Remove them and just open code the values.

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_labpc.h
drivers/staging/comedi/drivers/ni_labpc_common.c

index 83f878adbd5365e17dad60f686a3bfed7b13e596..74db3ba82ab5c5d14f402aeef30b5c68ae8d392c 100644 (file)
@@ -19,9 +19,6 @@
 #ifndef _NI_LABPC_H
 #define _NI_LABPC_H
 
-#define EEPROM_SIZE    256     /*  256 byte eeprom */
-#define NUM_AO_CHAN    2       /*  boards have two analog output channels */
-
 enum transfer_type { fifo_not_empty_transfer, fifo_half_full_transfer,
        isa_dma_transfer
 };
index 863afb28ee286c8271b260a68958eaba8a9c4a9b..55ab05e3196b9fffd43efa8ace91adc218d95261 100644 (file)
@@ -1261,7 +1261,7 @@ int labpc_common_attach(struct comedi_device *dev,
        if (board->has_ao) {
                s->type         = COMEDI_SUBD_AO;
                s->subdev_flags = SDF_READABLE | SDF_WRITABLE | SDF_GROUND;
-               s->n_chan       = NUM_AO_CHAN;
+               s->n_chan       = 2;
                s->maxdata      = 0x0fff;
                s->range_table  = &range_labpc_ao;
                s->insn_write   = labpc_ao_insn_write;
@@ -1307,12 +1307,12 @@ int labpc_common_attach(struct comedi_device *dev,
                s->type         = COMEDI_SUBD_UNUSED;
        }
 
-       /* EEPROM */
+       /* EEPROM (256 bytes) */
        s = &dev->subdevices[4];
        if (board->is_labpc1200) {
                s->type         = COMEDI_SUBD_MEMORY;
                s->subdev_flags = SDF_READABLE | SDF_WRITABLE | SDF_INTERNAL;
-               s->n_chan       = EEPROM_SIZE;
+               s->n_chan       = 256;
                s->maxdata      = 0xff;
                s->insn_write   = labpc_eeprom_insn_write;