diff options
author | Thomas Pugliese <thomas.pugliese@gmail.com> | 2013-10-23 15:44:29 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-10-29 19:44:49 -0400 |
commit | f74b75e7f920c700636cccca669c7d16d12e9202 (patch) | |
tree | 65428b3ee5c3303a7c0ccce2a34a326fd67931b6 /drivers/usb/wusbcore | |
parent | f07ddb9ef5c25c1044ab5b6509241320fb5e831a (diff) |
usb: wusbcore: change WA_SEGS_MAX to a legal value
change WA_SEGS_MAX to a number that is legal according to the WUSB
spec.
Signed-off-by: Thomas Pugliese <thomas.pugliese@gmail.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/wusbcore')
-rw-r--r-- | drivers/usb/wusbcore/wa-xfer.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/usb/wusbcore/wa-xfer.c b/drivers/usb/wusbcore/wa-xfer.c index 090ac308c756..ed5abe87b049 100644 --- a/drivers/usb/wusbcore/wa-xfer.c +++ b/drivers/usb/wusbcore/wa-xfer.c | |||
@@ -91,7 +91,8 @@ | |||
91 | #include "wusbhc.h" | 91 | #include "wusbhc.h" |
92 | 92 | ||
93 | enum { | 93 | enum { |
94 | WA_SEGS_MAX = 255, | 94 | /* [WUSB] section 8.3.3 allocates 7 bits for the segment index. */ |
95 | WA_SEGS_MAX = 128, | ||
95 | }; | 96 | }; |
96 | 97 | ||
97 | enum wa_seg_status { | 98 | enum wa_seg_status { |
@@ -588,7 +589,7 @@ static ssize_t __wa_xfer_setup_sizes(struct wa_xfer *xfer, | |||
588 | xfer->segs = 1; | 589 | xfer->segs = 1; |
589 | } | 590 | } |
590 | 591 | ||
591 | if (xfer->segs >= WA_SEGS_MAX) { | 592 | if (xfer->segs > WA_SEGS_MAX) { |
592 | dev_err(dev, "BUG? oops, number of segments %zu bigger than %d\n", | 593 | dev_err(dev, "BUG? oops, number of segments %zu bigger than %d\n", |
593 | (urb->transfer_buffer_length/xfer->seg_size), | 594 | (urb->transfer_buffer_length/xfer->seg_size), |
594 | WA_SEGS_MAX); | 595 | WA_SEGS_MAX); |