* [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles
@ 2015-04-09 6:21 Igal.Liberman
2015-04-09 17:02 ` [PATCH] powerpc/dts: Add some SoC specific "fsl,*-device-config" compatibles Scott Wood
0 siblings, 1 reply; 3+ messages in thread
From: Igal.Liberman @ 2015-04-09 6:21 UTC (permalink / raw)
To: devicetree, linuxppc-dev; +Cc: scottwood, Igal Liberman
From: Igal Liberman <Igal.Liberman@freescale.com>
This patch is required for: https://patchwork.ozlabs.org/patch/459584/
Signed-off-by: Igal Liberman <Igal.Liberman@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@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";
};
global-utilities@e1000 {
diff --git a/arch/powerpc/boot/dts/fsl/b4860si-post.dtsi b/arch/powerpc/boot/dts/fsl/b4860si-post.dtsi
index 02ccde6..74a9f5f 100644
--- a/arch/powerpc/boot/dts/fsl/b4860si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/b4860si-post.dtsi
@@ -248,7 +248,8 @@
};
guts: global-utilities@e0000 {
- compatible = "fsl,b4860-device-config", "fsl,qoriq-device-config-2.0";
+ compatible = "fsl,b4860-device-config", "fsl,b4-device-config",
+ "fsl,qoriq-device-config-2.0";
};
global-utilities@e1000 {
diff --git a/arch/powerpc/boot/dts/fsl/b4si-post.dtsi b/arch/powerpc/boot/dts/fsl/b4si-post.dtsi
index f8c325e..babb076 100644
--- a/arch/powerpc/boot/dts/fsl/b4si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/b4si-post.dtsi
@@ -391,7 +391,8 @@
/include/ "qoriq-mpic4.3.dtsi"
guts: global-utilities@e0000 {
- compatible = "fsl,b4-device-config";
+ compatible = "fsl,b4-device-config",
+ "fsl,qoriq-device-config-2.0";
reg = <0xe0000 0xe00>;
fsl,has-rstcr;
fsl,liodn-bits = <12>;
diff --git a/arch/powerpc/boot/dts/fsl/p2041si-post.dtsi b/arch/powerpc/boot/dts/fsl/p2041si-post.dtsi
index 1f18b8b..a7ae5ae 100644
--- a/arch/powerpc/boot/dts/fsl/p2041si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p2041si-post.dtsi
@@ -311,7 +311,8 @@
/include/ "qoriq-mpic.dtsi"
guts: global-utilities@e0000 {
- compatible = "fsl,qoriq-device-config-1.0";
+ compatible = "fsl,p2041-device-config",
+ "fsl,qoriq-device-config-1.0";
reg = <0xe0000 0xe00>;
fsl,has-rstcr;
#sleep-cells = <1>;
diff --git a/arch/powerpc/boot/dts/fsl/p3041si-post.dtsi b/arch/powerpc/boot/dts/fsl/p3041si-post.dtsi
index a555d24..89d08d6 100644
--- a/arch/powerpc/boot/dts/fsl/p3041si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p3041si-post.dtsi
@@ -338,7 +338,8 @@
/include/ "qoriq-mpic.dtsi"
guts: global-utilities@e0000 {
- compatible = "fsl,qoriq-device-config-1.0";
+ compatible = "fsl,p3041-device-config",
+ "fsl,qoriq-device-config-1.0";
reg = <0xe0000 0xe00>;
fsl,has-rstcr;
#sleep-cells = <1>;
diff --git a/arch/powerpc/boot/dts/fsl/p4080si-post.dtsi b/arch/powerpc/boot/dts/fsl/p4080si-post.dtsi
index 0fe7281..1863d6a 100644
--- a/arch/powerpc/boot/dts/fsl/p4080si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p4080si-post.dtsi
@@ -358,7 +358,8 @@
/include/ "qoriq-mpic.dtsi"
guts: global-utilities@e0000 {
- compatible = "fsl,qoriq-device-config-1.0";
+ compatible = "fsl,p4080-device-config",
+ "fsl,qoriq-device-config-1.0";
reg = <0xe0000 0xe00>;
fsl,has-rstcr;
#sleep-cells = <1>;
diff --git a/arch/powerpc/boot/dts/fsl/p5020si-post.dtsi b/arch/powerpc/boot/dts/fsl/p5020si-post.dtsi
index a34ca20..d979d01 100644
--- a/arch/powerpc/boot/dts/fsl/p5020si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p5020si-post.dtsi
@@ -343,7 +343,8 @@
/include/ "qoriq-mpic.dtsi"
guts: global-utilities@e0000 {
- compatible = "fsl,qoriq-device-config-1.0";
+ compatible = "fsl,p5020-device-config",
+ "fsl,qoriq-device-config-1.0";
reg = <0xe0000 0xe00>;
fsl,has-rstcr;
#sleep-cells = <1>;
--
1.7.9.5
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] powerpc/dts: Add some SoC specific "fsl,*-device-config" compatibles
2015-04-09 6:21 [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles Igal.Liberman
@ 2015-04-09 17:02 ` Scott Wood
2015-04-12 16:01 ` [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles Igal.Liberman
0 siblings, 1 reply; 3+ messages in thread
From: Scott Wood @ 2015-04-09 17:02 UTC (permalink / raw)
To: Igal.Liberman; +Cc: devicetree, linuxppc-dev
On Thu, 2015-04-09 at 09:21 +0300, Igal.Liberman wrote:
> From: Igal Liberman <Igal.Liberman@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?
> Signed-off-by: Igal Liberman <Igal.Liberman@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@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?
-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
^ permalink raw reply [flat|nested] 3+ messages in thread* RE: [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles
2015-04-09 17:02 ` [PATCH] powerpc/dts: Add some SoC specific "fsl,*-device-config" compatibles Scott Wood
@ 2015-04-12 16:01 ` Igal.Liberman
0 siblings, 0 replies; 3+ messages in thread
From: Igal.Liberman @ 2015-04-12 16:01 UTC (permalink / raw)
To: Scott Wood; +Cc: devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org
Regards,
Igal Liberman.
> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Thursday, April 09, 2015 8:02 PM
> To: Liberman Igal-B31950
> Cc: devicetree@vger.kernel.org; linuxppc-dev@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@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@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@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
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-04-12 16:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-09 6:21 [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles Igal.Liberman
2015-04-09 17:02 ` [PATCH] powerpc/dts: Add some SoC specific "fsl,*-device-config" compatibles Scott Wood
2015-04-12 16:01 ` [PATCH] powerpc/dts: Add some SoC specific "fsl, *-device-config" compatibles Igal.Liberman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).