[RFT PATCH v1 1/5] Revert "usb: ehci-hcd: Keep async schedule running"

Lukasz Majewski lukma at denx.de
Mon Mar 23 07:57:49 CET 2020


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. 

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?

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). 



Best regards,

Lukasz Majewski

--

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
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200323/02cc5973/attachment.sig>


More information about the U-Boot mailing list