linux-toradex.git/drivers/usb/gadget/dummy_hcd.c, branch v3.2.64

usb: gadget: dummy: fix enumeration with g_multi

2013-02-06T04:33:18+00:00

commit 1d16638e3b9cc195bac18a8fcbca748f33c1bc24 upstream.

If we do have endpoints named like "ep-a" then bEndpointAddress is
counted internally by the gadget framework.

If we do have endpoints named like "ep-1" then bEndpointAddress is
assigned from the digit after "ep-".

If we do have both, then it is likely that after we used up the
"generic" endpoints we will use the digits and thus assign one
bEndpointAddress to multiple endpoints.

This theory can be proofed by using the completely enabled g_multi.
Without this patch, the mass storage won't enumerate and times out
because it shares endpoints with RNDIS.

This patch also adds fills up the endpoints list so we have in total
endpoints 1 to 15 in + out available while some of them are restricted
to certain types like BULK or ISO. Without this change the nokia gadget
won't load because the system does not provide enough (BULK) endpoints
but it did before ep-a - ep-f were removed.

Signed-off-by: Sebastian Andrzej Siewior 
Acked-by: Alan Stern 
Signed-off-by: Felipe Balbi 
Signed-off-by: Ben Hutchings

usb: gadget: dummy_hcd: fixup error probe path

2012-10-10T02:31:14+00:00

commit 1b68a4ca2d038addb7314211d122fb6d7002b38b upstream.

If USB2 host controller probes fine but USB3 does not then we don't
remove the USB controller properly and lock up the system while the HUB
code will try to enumerate the USB2 controller and access memory which
is no longer available in case the dummy_hcd was compiled as a module.

This is a problem since 448b6eb1 ("USB: Make sure to fetch the BOS desc
for roothubs.) if used in USB3 mode because dummy does not provide this
descriptor and explodes later.

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Ben Hutchings

usb: gadget: dummy: do not call pullup() on udc_stop()

2012-05-11T12:14:42+00:00

commit 15b120d67019d691e4389372967332d74a80522a upstream.

pullup() is already called properly by udc-core.c and
there's no need to call it from udc_stop(), in fact that
will cause issues.

Reviewed-by: Alexander Shishkin 
Acked-by: Alan Stern 
Signed-off-by: Felipe Balbi 
Signed-off-by: Ben Hutchings

usb gadget: clean up FSF boilerplate text

2011-09-09T23:00:09+00:00

remove the following two paragraphs as they are not needed:

This program is distributed in the hope that it will be useful, but
WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
License for more details.

You should have received a copy of the GNU General Public License along with
this program; if not, write to the Free Software Foundation, Inc.,59
Temple Place - Suite 330, Boston, MA  02111-1307, USA.

Signed-off-by: Klaus Schwarzkopf 
Signed-off-by: Greg Kroah-Hartman

USB: use usb_endpoint_maxp() instead of le16_to_cpu()

2011-08-23T16:47:40+00:00

Now ${LINUX}/drivers/usb/* can use usb_endpoint_maxp(desc) to get maximum packet size
instead of le16_to_cpu(desc->wMaxPacketSize).
This patch fix it up

Cc: Armin Fuerst 
Cc: Pavel Machek 
Cc: Johannes Erdfelt 
Cc: Vojtech Pavlik 
Cc: Oliver Neukum 
Cc: David Kubicek 
Cc: Johan Hovold 
Cc: Brad Hards 
Acked-by: Felipe Balbi 
Cc: Sebastian Andrzej Siewior 
Cc: Thomas Dahlmann 
Cc: David Brownell 
Cc: David Lopo 
Cc: Alan Stern 
Cc: Michal Nazarewicz 
Cc: Xie Xiaobo 
Cc: Li Yang 
Cc: Jiang Bo 
Cc: Yuan-hsin Chen 
Cc: Darius Augulis 
Cc: Xiaochen Shen 
Cc: Yoshihiro Shimoda 
Cc: OKI SEMICONDUCTOR, 
Cc: Robert Jarzmik 
Cc: Ben Dooks 
Cc: Thomas Abraham 
Cc: Herbert Pötzl 
Cc: Arnaud Patard 
Cc: Roman Weissgaerber 
Acked-by: Sarah Sharp 
Cc: Tony Olech 
Cc: Florian Floe Echtler 
Cc: Christian Lucht 
Cc: Juergen Stuber 
Cc: Georges Toth 
Cc: Bill Ryder 
Cc: Kuba Ober 
Cc: Inaky Perez-Gonzalez 
Signed-off-by: Kuninori Morimoto 
Signed-off-by: Greg Kroah-Hartman

usb: gadget: dummy_hcd: convert to new-style udc-probe

2011-07-01T21:31:14+00:00

bind() and pull is moved to udc core, call callbacks are verified by the
upper layer.

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Greg Kroah-Hartman

usb: gadget: dummy_hcd: move selection of speed into ->pullup()

2011-07-01T21:31:13+00:00

The configuration is static however we only know the speed after we have
connected with the other side.

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Greg Kroah-Hartman

usb: gadget: dummy_hcd: init is_otg in init_dummy_udc_hw()

2011-07-01T21:31:13+00:00

This value is now assigned during bind(). The configuration depends on
static values assigned by dummy driver itself. So there is no need to
defer this assigment until one know the actuall speed since the
configuration is static and known early.

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Greg Kroah-Hartman

usb: gadget: dummy_hcd: move ep initialisation HW setup

2011-07-01T21:31:12+00:00

This is only required to be done once. There is no counter part to this
in ->stop() so there is no need to re-do it next time. While here also
init the max_stream size to 0 on SS speed.

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Greg Kroah-Hartman

usb: gadget: dummy_hcd: move common bits of suspend/resume into one function

2011-07-01T21:31:09+00:00

Signed-off-by: Sebastian Andrzej Siewior 
Signed-off-by: Felipe Balbi 
Signed-off-by: Greg Kroah-Hartman