diff options
| author | Ian Abbott <abbotti@mev.co.uk> | 2017-03-08 18:44:32 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-12 14:57:00 +0100 |
| commit | 6541964dcd0c4ebffb55f0fc3e79b3794c55a869 (patch) | |
| tree | 8444ebd7f2da87aa7db516dc0d00c2ba534b4d41 /drivers/staging/comedi | |
| parent | 8abde9acdfc82aacbfd96e65fe416c29c4856500 (diff) | |
staging: comedi: jr3_pci: fix initial range[8] max value
`jr3_pci_alloc_spriv()` initializes `spriv->range[8]` to use a maximum
value of 65536, but that will be overwritten with 65535 at a later time
by `jr3_pci_poll_subdevice()` once the "set full scales" command is
complete. The initial setting looks like a mistake. This range is only
associated with a couple of dummy channels (channels 56 and 57) to read
back the model number and serial number, so no user code should be
attempting to convert those numbers to physical units. Just change the
initial value to 65535 to match the final value.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/comedi')
| -rw-r--r-- | drivers/staging/comedi/drivers/jr3_pci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/comedi/drivers/jr3_pci.c b/drivers/staging/comedi/drivers/jr3_pci.c index a21344e1a4c3..e750be39103b 100644 --- a/drivers/staging/comedi/drivers/jr3_pci.c +++ b/drivers/staging/comedi/drivers/jr3_pci.c @@ -653,7 +653,7 @@ jr3_pci_alloc_spriv(struct comedi_device *dev, struct comedi_subdevice *s) } spriv->range[8].l.length = 1; spriv->range[8].l.range[0].min = 0; - spriv->range[8].l.range[0].max = 65536; + spriv->range[8].l.range[0].max = 65535; spriv->range_table_list[56] = &spriv->range[8].l; spriv->range_table_list[57] = &spriv->range[8].l; |