diff options
author | Sarah Sharp <sarah.a.sharp@linux.intel.com> | 2008-09-29 13:58:35 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-10-17 17:41:02 -0400 |
commit | c0f082c5367a02e8493d779e16ad336167e14718 (patch) | |
tree | 173018ddaac12eb12a7d001e9802f1346d650364 /drivers/usb/misc | |
parent | ce5dee50edc8b1ac8028c17040d40a78c4b33232 (diff) |
USB: ftdi-elan: Always pass usb_bulk_msg() a timeout in milliseconds.
The kernel doc for usb_bulk_msg() says the timeout for a bulk message should be
specified in milliseconds. The ftdi-elan driver converts milliseconds to
jiffies before passing the timeout to usb_bulk_msg(). This is mostly harmless,
since it will just lead to very long timeouts, but was obviously not the intent
of the original author.
Signed-off-by: Sarah Sharp <sarah.a.sharp@intel.com>
Acked-by: Tony Olech <tony.olech@elandigitalsystems.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/misc')
-rw-r--r-- | drivers/usb/misc/ftdi-elan.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/usb/misc/ftdi-elan.c b/drivers/usb/misc/ftdi-elan.c index 97c280971532..79a7668ef264 100644 --- a/drivers/usb/misc/ftdi-elan.c +++ b/drivers/usb/misc/ftdi-elan.c | |||
@@ -698,7 +698,7 @@ static ssize_t ftdi_elan_read(struct file *file, char __user *buffer, | |||
698 | int retval = usb_bulk_msg(ftdi->udev, | 698 | int retval = usb_bulk_msg(ftdi->udev, |
699 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), | 699 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), |
700 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, | 700 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, |
701 | &packet_bytes, msecs_to_jiffies(50)); | 701 | &packet_bytes, 50); |
702 | if (packet_bytes > 2) { | 702 | if (packet_bytes > 2) { |
703 | ftdi->bulk_in_left = packet_bytes - 2; | 703 | ftdi->bulk_in_left = packet_bytes - 2; |
704 | ftdi->bulk_in_last = 1; | 704 | ftdi->bulk_in_last = 1; |
@@ -960,7 +960,7 @@ static int ftdi_elan_respond_engine(struct usb_ftdi *ftdi) | |||
960 | int retval = usb_bulk_msg(ftdi->udev, | 960 | int retval = usb_bulk_msg(ftdi->udev, |
961 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), | 961 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), |
962 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, | 962 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, |
963 | &packet_bytes, msecs_to_jiffies(500)); | 963 | &packet_bytes, 500); |
964 | char diag[30 *3 + 4]; | 964 | char diag[30 *3 + 4]; |
965 | char *d = diag; | 965 | char *d = diag; |
966 | int m = packet_bytes; | 966 | int m = packet_bytes; |
@@ -1880,7 +1880,7 @@ static int ftdi_elan_flush_input_fifo(struct usb_ftdi *ftdi) | |||
1880 | int retval = usb_bulk_msg(ftdi->udev, | 1880 | int retval = usb_bulk_msg(ftdi->udev, |
1881 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), | 1881 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), |
1882 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, | 1882 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, |
1883 | &packet_bytes, msecs_to_jiffies(100)); | 1883 | &packet_bytes, 100); |
1884 | if (packet_bytes > 2) { | 1884 | if (packet_bytes > 2) { |
1885 | char diag[30 *3 + 4]; | 1885 | char diag[30 *3 + 4]; |
1886 | char *d = diag; | 1886 | char *d = diag; |
@@ -2067,7 +2067,7 @@ static int ftdi_elan_synchronize(struct usb_ftdi *ftdi) | |||
2067 | usb_rcvbulkpipe(ftdi->udev, | 2067 | usb_rcvbulkpipe(ftdi->udev, |
2068 | ftdi->bulk_in_endpointAddr), | 2068 | ftdi->bulk_in_endpointAddr), |
2069 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, | 2069 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, |
2070 | &packet_bytes, msecs_to_jiffies(500)); | 2070 | &packet_bytes, 500); |
2071 | if (packet_bytes > 2) { | 2071 | if (packet_bytes > 2) { |
2072 | char diag[30 *3 + 4]; | 2072 | char diag[30 *3 + 4]; |
2073 | char *d = diag; | 2073 | char *d = diag; |
@@ -2176,7 +2176,7 @@ static int ftdi_elan_stuck_waiting(struct usb_ftdi *ftdi) | |||
2176 | int retval = usb_bulk_msg(ftdi->udev, | 2176 | int retval = usb_bulk_msg(ftdi->udev, |
2177 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), | 2177 | usb_rcvbulkpipe(ftdi->udev, ftdi->bulk_in_endpointAddr), |
2178 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, | 2178 | ftdi->bulk_in_buffer, ftdi->bulk_in_size, |
2179 | &packet_bytes, msecs_to_jiffies(1000)); | 2179 | &packet_bytes, 1000); |
2180 | if (packet_bytes > 2) { | 2180 | if (packet_bytes > 2) { |
2181 | char diag[30 *3 + 4]; | 2181 | char diag[30 *3 + 4]; |
2182 | char *d = diag; | 2182 | char *d = diag; |