[U-Boot] [PATCH] drivers/usb/ehci: Use platform-specific accessors

Alexey Brodkin Alexey.Brodkin at synopsys.com
Fri Mar 3 13:22:47 UTC 2017


Hi Marek,

On Fri, 2017-03-03 at 00:57 +0100, Marek Vasut wrote:
> On 03/01/2017 01:52 PM, Alexey Brodkin wrote:
> > 
> > Hi Marek,
> > 
> > On Fri, 2017-02-10 at 21:33 +0100, Marek Vasut wrote:
> > > 
> > > On 02/10/2017 09:23 PM, Alexey Brodkin wrote:
> > > > 
> > > > 
> > > > Current implementation doesn't allow utilization of platform-specific
> > > > reads and writes.
> > > > 
> > > > But some arches or platforms may want to use their accessors that do
> > > > some extra work like adding barriers for data serialization etc.
> > > > 
> > > > Interesting enough OHCI accessors already do that so just aligning
> > > > EHCI to it now.
> > > > 
> > > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> > > > Cc: Marek Vasut <marex at denx.de>
> > > > Cc: Simon Glass <sjg at chromium.org>
> > > > Cc: Mateusz Kulikowski <mateusz.kulikowski at gmail.com>
> > > 
> > > IMO looks OK,
> > > 
> > > Acked-by: Marek Vasut <marex at denx.de>
> > > 
> > > I'd like to have a few more reviews of this before applying it for
> > > 2017.05 . Also CCing Wills, it'd be great to get a test on atheros
> > > MIPS .
> > 
> > It's been quite some time since your request to try the patch
> > on other systems and so far nobody answered. May we consider that
> > one for application then?
> > 
> > Essentially I'm not talking about current Tom's master branch
> > but would be good if the patch lands somewhere and once 2017.03
> > gets cut is merged in the main branch?
> 
> Yeah, it's not happening for 2017.03, it'll go into the next one.
> 
> Can you just rebase and repost the patch please ? I'll get to it next 
> week, I'm traveling until the 10th.

I just tried and the same .mbox applies cleanly on top of today's
U-Boot master so I believe there's no point in resending it.

Are you OK to just grab existing one?

-Alexey


More information about the U-Boot mailing list