On 03/24/2017 01:56 PM, Marek Vasut wrote: > 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 <abrod...@synopsys.com> >>>>>>>> Cc: Marek Vasut <ma...@denx.de> >>>>>>>> Cc: Simon Glass <s...@chromium.org> >>>>>>>> Cc: Mateusz Kulikowski <mateusz.kulikow...@gmail.com> >>>>>>> >>>>>>> IMO looks OK, >>>>>>> >>>>>>> Acked-by: Marek Vasut <ma...@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. >
Even better, this patch breaks powerpc board lwmon5, so dropped. -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
