staging: comedi: adl_pci9111: remove pci9111_ai_channel_[gs]et macros
authorH Hartley Sweeten <hartleys@visionengravers.com>
Tue, 11 Sep 2012 02:01:31 +0000 (19:01 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Sep 2012 03:05:56 +0000 (20:05 -0700)
These macros rely on a local variable having a specific name.

The comedi code makes sure the channel number is valid so the mask
of PCI9111_CHANNEL_MASK can be remove.

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

index 1eef86d83f5cea130459fb56666ca37134c9c824..71311796e74cb39844133a207ac429edafc3140a 100644 (file)
@@ -95,8 +95,7 @@ TODO:
 #define PCI9111_AO_REG                                 0x00
 #define PCI9111_DIO_REG                                        0x02
 #define PCI9111_REGISTER_EXTENDED_IO_PORTS             0x04
-#define PCI9111_REGISTER_AD_CHANNEL_CONTROL            0x06 /* Channel
-                                                               selection */
+#define PCI9111_AI_CHANNEL_REG                         0x06
 #define PCI9111_REGISTER_AD_CHANNEL_READBACK           0x06
 #define PCI9111_AI_RANGE_REG                           0x08
 #define PCI9111_RANGE_STATUS_REG                       0x08
@@ -124,8 +123,6 @@ TODO:
 #define PCI9111_FFEN_SET_FIFO_ENABLE                   (0 << 2)
 #define PCI9111_FFEN_SET_FIFO_DISABLE                  (1 << 2)
 
-#define PCI9111_CHANNEL_MASK                           0x0F
-
 #define PCI9111_RANGE_MASK                             0x07
 #define PCI9111_FIFO_EMPTY_MASK                                0x10
 #define PCI9111_FIFO_HALF_FULL_MASK                    0x20
@@ -176,14 +173,6 @@ TODO:
        ((inb(dev->iobase + PCI9111_RANGE_STATUS_REG)& \
                PCI9111_FIFO_EMPTY_MASK) == 0)
 
-#define pci9111_ai_channel_set(channel) \
-       outb((channel)&PCI9111_CHANNEL_MASK, \
-               dev->iobase + PCI9111_REGISTER_AD_CHANNEL_CONTROL)
-
-#define pci9111_ai_channel_get() \
-       (inb(dev->iobase + PCI9111_REGISTER_AD_CHANNEL_READBACK) \
-               &PCI9111_CHANNEL_MASK)
-
 static const struct comedi_lrange pci9111_hr_ai_range = {
        5,
        {
@@ -594,10 +583,12 @@ static int pci9111_ai_do_cmd(struct comedi_device *dev,
        /*  TODO: handle the case of an external multiplexer */
 
        if (async_cmd->chanlist_len > 1) {
-               pci9111_ai_channel_set((async_cmd->chanlist_len) - 1);
+               outb(async_cmd->chanlist_len - 1,
+                       dev->iobase + PCI9111_AI_CHANNEL_REG);
                pci9111_autoscan_set(dev, true);
        } else {
-               pci9111_ai_channel_set(CR_CHAN(async_cmd->chanlist[0]));
+               outb(CR_CHAN(async_cmd->chanlist[0]),
+                       dev->iobase + PCI9111_AI_CHANNEL_REG);
                pci9111_autoscan_set(dev, false);
        }
 
@@ -857,7 +848,7 @@ static int pci9111_ai_insn_read(struct comedi_device *dev,
        int timeout;
        int i;
 
-       pci9111_ai_channel_set(chan);
+       outb(chan, dev->iobase + PCI9111_AI_CHANNEL_REG);
 
        current_range = inb(dev->iobase + PCI9111_RANGE_STATUS_REG);
        if ((current_range & PCI9111_RANGE_MASK) != range) {