]> git.baikalelectronics.ru Git - kernel.git/commit
usb: gadget: epautoconf: do not change number of streams
authorFelipe Balbi <balbi@ti.com>
Mon, 19 Dec 2011 11:45:01 +0000 (13:45 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 19 Dec 2011 11:45:01 +0000 (13:45 +0200)
commit81e8907c231334e634a21a8a63c7e85379f55509
treefb08da5707bb6498a42eef28e10200fd79356e68
parentbb061ca9c43a370e9116dccf9bff5c0446400541
usb: gadget: epautoconf: do not change number of streams

We should not change gadget driver's descriptors just
because we think it's right to do so.

There are several of reasons which would support this
statement but it suffices to say that this was probably
never tested because it updates bmAttributes without
asking the driver if it's ok to do so.

This means that e.g. on UASP gadget it would enable
stream support even for the command endpoint which must
not have stream support enabled.

In fact, this change is fixing the bug introduced by
commit 9bc60ed (usb: gadget: add streams support to
the gadget framework) which was caught when testing
UASP gadget with dwc3 driver.

Cc: Tatyana Brokhman <tlinder@codeaurora.org>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/epautoconf.c