]> git.ipfire.org Git - people/arne_f/kernel.git/commit - drivers/usb/serial/ftdi_sio.c
USB: serial: ftdi_sio: only allow valid latency timer values
authorIan Abbott <abbotti@mev.co.uk>
Wed, 8 Mar 2017 16:02:59 +0000 (16:02 +0000)
committerJohan Hovold <johan@kernel.org>
Mon, 13 Mar 2017 15:55:40 +0000 (16:55 +0100)
commitdb9240662a44e1b7781c8dbf35631bcec7a369c6
treeb1309bbd05ebb4bcc9288a109d24038243f107c2
parent7e1e6ceda3f07a40445ae1c4ac5549a899d5e252
USB: serial: ftdi_sio: only allow valid latency timer values

Valid latency timer values are between 1 ms and 255 ms in 1 ms steps.
The store function for the "latency_timer" device attribute currently
allows any value, although only the lower 16 bits will be sent to the
device, and the device only stores the lower 8 bits.  The hardware
appears to accept the (invalid) value 0 and treats it the same as 1
(resulting in a latency of 1 ms).

Change the latency_timer_store() function to accept only the values 0 to
255, returning an error -EINVAL for out-of-range values.  Call
kstrtou8() to parse the integer instead of the obsolete
simple_strtoul().

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/ftdi_sio.c