summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorBjørn Mork <bjorn@mork.no>2014-03-17 16:25:18 +0100
committerDavid S. Miller <davem@davemloft.net>2014-03-18 15:32:32 -0400
commitff0992e9036e9810e7cd45234fa32ca1e79750e2 (patch)
tree92187e84bf62c3d9f6c8f474427dc0b94cd90af3 /include
parente367c2d03dba4c9bcafad24688fadb79dd95b218 (diff)
downloadlinux-ff0992e9036e9810e7cd45234fa32ca1e79750e2.tar.gz
linux-ff0992e9036e9810e7cd45234fa32ca1e79750e2.tar.bz2
linux-ff0992e9036e9810e7cd45234fa32ca1e79750e2.zip
net: cdc_ncm: fix control message ordering
This is a context modified revert of commit 6a9612e2cb22 ("net: cdc_ncm: remove ncm_parm field") which introduced a NCM specification violation, causing setup errors for some devices. These errors resulted in the device and host disagreeing about shared settings, with complete failure to communicate as the end result. The NCM specification require that many of the NCM specific control reuests are sent only while the NCM Data Interface is in alternate setting 0. Reverting the commit ensures that we follow this requirement. Fixes: 6a9612e2cb22 ("net: cdc_ncm: remove ncm_parm field") Reported-and-tested-by: Pasi Kärkkäinen <pasik@iki.fi> Reported-by: Thomas Schäfer <tschaefer@t-online.de> Signed-off-by: Bjørn Mork <bjorn@mork.no> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
-rw-r--r--include/linux/usb/cdc_ncm.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/usb/cdc_ncm.h b/include/linux/usb/cdc_ncm.h
index c3fa80745996..2c14d9cdd57a 100644
--- a/include/linux/usb/cdc_ncm.h
+++ b/include/linux/usb/cdc_ncm.h
@@ -88,6 +88,7 @@
#define cdc_ncm_data_intf_is_mbim(x) ((x)->desc.bInterfaceProtocol == USB_CDC_MBIM_PROTO_NTB)
struct cdc_ncm_ctx {
+ struct usb_cdc_ncm_ntb_parameters ncm_parm;
struct hrtimer tx_timer;
struct tasklet_struct bh;