[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAOGqxeVmvJxqXEqymuCJRXqLH9O70Lc7GLTBdfYt_zpB6jexyA@mail.gmail.com>
Date: Fri, 8 May 2020 16:32:39 -0400
From: Alan Cooper <alcooperx@...il.com>
To: Alan Stern <stern@...land.harvard.edu>
Cc: ": Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
Andy Shevchenko <andy.shevchenko@...il.com>,
BCM Kernel Feedback <bcm-kernel-feedback-list@...adcom.com>,
DTML <devicetree@...r.kernel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
USB list <linux-usb@...r.kernel.org>,
Mathias Nyman <mathias.nyman@...el.com>,
Rob Herring <robh+dt@...nel.org>,
Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Subject: Re: [PATCH v7 4/5] usb: ehci: Add new EHCI driver for Broadcom STB SoC's
On Fri, May 8, 2020 at 2:46 PM Alan Stern <stern@...land.harvard.edu> wrote:
>
> 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@...il.com>
> > Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>
>
> 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
>
I mistakenly sent a partially fixed version instead of the version
with all the fixes (git stashed). I'm sending v8 will all the fixes.
Thanks
Thanks
Al
Powered by blists - more mailing lists