diff options
| author | Johan Hovold <johan@kernel.org> | 2019-11-07 14:21:18 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-12-01 09:17:43 +0100 |
| commit | abbda35d6a6e9449fcf8c40a8a79cfc208a3a127 (patch) | |
| tree | 98695254cb80fad86430cf51331699ebe0c9b427 /drivers/usb | |
| parent | 84743898d60bd9ac8a3ad0cac53aa01d508175bb (diff) | |
| download | linux-stable-abbda35d6a6e9449fcf8c40a8a79cfc208a3a127.tar.gz linux-stable-abbda35d6a6e9449fcf8c40a8a79cfc208a3a127.tar.bz2 linux-stable-abbda35d6a6e9449fcf8c40a8a79cfc208a3a127.zip | |
USB: serial: mos7720: fix remote wakeup
commit ea422312a462696093b5db59d294439796cba4ad upstream.
The driver was setting the device remote-wakeup feature during probe in
violation of the USB specification (which says it should only be set
just prior to suspending the device). This could potentially waste
power during suspend as well as lead to spurious wakeups.
Note that USB core would clear the remote-wakeup feature at first
resume.
Fixes: 0f64478cbc7a ("USB: add USB serial mos7720 driver")
Cc: stable <stable@vger.kernel.org> # 2.6.19
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb')
| -rw-r--r-- | drivers/usb/serial/mos7720.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/usb/serial/mos7720.c b/drivers/usb/serial/mos7720.c index e8f275a0326d..c0232b67a40f 100644 --- a/drivers/usb/serial/mos7720.c +++ b/drivers/usb/serial/mos7720.c @@ -1894,10 +1894,6 @@ static int mos7720_startup(struct usb_serial *serial) product = le16_to_cpu(serial->dev->descriptor.idProduct); dev = serial->dev; - /* setting configuration feature to one */ - usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0), - (__u8)0x03, 0x00, 0x01, 0x00, NULL, 0x00, 5000); - if (product == MOSCHIP_DEVICE_ID_7715) { struct urb *urb = serial->port[0]->interrupt_in_urb; |