From: Gavin Schenk <g.schenk@eckelmann.de>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Barebox List <barebox@lists.infradead.org>, ukl@pengutronix.de
Subject: Re: [PATCH] usb: Fix initial max packet size
Date: Tue, 27 Oct 2020 13:10:25 +0100 [thread overview]
Message-ID: <20201027121025.as7ca5k6vb3n2cj3@NB061.eckelmann.group> (raw)
In-Reply-To: <20201027094242.32582-1-s.hauer@pengutronix.de>
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@pengutronix.de>
Tested-by: Gavin Schenk <g.schenk@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@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
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
prev parent reply other threads:[~2020-10-27 12:10 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-27 9:42 Sascha Hauer
2020-10-27 12:10 ` Gavin Schenk [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201027121025.as7ca5k6vb3n2cj3@NB061.eckelmann.group \
--to=g.schenk@eckelmann.de \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.de \
--cc=ukl@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox