[PATCH] usb: Keep async schedule running only across mass storage xfers

Lukasz Majewski lukma at denx.de
Fri Apr 10 08:21:20 CEST 2020


On Thu, 9 Apr 2020 19:23:26 -0400
Tom Rini <trini at konsulko.com> wrote:

> On Mon, Apr 06, 2020 at 02:29:44PM +0200, Marek Vasut wrote:
> 
> > Rather than keeping the asynchronous schedule running always, keep
> > it running only across USB mass storage transfers for now, as it
> > seems that keeping it running all the time interferes with certain
> > control transfers during device enumeration.
> > 
> > Note that running the async schedule all the time should not be an
> > issue, especially on EHCI HCD, as that one implements most of the
> > transfers using async schedule.
> > 
> > Note that we have usb_disable_asynch(), which however is utterly
> > broken. The usb_disable_asynch() blocks the USB core from doing
> > async transfers by setting a global flag. The async schedule should
> > however be disabled per USB controller. Moreover, setting a global
> > flag does not prevent the controller from using the async schedule,
> > which e.g. the EHCI HCD does.
> > 
> > This patch implements additional callback to the controller, which
> > permits it to lock the async schedule and keep it running across
> > multiple transfers. Once the schedule is unlocked, it must also be
> > disabled. This thus prevents the async schedule from running outside
> > of the USB mass storage transfers.
> > 
> > Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> > Cc: Lukasz Majewski <lukma at denx.de>
> > Cc: Tom Rini <trini at konsulko.com>
> > Tested-by: Tom Rini <trini at konsulko.com> [omap3_beagle, previously
> > failing]  
> 
> Applied to u-boot/master, thanks!
> 

It is a really short time between posting the patch (Monday) and being
applied (midnight on Friday).

It is the Pre-Easter period with some other world-wide issues... and not
all of us have enough time now to review and test patches.

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/20200410/7d8e1d42/attachment.sig>


More information about the U-Boot mailing list