From mboxrd@z Thu Jan  1 00:00:00 1970
From: horms@verge.net.au (Simon Horman)
Date: Tue, 5 Mar 2013 09:26:55 +0900
Subject: [PATCH v4 2/3] ARM: mach-shmobile: r8a7779: add SATA support
In-Reply-To: <20130301072345.GD14376@verge.net.au>
References: <201302272339.15402.sergei.shtylyov@cogentembedded.com>
 <20130301014148.GB9232@quad.lixom.net>
 <20130301072345.GD14376@verge.net.au>
Message-ID: <20130305002653.GA9439@verge.net.au>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Fri, Mar 01, 2013 at 04:23:45PM +0900, Simon Horman wrote:
> 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?

Ping.