Re: [PATCH v7 4/5] usb: ehci: Add new EHCI driver for Broadcom STB SoC's

From: Alan Stern
Date: Fri May 08 2020 - 14:46:54 EST


On Thu, 7 May 2020, Al Cooper wrote:

> Add a new EHCI driver for Broadcom STB SoC's. A new EHCI driver
> was created instead of adding support to the existing ehci platform
> driver because of the code required to workaround bugs in the EHCI
> controller. The primary workround is for a bug where the Core
> violates the SOF interval between the first two SOFs transmitted after
> resume. This only happens if the resume occurs near the end of a
> microframe. The fix is to intercept the echi-hcd request to complete
> RESUME and align it to the start of the next microframe.
>
> Signed-off-by: Al Cooper <alcooperx@xxxxxxxxx>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

Adding a new EHCI platform-specific driver is okay with me. However,
this patch does not include most of the changes you discussed with
Greg. I assume you will submit a revised version with those changes in
place; when you do I will Ack it.

Alan Stern