linux-toradex.git/drivers/usb/class, branch tegra

usb: cdc-acm: fix packet loss issue

2012-07-26T14:41:00+00:00

There is race between acm_suspend and acm_read_bulk_callback. Host may
receive bulk transfer right before suspend. The packet will be discarded
if this urb is killed in acm_suspend. This patch checks the actual length
of urb and processes it in this case.

Bug 996268

Signed-off-by: Steve Lin 

Change-Id: Ief2b42708160b67903f976ec60da825d46c4720b
Reviewed-on: http://git-master/r/117135
(cherry picked from commit af3e96c987fbae8a135d1ff18872b9c32e09b67f)
Reviewed-on: http://git-master/r/118105
Reviewed-by: Automatic_Commit_Validation_User
Tested-by: Steve Lin 
Reviewed-by: Steve Lin

USB: cdc-acm: disconnect stuck issue and acm_tty_open crash

2012-04-16T21:35:32+00:00

acm_disconnect() should not kill the anchored URB because
they are already killed by stop_data_traffic().

Submit read URBs before control urb is sent because there is a
possibility of response coming immediately after ctrl is sent.

Bug 957744
Bug 961808

(cherry picked from commit 05c10cbe01f0275e5fe121d763692261c51987fc)
Reviewed-on: http://git-master/r/93673
Signed-off-by: Vinayak Pane 

Change-Id: I5597e239ec3722afb6b4c1cd5fbe228e30af2a2d
Reviewed-on: http://git-master/r/96585
Reviewed-by: Simone Willett 
Tested-by: Simone Willett

usb: cdc-acm: fix anchored urb leak

2012-04-16T21:11:20+00:00

Anchored urbs needs to put ref count after unanchoring
them at resume.

Bug 944250

Signed-off-by: Vinayak Pane 
Reviewed-on: http://git-master/r/86015
(cherry picked from commit eb88ad392bdc05ce216d6adeec54c4f4856ec78a)

Change-Id: I7e5413fb8866db9a15da57166402cb567f07391a
Reviewed-on: http://git-master/r/96333
Tested-by: Vinayak Pane 
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Steve Lin

arm : tegra : comm: AT command loss WAR

2012-03-06T01:41:25+00:00

Workaround to avoid make cdc-acm susp_count
to negative.

Bug 935834

Change-Id: I251049537e21662de329f11ecbad0ce15abb1037
Signed-off-by: Seshendra Gadagottu
Reviewed-on: http://git-master/r/84288
(cherry picked from commit d056c04d453bc641e856a61251e7d0aa2dcce73b)
Reviewed-on: http://git-master/r/87505
Reviewed-by: Simone Willett 
Tested-by: Simone Willett

usb: cdc-acm: Add acm handle validity check

2012-03-06T01:27:02+00:00

Added check for acm handle validity before doing any action
in acm_suspend, acm_resume and acm_reset_resume functions.

Bug 939237

Change-Id: Idc5d7db6bd405056a90b85009825ccbd03547757
Signed-off-by: Seshendra Gadagottu
Reviewed-on: http://git-master/r/83413
(cherry picked from commit 1dd9736cd2df12c0315a44c95010cb64eee04050)
Reviewed-on: http://git-master/r/87504
Reviewed-by: Steve Lin

usb: cdc-acm: Fix compilation warning in cdc-acm driver.

2012-02-21T17:11:49+00:00

Bug 934177

Change-Id: If101278cf553bd92ad98a9485cedf0c3bf1df39a
Signed-off-by: Steve Lin 
Reviewed-on: http://git-master/r/83913

USB: cdc-wdm: call wake_up_all to allow driver to shutdown on device removal

2012-02-10T00:04:16+00:00

commit 62aaf24dc125d7c55c93e313d15611f152b030c7 upstream.

wdm_disconnect() waits for the mutex held by wdm_read() before
calling wake_up_all().  This causes a deadlock, preventing device removal
to complete.  Do the wake_up_all() before we start waiting for the locks.

Signed-off-by: Bjørn Mork 
Cc: Oliver Neukum 
Cc: stable 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Varun Wadekar 
Change-Id: Ic92d101f26698708a10ba276eff70825d19d6c89
Reviewed-on: http://git-master/r/79686
Reviewed-by: Simone Willett 
Tested-by: Simone Willett

USB: cdc-wdm: use two mutexes to allow simultaneous read and write

2012-02-09T23:57:49+00:00

commit e8537bd2c4f325a4796da33564ddcef9489b7feb upstream.

using a separate read and write mutex for locking is sufficient to make the
driver accept simultaneous read and write. This improves useability a lot.

Signed-off-by: Bjørn Mork 
Cc: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Varun Wadekar 
Change-Id: I10ed5f0e41b379f0fef8a22abde46d4231162ff9
Reviewed-on: http://git-master/r/79672
Reviewed-by: Simone Willett 
Tested-by: Simone Willett

USB: cdc-wdm: updating desc->length must be protected by spin_lock

2012-02-09T23:52:53+00:00

commit c428b70c1e115c5649707a602742e34130d19428 upstream.

wdm_in_callback() will also touch this field, so we cannot change it without locking

Signed-off-by: Bjørn Mork 
Acked-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Varun Wadekar 
Change-Id: I7eb2367de1a74be1ac68578354113154595d02cc
Reviewed-on: http://git-master/r/79671
Reviewed-by: Simone Willett 
Tested-by: Simone Willett

USB: cdc-wdm: better allocate a buffer that is at least as big as we tell the USB core

2012-02-08T22:41:05+00:00

commit 655e247daf52b202a6c2d0f8a06dd2051e756ce4 upstream.

As it turns out, there was a mismatch between the allocated inbuf size
(desc->bMaxPacketSize0, typically something like 64) and the length we
specified in the URB (desc->wMaxCommand, typically something like 2048)

Signed-off-by: Bjørn Mork 
Cc: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Varun Wadekar 

Change-Id: I00872baa6896dfdd076fdcfc57956f1420973a7f
Reviewed-on: http://git-master/r/79687
Reviewed-by: Simone Willett 
Tested-by: Simone Willett