Commit cba9d4aa authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: das_08: remove the cached 8254 'mode'

The driver initially sets all the cached 'mode' values for the
8254 timers to (I8254_MODE0 | I8254_BINARY). It then sets the
timers to that 'mode'. Configuring the counters with the comedi
INSN_CONFIG_SET_COUNTER_MODE updates the 'mode' and then sets
the timers to the 'mode'. The cached value is never read or
used other than for storage.

Just remove the 'mode' usage as it serves no purpose.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent afdd107c
...@@ -435,10 +435,11 @@ das08ao_ao_winsn(struct comedi_device *dev, struct comedi_subdevice *s, ...@@ -435,10 +435,11 @@ das08ao_ao_winsn(struct comedi_device *dev, struct comedi_subdevice *s,
static void i8254_initialize(struct i8254_struct *st) static void i8254_initialize(struct i8254_struct *st)
{ {
unsigned int mode = I8254_MODE0 | I8254_BINARY;
int i; int i;
for (i = 0; i < 3; ++i) for (i = 0; i < 3; ++i)
i8254_set_mode(st->iobase, 0, i, st->mode[i]); i8254_set_mode(st->iobase, 0, i, mode);
} }
static int das08_counter_read(struct comedi_device *dev, static int das08_counter_read(struct comedi_device *dev,
...@@ -478,7 +479,6 @@ static int das08_counter_config(struct comedi_device *dev, ...@@ -478,7 +479,6 @@ static int das08_counter_config(struct comedi_device *dev,
switch (data[0]) { switch (data[0]) {
case INSN_CONFIG_SET_COUNTER_MODE: case INSN_CONFIG_SET_COUNTER_MODE:
st->mode[chan] = data[1];
i8254_set_mode(st->iobase, 0, chan, data[1]); i8254_set_mode(st->iobase, 0, chan, data[1]);
break; break;
case INSN_CONFIG_8254_READ_STATUS: case INSN_CONFIG_8254_READ_STATUS:
...@@ -852,9 +852,6 @@ int das08_common_attach(struct comedi_device *dev, unsigned long iobase) ...@@ -852,9 +852,6 @@ int das08_common_attach(struct comedi_device *dev, unsigned long iobase)
devpriv->i8254.logic2phys[1] = 1; devpriv->i8254.logic2phys[1] = 1;
devpriv->i8254.logic2phys[2] = 2; devpriv->i8254.logic2phys[2] = 2;
devpriv->i8254.iobase = iobase + thisboard->i8254_offset; devpriv->i8254.iobase = iobase + thisboard->i8254_offset;
devpriv->i8254.mode[0] =
devpriv->i8254.mode[1] =
devpriv->i8254.mode[2] = I8254_MODE0 | I8254_BINARY;
i8254_initialize(&devpriv->i8254); i8254_initialize(&devpriv->i8254);
} else { } else {
s->type = COMEDI_SUBD_UNUSED; s->type = COMEDI_SUBD_UNUSED;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment