public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] pata-generic/of: Make probing via device tree non-powerpc-specific
Date: Sat, 17 Sep 2011 20:35:57 +0200	[thread overview]
Message-ID: <2922376.ufKFh1svrE@wuerfel> (raw)
In-Reply-To: <20110916214313.GB16381@n2100.arm.linux.org.uk>

On Friday 16 September 2011 22:43:13 Russell King - ARM Linux wrote:
> On Fri, Sep 16, 2011 at 03:38:10PM +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>
> > ---
> > 
> > Tested on ARM Versatile Express, with my soon-to-be-posted device
> > tree support patches.
> 
> This may not be the correct way to support the CF slot on Versatile
> Express - it depends whether the CF slot on VE supports just CF
> memory cards or whether it can take any CF card.
> 
> If the latter, then what may be inserted could be a CF network card,
> and that means it's probably wrong to tell the kernel that what's
> there is a PATA interface.

The documentation at
http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0447e/CACECHFJ.html
claims that the slot only supports TrueIDE and PIO/Taskfile mode, which
are the two modes for CF memory cards. Of course this is highly dependent
on what you load into the FPGA that implements the CF interface on
the versatile express, but at least for the currect Versatile Express
FPGA load it should be right.

Also, the patch makes sense regardless of versatile express, it's just
basic cleanup.

One extension we might want to add eventually is support for the UDMA
modes that are present on most of the current CF cards but not supported
by this driver (AFAICT).

	Arnd

  reply	other threads:[~2011-09-17 18:35 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
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 [this message]
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=2922376.ufKFh1svrE@wuerfel \
    --to=arnd@arndb.de \
    --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