staging: comedi: addi_apci_1516: define the watchdog control register
authorH Hartley Sweeten <hartleys@visionengravers.com>
Tue, 13 Nov 2012 20:37:31 +0000 (13:37 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 13 Nov 2012 23:12:45 +0000 (15:12 -0800)
Add defines for the bits in the watchdog control register and use
them to remove the "magic" numbers.

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/addi-data/hwdrv_apci1516.c

index f1b209d..a0d29be 100644 (file)
@@ -64,6 +64,8 @@ You should also find the complete GPL in the COPYING file accompanying this sour
 #define APCI1516_WDOG_RELOAD_LSB_REG           0x04
 #define APCI1516_WDOG_RELOAD_MSB_REG           0x06
 #define APCI1516_WDOG_CTRL_REG                 0x0c
+#define APCI1516_WDOG_CTRL_ENABLE              (1 << 0)
+#define APCI1516_WDOG_CTRL_SOFT_TRIG           (1 << 9)
 #define APCI1516_WDOG_STATUS_REG               0x10
 
 #define ADDIDATA_WATCHDOG                      2
@@ -182,10 +184,12 @@ static int i_APCI1516_StartStopWriteWatchdog(struct comedi_device *dev,
                outw(0x0, devpriv->i_IobaseAddon + APCI1516_WDOG_CTRL_REG);
                break;
        case 1:         /* start the watchdog */
-               outw(0x0001, devpriv->i_IobaseAddon + APCI1516_WDOG_CTRL_REG);
+               outw(APCI1516_WDOG_CTRL_ENABLE,
+                    devpriv->i_IobaseAddon + APCI1516_WDOG_CTRL_REG);
                break;
        case 2:         /* Software trigger */
-               outw(0x0201, devpriv->i_IobaseAddon + APCI1516_WDOG_CTRL_REG);
+               outw(APCI1516_WDOG_CTRL_ENABLE | APCI1516_WDOG_CTRL_SOFT_TRIG,
+                    devpriv->i_IobaseAddon + APCI1516_WDOG_CTRL_REG);
                break;
        default:
                printk("\nSpecified functionality does not exist\n");