From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 2/3] ARM: mach-shmobile: r8a7779: add SATA support
Date: Fri, 1 Mar 2013 16:23:45 +0900 [thread overview]
Message-ID: <20130301072345.GD14376@verge.net.au> (raw)
In-Reply-To: <20130301014148.GB9232@quad.lixom.net>
On Thu, Feb 28, 2013 at 05:41:48PM -0800, Olof Johansson wrote:
> On Wed, Feb 27, 2013 at 11:39:14PM +0300, Sergei Shtylyov wrote:
> > From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >
> > Add SATA clock for r8a7779 SoC (for both device tree and usual cases).
> > Register SATA controller as a "late" platform device on r8a7779 SoC.
>
> Hi,
>
> If you have the a binding in the device tree (which you do through patch 1/3),
> then there's no reason to have a platform device for it.
Hi Olof,
the DT exists but currently the marzen board brings up all
of its devices using platform devices. Which if nothing else is
internally consistent.
I suppose it would be possible to add a call to
r8a7779_add_standard_devices_dt() and have the board bring
up this device using DT and the rest using platform drivers
(until the drivers are migrated to DT).
Would that be your preferred option?
Magnus, how do you feel about this idea?
next prev parent reply other threads:[~2013-03-01 7:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-27 20:39 [PATCH v4 2/3] ARM: mach-shmobile: r8a7779: add SATA support Sergei Shtylyov
2013-03-01 1:41 ` Olof Johansson
2013-03-01 7:23 ` Simon Horman [this message]
2013-03-05 0:26 ` Simon Horman
2013-03-05 1:05 ` Olof Johansson
2013-03-05 1:57 ` Simon Horman
2013-03-05 1:28 ` Magnus Damm
2013-03-05 2:05 ` Simon Horman
2013-03-05 2:22 ` Olof Johansson
2013-03-05 4:33 ` Magnus Damm
2013-03-05 5:52 ` Simon Horman
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=20130301072345.GD14376@verge.net.au \
--to=horms@verge.net.au \
--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