[PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles

[Igal.Liberman at freescale.com]

Regards,
Igal Liberman.

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Thursday, April 09, 2015 8:02 PM
> To: Liberman Igal-B31950
> Cc: devicetree at vger.kernel.org; linuxppc-dev at lists.ozlabs.org
> Subject: Re: [PATCH] powerpc/dts: Add some SoC specific "fsl,*-device-
> config" compatibles
> 
> On Thu, 2015-04-09 at 09:21 +0300, Igal.Liberman wrote:
> > From: Igal Liberman <Igal.Liberman at freescale.com>
> >
> > This patch is required for: https://patchwork.ozlabs.org/patch/459584/
> 
> If it's required for that patch, why aren't they part of a patchset rather than
> independently posted?  I don't see any mention of a dependency in that
> patch, so how would I have known not to apply it?
> 

This patch was left out from the original submissions.
I'll add a comment in the next submission of https://patchwork.ozlabs.org/patch/459584/


> > Signed-off-by: Igal Liberman <Igal.Liberman at freescale.com>
> > ---
> >  arch/powerpc/boot/dts/fsl/b4420si-post.dtsi |    3 ++-
> >  arch/powerpc/boot/dts/fsl/b4860si-post.dtsi |    3 ++-
> >  arch/powerpc/boot/dts/fsl/b4si-post.dtsi    |    3 ++-
> >  arch/powerpc/boot/dts/fsl/p2041si-post.dtsi |    3 ++-
> >  arch/powerpc/boot/dts/fsl/p3041si-post.dtsi |    3 ++-
> >  arch/powerpc/boot/dts/fsl/p4080si-post.dtsi |    3 ++-
> >  arch/powerpc/boot/dts/fsl/p5020si-post.dtsi |    3 ++-
> >  7 files changed, 14 insertions(+), 7 deletions(-)
> >
> > diff --git a/arch/powerpc/boot/dts/fsl/b4420si-post.dtsi
> > b/arch/powerpc/boot/dts/fsl/b4420si-post.dtsi
> > index 1ea8602..9b14f24 100644
> > --- a/arch/powerpc/boot/dts/fsl/b4420si-post.dtsi
> > +++ b/arch/powerpc/boot/dts/fsl/b4420si-post.dtsi
> > @@ -77,7 +77,8 @@
> >  	};
> >
> >  	guts: global-utilities at e0000 {
> > -		compatible = "fsl,b4420-device-config", "fsl,qoriq-device-
> config-2.0";
> > +		compatible = "fsl,b4420-device-config", "fsl,b4-device-
> config",
> > +			      "fsl,qoriq-device-config-2.0";
> 
> Why?
> 

We wanted to be able to determine b4 derivatives together in the FMan clock initialization (without checking b4860/b4420 compatible). 

> -Scott
> 

Igal