From: Mark Brown <broonie@kernel.org>
To: Anders Berg <anders.berg@avagotech.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
"linux-spi@vger.kernel.org" <linux-spi@vger.kernel.org>,
Alexandre Courbot <gnurou@gmail.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] gpio: Add driver for AXM55xx SSP chip selects
Date: Mon, 8 Sep 2014 12:35:47 +0100 [thread overview]
Message-ID: <20140908113547.GJ4015@sirena.org.uk> (raw)
In-Reply-To: <CAE0=X_2q1Z+pGuh6OY9Si5jbrF9HkczNFUqUgK_8xBekNvMeyQ@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 911 bytes --]
On Mon, Sep 08, 2014 at 01:02:32PM +0200, Anders Berg wrote:
> On Sat, Sep 6, 2014 at 3:53 PM, Mark Brown <broonie@kernel.org> wrote:
> > Would it not be more straightforward to just register it as a platform
> > device in this case, or use a separate OF ID in the DT? We appear to
> > specify a compatible string when registering AMBA devices even if we are
> > capable of identifying using the ID registers.
> I'll drop this gpio patch for now and submit a patch on the pl022
> driver instead. I read the arm,primecell binding and learned that
> there is a property (arm,primecell-periphid) that can be used to
> override the HW PrimeCell ID. Seems like something to use here, right?
Possibly, though if there's no hardware ID to override to that might not
be the best option. You could always define a property that's specific
to the pl022 driver flagging that it's got these extra chip selects though.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2014-09-08 11:36 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-01 12:53 [PATCH 0/2] " Anders Berg
2014-09-01 12:53 ` [PATCH 1/2] gpio: " Anders Berg
2014-09-04 17:04 ` Linus Walleij
2014-09-04 17:06 ` Mark Brown
2014-09-04 17:32 ` Linus Walleij
2014-09-05 9:05 ` Anders Berg
2014-09-06 13:53 ` Mark Brown
2014-09-08 11:02 ` Anders Berg
2014-09-08 11:35 ` Mark Brown [this message]
2014-09-05 9:47 ` Russell King - ARM Linux
2014-09-05 13:40 ` Anders Berg
2014-09-01 12:53 ` [PATCH 2/2] Documentation: DT bindings for AXM55xx SSP CS Anders Berg
2014-09-04 17:05 ` Linus Walleij
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=20140908113547.GJ4015@sirena.org.uk \
--to=broonie@kernel.org \
--cc=anders.berg@avagotech.com \
--cc=devicetree@vger.kernel.org \
--cc=gnurou@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®