From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ehci-pci: Prepare for usage of readl()/writel() accessors
Date: Wed, 14 Jun 2017 09:55:23 +0000 [thread overview]
Message-ID: <1497434123.6424.17.camel@synopsys.com> (raw)
In-Reply-To: <0c78d643-41fb-b9b6-6d5e-2d2f39c2b9b7@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
next prev parent reply other threads:[~2017-06-14 9:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-05 7:19 [U-Boot] [PATCH] ehci-pci: Prepare for usage of readl()/writel() accessors Alexey Brodkin
2017-06-05 11:42 ` Marek Vasut
2017-06-05 12:03 ` Alexey Brodkin
2017-06-05 12:06 ` Marek Vasut
2017-06-05 12:24 ` Alexey Brodkin
2017-06-14 9:55 ` Alexey Brodkin [this message]
2017-06-15 17:00 ` Marek Vasut
2017-06-15 17:19 ` Tom Rini
2017-06-15 17:23 ` Marek Vasut
2017-06-15 17:48 ` Alexey Brodkin
2017-06-15 17:51 ` Marek Vasut
2017-06-15 17:52 ` Alexey Brodkin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1497434123.6424.17.camel@synopsys.com \
--to=alexey.brodkin@synopsys.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.