[PATCH] usb: Fix initial max packet size
Gavin Schenk
g.schenk at eckelmann.de
Tue Oct 27 08:10:25 EDT 2020
Hi,
thanks a lot, that's great.
On Tue, Oct 27, 2020 at 10:42:42AM +0100, Sascha Hauer wrote:
> In case the USB device is a full speed device we have to set
> dev->epmaxpacketin[0] and dev->epmaxpacketout[0] before doing a
> get_descriptor_len() because otherwise it's unset during that call.
>
> Fixes: 6d1d95a386 ("usb: factor out a usb_setup_descriptor() function")
> Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
Tested-by: Gavin Schenk <g.schenk at eckelmann.de>
On one of our i.MX25 based platforms there are currently four revert commits to
make USB work on barebox 2020.06.
(Reverted commits are:)
62d565ba0 (usb: hub: When no connection came up remove from scanning list)
1f8c8af48 (usb: Add super speed support)
bb2cbfb2b (usb: hub: Translate USB 3.0 hub port status into old version)
6d1d95a38 (usb: factor out a usb_setup_descriptor() function)
Without reverting the behaviour is:
usb: USB: scanning bus for devices...
usb1: Bus 001 Device 001: ID 0000:0000 EHCI Host Controller
ERROR: usb1-0: unable to get device descriptor (error=-110)
ERROR: usb1-0: USB device not accepting new address (error=80000000)
usb: 1 USB Device(s) found
With this patch applied USB is working again:
usb: USB: scanning bus for devices...
usb1: Bus 001 Device 001: ID 0000:0000 EHCI Host Controller
usb1-0: Bus 001 Device 002: ID 0951:161a DataTraveler 410
Using index 0 for the new disk
usb: 2 USB Device(s) found
barebox at Eckelmann AG ECU02:/ ls /mnt/usbstorage/
. ..
.Trash-1000 2016-barebox-eag-ecu02.img
2020-barebox-eckelmann-ecu02.img barebox-eag-ecu02.img
Regards
Gavin Schenk
Eckelmann AG
Vorstand: Dipl.-Ing. Peter Frankenbach (Sprecher) Dipl.-Wi.-Ing. Philipp Eckelmann
Dr.-Ing. Marco Münchhof Dr.-Ing. Frank Uhlemann
Vorsitzender des Aufsichtsrats: Hubertus G. Krossa Stv. Vorsitzender des Aufsichtsrats: Dr.-Ing. Gerd Eckelmann
Sitz der Gesellschaft: Berliner Str. 161, 65205 Wiesbaden, Amtsgericht Wiesbaden HRB 12636
www.eckelmann.de
More information about the barebox
mailing list