Skip to content

Commit

Permalink
staging: comedi: addi_apci_2032: fix i_APCI2032_ReadWatchdog()
Browse files Browse the repository at this point in the history
This function is used by the watchdog subdevice to read the status of
the watchdog. Rename the CamelCase function to apci2032_wdog_insn_read
and fix the function to return the status value insn->n times like
the comedi core expects.

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>
  • Loading branch information
H Hartley Sweeten authored and Greg Kroah-Hartman committed Nov 30, 2012
1 parent 7180eb3 commit 7b5dd1c
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions drivers/staging/comedi/drivers/addi_apci_2032.c
Original file line number Diff line number Diff line change
Expand Up @@ -156,12 +156,16 @@ static int i_APCI2032_StartStopWriteWatchdog(struct comedi_device *dev,
return insn->n;
}

static int i_APCI2032_ReadWatchdog(struct comedi_device *dev,
static int apci1516_wdog_insn_read(struct comedi_device *dev,
struct comedi_subdevice *s,
struct comedi_insn *insn,
unsigned int *data)
{
data[0] = inl(dev->iobase + APCI2032_WDOG_STATUS_REG) & 0x1;
int i;

for (i = 0; i < insn->n; i++)
data[i] = inl(dev->iobase + APCI2032_WDOG_STATUS_REG);

return insn->n;
}

Expand Down Expand Up @@ -268,7 +272,7 @@ static int apci2032_auto_attach(struct comedi_device *dev,
s->len_chanlist = 1;
s->range_table = &range_digital;
s->insn_write = i_APCI2032_StartStopWriteWatchdog;
s->insn_read = i_APCI2032_ReadWatchdog;
s->insn_read = apci1516_wdog_insn_read;
s->insn_config = i_APCI2032_ConfigWatchdog;

apci2032_reset(dev);
Expand Down

0 comments on commit 7b5dd1c

Please sign in to comment.