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

[Marek Vasut]

On 03/24/2017 12:08 PM, Alexey Brodkin wrote:
> Hi Marek,
> 
> On Sun, 2017-03-05 at 02:09 +0100, Marek Vasut wrote:
>> On 03/03/2017 02:22 PM, Alexey Brodkin wrote:
>>>
>>> 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?
>>
>> I no longer have the patch in my mailbox, which is why I asked you to 
>> resend it. Moreover, did you just test that it applies or did you 
>> actually build-test and retest the patch on real hardware too ?
> 
> I did test it on my boards again on top of today's master branch and
> everything seem to work perfectly fine so please consider applying this one.
> 
> You may use mbox from here http://patchwork.ozlabs.org/patch/726714/mbox/
> Al least it worked for me this way:
> --------------------->8-----------------------
> curl http://patchwork.ozlabs.org/patch/726714/mbox/ | git am
> --------------------->8-----------------------

Thank you for NOT resending the patch and making my life harder. Applied.

-- 
Best regards,
Marek Vasut