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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





Regards,
Igal Liberman.

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Thursday, April 09, 2015 8:02 PM
> To: Liberman Igal-B31950
> Cc: devicetree@xxxxxxxxxxxxxxx; linuxppc-dev@xxxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxxx>
> > ---
> >  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@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
��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux