For aesthetics, rename the analog input register defines and convert the offsets to hex. Signed-off-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> Cc: Ian Abbott <abbotti@xxxxxxxxx> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/staging/comedi/drivers/pcl812.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/drivers/staging/comedi/drivers/pcl812.c b/drivers/staging/comedi/drivers/pcl812.c index 9fb3e84..7d13a99 100644 --- a/drivers/staging/comedi/drivers/pcl812.c +++ b/drivers/staging/comedi/drivers/pcl812.c @@ -132,8 +132,9 @@ #define boardA821 9 /* PGH, PGL, PGL/NDA versions */ #define PCL812_TIMER_BASE 0x00 -#define PCL812_AD_LO 4 -#define PCL812_AD_HI 5 +#define PCL812_AI_LSB_REG 0x04 +#define PCL812_AI_MSB_REG 0x05 +#define PCL812_AI_MSB_DRDY (1 << 4) #define PCL812_AO_LSB_REG(x) (0x04 + ((x) * 2)) #define PCL812_AO_MSB_REG(x) (0x05 + ((x) * 2)) #define PCL812_DI_LSB_REG 0x06 @@ -147,8 +148,6 @@ #define PCL812_DO_LSB_REG 0x0d #define PCL812_DO_MSB_REG 0x0e -#define PCL812_DRDY 0x10 /* =0 data ready */ - #define ACL8216_STATUS 8 /* 5. bit signalize data ready */ #define ACL8216_DRDY 0x20 /* =0 data ready */ @@ -654,8 +653,8 @@ static unsigned int pcl812_ai_get_sample(struct comedi_device *dev, { unsigned int val; - val = inb(dev->iobase + PCL812_AD_HI) << 8; - val |= inb(dev->iobase + PCL812_AD_LO); + val = inb(dev->iobase + PCL812_AI_MSB_REG) << 8; + val |= inb(dev->iobase + PCL812_AI_LSB_REG); return val & s->maxdata; } @@ -672,8 +671,8 @@ static int pcl812_ai_eoc(struct comedi_device *dev, if ((status & ACL8216_DRDY) == 0) return 0; } else { - status = inb(dev->iobase + PCL812_AD_HI); - if ((status & PCL812_DRDY) == 0) + status = inb(dev->iobase + PCL812_AI_MSB_REG); + if ((status & PCL812_AI_MSB_DRDY) == 0) return 0; } return -EBUSY; -- 1.8.5.2 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel