[RFT PATCH v1 1/5] Revert "usb: ehci-hcd: Keep async schedule running"
lukma at denx.de
Mon Mar 23 13:41:35 CET 2020
> On 3/23/20 7:57 AM, Lukasz Majewski wrote:
> > Hi Marek,
> >> On 3/22/20 2:00 PM, Lukasz Majewski wrote:
> >>> This reverts commit 02b0e1a36c5bc20174299312556ec4e266872bd6.
> >>> Signed-off-by: Lukasz Majewski <lukma at denx.de>
> >> This patch lacks any and all explanation why this is being
> >> reverted. The patch you are reverting here explains why it was
> >> added and what real issues it was fixing, so instead of reverting
> >> it, if there is an issue with that patch, you should identify the
> >> issue and fix it.
> > Marek, have you received the cover letter for this patch series?
> > In the cover letter I've written the rationale for reverting this
> > patch.
> That should have been explained in this patch description.
> > In short - qhtoken has value of 0x0, when the token variable shows
> > errors. As a result the error handling is broken.
> > Could you comment on those arguments?
> Maybe you are referencing/reading the wrong token ?
I'm printing the token which is used afterwards for reacting on possible
> You should probably figure out why this doesn't work first and then
> add fixes on top.
Haven't you seen such problem during code development on your setup
when developing this patch?
> > Moreover, I've explicitly stated that this is a Request For
> > Testing like patch series with a detailed report of testing
> > procedure (for my use case) for the USB in U-Boot (as Tom has
> > tested the patch with some ETH dongles).
> I was still unable to replicate the ethernet device failure.
Which boards and SoCs do you used for your test setup?
For me the issue is visible on i.MX53 and i.MX6Q.
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 488 bytes
Desc: OpenPGP digital signature
More information about the U-Boot