Hi Marek, > On 3/22/20 2:00 PM, Lukasz Majewski wrote: > > This reverts commit 02b0e1a36c5bc20174299312556ec4e266872bd6. > > > > Signed-off-by: Lukasz Majewski <lu...@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: lu...@denx.de
pgpIt40yZiRlo.pgp
Description: OpenPGP digital signature