[PATCH 07/15] staging: comedi: ni_labpc: tidy up labpc_8255_mmio()

H Hartley Sweeten hsweeten at visionengravers.com
Fri Aug 1 21:18:18 UTC 2014


The 8255 driver (*io) callback now includes the comedi_device pointer.

Instead of passing the (cast) pointer to subdev_8255_init(), pass the
'iobase' of the 8255 registers (DIO_BASE_REG).

Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
Cc: Ian Abbott <abbotti at mev.co.uk>
Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
---
 drivers/staging/comedi/drivers/ni_labpc.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/comedi/drivers/ni_labpc.c
index fa108b9..15ead27 100644
--- a/drivers/staging/comedi/drivers/ni_labpc.c
+++ b/drivers/staging/comedi/drivers/ni_labpc.c
@@ -1035,17 +1035,15 @@ static int labpc_ao_insn_read(struct comedi_device *dev,
 	return 1;
 }
 
-static int labpc_8255_mmio(struct comedi_device *cdev,
-			   int dir, int port, int data, unsigned long arg)
+static int labpc_8255_mmio(struct comedi_device *dev,
+			   int dir, int port, int data, unsigned long iobase)
 {
-	struct comedi_device *dev = (struct comedi_device *)arg;
-
 	if (dir) {
-		writeb(data, dev->mmio + DIO_BASE_REG + port);
+		writeb(data, dev->mmio + iobase + port);
 		return 0;
 	}
 
-	return readb(dev->mmio + DIO_BASE_REG + port);
+	return readb(dev->mmio + iobase + port);
 }
 
 /* lowlevel write to eeprom/dac */
@@ -1405,7 +1403,7 @@ int labpc_common_attach(struct comedi_device *dev,
 	s = &dev->subdevices[2];
 	if (dev->mmio) {
 		ret = subdev_8255_init(dev, s, labpc_8255_mmio,
-				       (unsigned long)dev);
+				       DIO_BASE_REG);
 	} else {
 		ret = subdev_8255_init(dev, s, NULL,
 				       dev->iobase + DIO_BASE_REG);
-- 
2.0.3



More information about the devel mailing list