From: Lukasz Majewski <lukma@denx.de>
To: u-boot@lists.denx.de
Subject: [RFT PATCH v1 0/5] usb: Improve robustness of ehci-hcd controller operation
Date: Mon, 23 Mar 2020 23:11:03 +0100 [thread overview]
Message-ID: <20200323231103.180c136e@jawa> (raw)
In-Reply-To: <20200323205800.GN5793@bill-the-cat>
Hi Tom,
> On Sun, Mar 22, 2020 at 02:00:26PM +0100, Lukasz Majewski wrote:
>
> > This patch set is rather a request for testing (and a starting
> > point for the discussion), as it may improve the robustness of USB
> > with some pendrives - and yes sacrifice some performance for
> > reliability. The previous version of this patch:
> > https://patchwork.ozlabs.org/patch/1244928/ fixed issue for some
> > network USB adapters and improved stability on TI boards. This
> > patch also provides very detailed explanation of the problem in the
> > commit message.
> >
> > With the async support patch applied (
> > SHA1: 02b0e1a36c5bc20174299312556ec4e266872bd6 ), the qhtoken
> > variable has value 0x00 when token shows errors. As a result the
> > error handling path is not executed. This looks like some
> > missing/broken cache flushing - for easier bisecting this patch has
> > been reverted for now
>
> Note that while the original patch returned USB ethernet on the
> Beagleboard to a functional state with this series applied it's back
> to non-functional.
>
Thanks for testing.
The _only_ difference between the first version of this patch and this
one is the lack of dynamic reduction of transfer size for the latter.
The former reduces the transfer size to 64 blocks (instead of default
240). And with 64 blocks it retries two times the transmission.
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/3c9e4559/attachment.sig>
prev parent reply other threads:[~2020-03-23 22:11 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-22 13:00 [RFT PATCH v1 0/5] usb: Improve robustness of ehci-hcd controller operation Lukasz Majewski
2020-03-22 13:00 ` [RFT PATCH v1 1/5] Revert "usb: ehci-hcd: Keep async schedule running" Lukasz Majewski
2020-03-22 13:18 ` Marek Vasut
2020-03-23 6:57 ` Lukasz Majewski
2020-03-23 11:46 ` Marek Vasut
2020-03-23 12:41 ` Lukasz Majewski
2020-03-24 0:58 ` Marek Vasut
2020-03-24 7:06 ` Lukasz Majewski
2020-03-24 15:07 ` Marek Vasut
2020-03-24 18:11 ` Lukasz Majewski
2020-03-24 18:33 ` Marek Vasut
2020-03-22 13:00 ` [RFT PATCH v1 2/5] usb: Handle XACTERR error in DATA phase of USB storage Lukasz Majewski
2020-03-22 13:23 ` Marek Vasut
2020-03-23 7:00 ` Lukasz Majewski
2020-03-23 11:50 ` Marek Vasut
2020-03-23 13:03 ` Lukasz Majewski
2020-03-24 1:01 ` Marek Vasut
2020-03-22 13:00 ` [RFT PATCH v1 3/5] usb: Add some delay to wait for slow USB devices to be operational Lukasz Majewski
2020-03-22 13:29 ` Marek Vasut
2020-03-23 7:04 ` Lukasz Majewski
2020-03-23 11:52 ` Marek Vasut
2020-03-22 13:00 ` [RFT PATCH v1 4/5] usb: Provide code to handle spinup of USB usb devices (mostly HDDs) Lukasz Majewski
2020-03-22 13:32 ` Marek Vasut
2020-03-23 7:53 ` Lukasz Majewski
2020-03-23 11:57 ` Marek Vasut
2020-03-23 12:54 ` Lukasz Majewski
2020-03-24 1:04 ` Marek Vasut
2020-03-22 13:00 ` [RFT PATCH v1 5/5] usb: Handle QT_TOKEN_STATUS_XACTERR error when sending data Lukasz Majewski
2020-03-22 13:45 ` Marek Vasut
2020-03-23 7:18 ` Lukasz Majewski
2020-03-23 11:59 ` Marek Vasut
2020-03-23 12:58 ` Lukasz Majewski
2020-03-24 1:06 ` Marek Vasut
2020-03-23 20:58 ` [RFT PATCH v1 0/5] usb: Improve robustness of ehci-hcd controller operation Tom Rini
2020-03-23 22:11 ` Lukasz Majewski [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=20200323231103.180c136e@jawa \
--to=lukma@denx.de \
--cc=u-boot@lists.denx.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