From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey Brodkin Date: Wed, 14 Jun 2017 09:55:23 +0000 Subject: [U-Boot] [PATCH] ehci-pci: Prepare for usage of readl()/writel() accessors In-Reply-To: <0c78d643-41fb-b9b6-6d5e-2d2f39c2b9b7@denx.de> References: <1496647166-6661-1-git-send-email-abrodkin@synopsys.com> <64c48bb3-1943-7387-1680-2d6f41d5193f@denx.de> <1496664202.3931.8.camel@synopsys.com> <0c78d643-41fb-b9b6-6d5e-2d2f39c2b9b7@denx.de> Message-ID: <1497434123.6424.17.camel@synopsys.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Hi Marek, On Mon, 2017-06-05 at 14:06 +0200, Marek Vasut wrote: > On 06/05/2017 02:03 PM, Alexey Brodkin wrote: > > > > Hi Marek, > > Hi, > > > > > On Mon, 2017-06-05 at 13:42 +0200, Marek Vasut wrote: > > > > > > On 06/05/2017 09:19 AM, Alexey Brodkin wrote: > > > > > > > > > > > > We used to have opencoded ehci_readl()/writel() which required no > > > > external functions to be called. > > > > > > Applied, thanks. > > > > Now when we get everything ready for building fine > > see https://urldefense.proofpoint.com/v2/url?u=https-3A__travis-2Dci.org_abrodkin_u-2Dboot_builds_238394634&d=DwICaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=lq > > deeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=6zuMNpU2sqTEauB3FyrBrLk_RR6Ko4zfrK2Jtmw_WLo&s=FxquunKntzi7lTQhylwOxuVroRKBcROy8uH08Fs84AI&e=  > > could you please apply my old patch with conversion to platform- > > specific accessors? > > > > I mean this one https://urldefense.proofpoint.com/v2/url?u=http-3A__patchwork.ozlabs.org_patch_726714_&d=DwICaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=lqdeeSS > > Ees0GFDDl656eViXO7breS55ytWkhpk5R81I&m=6zuMNpU2sqTEauB3FyrBrLk_RR6Ko4zfrK2Jtmw_WLo&s=qFC1yxpdfhfEijsMnTgN0g_-1ckUkUNt8RdsQBJC1f0&e=  > > Yes, rebase, retest and repost. > That last comment was about another patch which depends on this change. So I'm wondering if there's a reason for "ehci-pci: Prepare for usage of readl()/writel() accessors" to not be applied? And BTW I did retested and resent the one that depends on this, see http://patchwork.ozlabs.org/patch/771436/. -Alexey