From: grant.likely@secretlab.ca (Grant Likely)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] pata-generic/of: Make probing via device tree non-powerpc-specific
Date: Sat, 17 Sep 2011 09:37:32 -0600 [thread overview]
Message-ID: <20110917153732.GR3523@ponder.secretlab.ca> (raw)
In-Reply-To: <1316190120-17010-1-git-send-email-dave.martin@linaro.org>
On Fri, Sep 16, 2011 at 05:22:00PM +0100, Dave Martin wrote:
> This patch enables device-tree-based probing of the pata-generic
> platform driver across all architectures:
>
> * make the pata_of_generic module depend on OF instead of PPC_OF;
> * supply some missing inclues;
> * replace endianness-sensitive raw access to device tree data
> with of_property_read_u32() calls.
>
> Signed-off-by: Dave Martin <dave.martin@linaro.org>
> ---
> v2: correct sense of the check of_property_read_u32(dn, "pio-mode",
> &pio_mode). Somehow I posted an old version of this patch, depite
> having already fixed this...
>
> Tested on ARM Versatile Express, with my soon-to-be-posted device
> tree support patches.
>
> I'm not in a position to build/test this for powerpc easily --
> if anyone is able to do that, it would be appreciated.
What driver is normally used for versatile express pata? This driver
is kind of legacy in that it was created when there was a split
between platform_device and of_platform_devices. But that split was a
bad idea and the same driver should be used regardless of whether or
not DT is enabled. pata_of_platform.c really should be removed.
So, instead of removing the PPC_OF restriction from
pata_of_platform.c, you should look at adding DT support to the
existing binding. Bonus points if you can roll pata_of_platform.c
support into pata_platform.c (assuming that is the correct driver).
g.
next prev parent reply other threads:[~2011-09-17 15:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-16 14:38 [PATCH] pata-generic/of: Make probing via device tree non-powerpc-specific Dave Martin
2011-09-16 16:22 ` [PATCH v2] " Dave Martin
2011-09-16 21:34 ` Rob Herring
2011-09-19 10:10 ` Dave Martin
2011-09-20 19:30 ` Jeff Garzik
2011-09-17 15:37 ` Grant Likely [this message]
2011-09-17 18:40 ` Arnd Bergmann
2011-09-17 21:30 ` Grant Likely
2011-09-16 21:43 ` [PATCH] " Russell King - ARM Linux
2011-09-17 18:35 ` Arnd Bergmann
2011-09-19 10:05 ` Dave Martin
2011-09-19 10:23 ` Pawel Moll
2011-09-19 10:56 ` Dave Martin
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=20110917153732.GR3523@ponder.secretlab.ca \
--to=grant.likely@secretlab.ca \
--cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).