staging: comedi: pcmuio: tidy up switch_page()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 18 Jun 2013 20:21:13 +0000 (13:21 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 24 Jun 2013 22:50:38 +0000 (15:50 -0700)
All the I/O ports are left unlocked in the driver so the 'pagelock'
in the private data is not necessary. The paranoia sanity checks
are also unnecessary, Remove them.

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/pcmuio.c

index 9fb75a4..3c98a00 100644 (file)
@@ -182,8 +182,6 @@ struct pcmuio_subdev_private {
 
 struct pcmuio_private {
        struct {
-               /* current page and lock */
-               unsigned char pagelock;
                /* shadow of POLx registers */
                unsigned char pol[NUM_PAGED_REGS];
                /* shadow of ENABx registers */
@@ -312,19 +310,7 @@ static int pcmuio_dio_insn_config(struct comedi_device *dev,
 
 static void switch_page(struct comedi_device *dev, int asic, int page)
 {
-       const struct pcmuio_board *board = comedi_board(dev);
-       struct pcmuio_private *devpriv = dev->private;
-
-       if (asic < 0 || asic >= board->num_asics)
-               return;         /* paranoia */
-       if (page < 0 || page >= NUM_PAGES)
-               return;         /* more paranoia */
-
-       devpriv->asics[asic].pagelock &= ~PCMUIO_PAGE_MASK;
-       devpriv->asics[asic].pagelock |= PCMUIO_PAGE(page);
-
-       /* now write out the shadow register */
-       outb(devpriv->asics[asic].pagelock,
+       outb(PCMUIO_PAGE(page),
             dev->iobase + ASIC_IOSIZE * asic + PCMUIO_PAGE_LOCK_REG);
 }