[PATCH 11/21] staging: comedi: c6xdigio: define the register map
H Hartley Sweeten
hsweeten at visionengravers.com
Mon Mar 10 21:07:32 UTC 2014
The register map was already defined but it was not being used. Rename
to defines and use them in the code.
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/c6xdigio.c | 29 +++++++++++++++--------------
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/comedi/drivers/c6xdigio.c b/drivers/staging/comedi/drivers/c6xdigio.c
index 2189e5b..a05196b 100644
--- a/drivers/staging/comedi/drivers/c6xdigio.c
+++ b/drivers/staging/comedi/drivers/c6xdigio.c
@@ -44,11 +44,12 @@ http://robot0.ge.uiuc.edu/~spong/mecha/
#include "../comedidev.h"
/*
- * port offsets
+ * Register I/O map
*/
-#define C6XDIGIO_PARALLEL_DATA 0
-#define C6XDIGIO_PARALLEL_STATUS 1
-#define C6XDIGIO_PARALLEL_CONTROL 2
+#define C6XDIGIO_DATA_REG 0x00
+#define C6XDIGIO_STATUS_REG 0x01
+#define C6XDIGIO_CTRL_REG 0x02
+
struct pwmbitstype {
unsigned sb0:2;
unsigned sb1:2;
@@ -83,7 +84,7 @@ static int c6xdigio_chk_status(struct comedi_device *dev, unsigned long context)
int timeout = 0;
do {
- status = inb(dev->iobase + 1);
+ status = inb(dev->iobase + C6XDIGIO_STATUS_REG);
if ((status & 0x80) != context)
return 0;
timeout++;
@@ -95,7 +96,7 @@ static int c6xdigio_chk_status(struct comedi_device *dev, unsigned long context)
static int c6xdigio_write_data(struct comedi_device *dev,
unsigned int val, unsigned int status)
{
- outb_p(val, dev->iobase);
+ outb_p(val, dev->iobase + C6XDIGIO_DATA_REG);
return c6xdigio_chk_status(dev, status);
}
@@ -146,28 +147,28 @@ static int c6xdigio_encoder_read(struct comedi_device *dev,
c6xdigio_write_data(dev, ppcmd, 0x00);
- enc.bits.sb0 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb0 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd + 0x4, 0x80);
- enc.bits.sb1 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb1 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd, 0x00);
- enc.bits.sb2 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb2 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd + 0x4, 0x80);
- enc.bits.sb3 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb3 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd, 0x00);
- enc.bits.sb4 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb4 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd + 0x4, 0x80);
- enc.bits.sb5 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb5 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd, 0x00);
- enc.bits.sb6 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb6 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd + 0x4, 0x80);
- enc.bits.sb7 = ((inb(dev->iobase + 1) >> 3) & 0x7);
+ enc.bits.sb7 = ((inb(dev->iobase + C6XDIGIO_STATUS_REG) >> 3) & 0x7);
c6xdigio_write_data(dev, ppcmd, 0x00);
c6xdigio_write_data(dev, 0x00, 0x80);
--
1.8.5.2
More information about the devel
mailing list