[PATCH v4 2/3] ARM: mach-shmobile: r8a7779: add SATA support

Simon Horman horms at verge.net.au
Fri Mar 1 02:23:45 EST 2013


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 at 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?



More information about the linux-arm-kernel mailing list