'BITS' is a pretty generic name for a define. It's only used in
the board attach to set the subdevice 'maxdata' so instead of
renaming the define just open code the value and remove the define.

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Ian Abbott <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/comedi/drivers/pcmda12.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/comedi/drivers/pcmda12.c 
b/drivers/staging/comedi/drivers/pcmda12.c
index 9b3f061..2aba7bf 100644
--- a/drivers/staging/comedi/drivers/pcmda12.c
+++ b/drivers/staging/comedi/drivers/pcmda12.c
@@ -55,7 +55,6 @@ Configuration Options:
 #define MSB(x) ((unsigned char)((((unsigned short)(x))>>8) & 0xff))
 #define LSB_PORT(chan) (dev->iobase + (chan)*2)
 #define MSB_PORT(chan) (LSB_PORT(chan)+1)
-#define BITS 12
 
 /* note these have no effect and are merely here for reference..
    these are configured by jumpering the board! */
@@ -169,7 +168,7 @@ static int pcmda12_attach(struct comedi_device *dev,
 
        s = &dev->subdevices[0];
        s->private = NULL;
-       s->maxdata = (0x1 << BITS) - 1;
+       s->maxdata = 0x0fff;
        s->range_table = &pcmda12_ranges;
        s->type = COMEDI_SUBD_AO;
        s->subdev_flags = SDF_READABLE | SDF_WRITABLE;
-- 
1.8.1.4

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to