[RFC 09/15] ARM: shmobile: Include DTSI of r8a7740 to armadillo800eva

[Laurent Pinchart]

Hi Simon,

On Wednesday 21 November 2012 09:14:57 Simon Horman wrote:
> On Tue, Nov 20, 2012 at 01:45:20PM +0100, Laurent Pinchart wrote:
> > Hi Simon,
> > 
> > Thank you for the patch.
> > 
> > On Monday 19 November 2012 14:13:21 Simon Horman wrote:
> > > From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj at renesas.com>
> > > 
> > > Cc: Magnus Damm <damm at opensource.se>
> > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj at renesas.com>
> > > Signed-off-by: Simon Horman <horms at verge.net.au>
> > 
> > I came up today with pretty much the same patch :-)
> > 
> > > ---
> > > 
> > >  arch/arm/boot/dts/r8a7740-armadillo800eva.dts |    2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > > b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts index a7505a9..a8e7bd9
> > > 100644
> > > --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > > +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > > @@ -9,7 +9,7 @@
> > > 
> > >   */
> > >  
> > >  /dts-v1/;
> > > 
> > > -/include/ "skeleton.dtsi"
> > > +/include/ "r8a7740.dtsi"
> > > 
> > >  / {
> > >  
> > >  	model = "armadillo 800 eva";
> > 
> > Shouldn't the compatible property be modified to the following values ?
> > 
> > 	compatible = "renesas,armadillo800eva", "renesas,r8a7740";
> 
> To be honest, I'm not sure.
> But as this is a dts file for a board "renesas,r8a7740" seems strange to me.

If I understand DT correctly, the goal is to get rid of the board file at some 
point and only keep the generic r8a7740 board code. Shouldn't we then match on 
"renesas,r8a7740" ?

-- 
Regards,

Laurent Pinchart