linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node
@ 2022-03-22  2:28 Li Yang
  2022-03-22  2:28 ` [PATCH 2/6] arm64: dts: " Li Yang
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Li Yang @ 2022-03-22  2:28 UTC (permalink / raw)
  To: Shawn Guo, Li Yang, Rob Herring
  Cc: linux-arm-kernel, devicetree, linux-kernel

The binding of ifc device has been updated.  Update dts to match
accordingly.

Signed-off-by: Li Yang <leoyang.li@nxp.com>
---
 arch/arm/boot/dts/ls1021a.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi
index 2e69d6eab4d1..52835d3dfa1b 100644
--- a/arch/arm/boot/dts/ls1021a.dtsi
+++ b/arch/arm/boot/dts/ls1021a.dtsi
@@ -123,7 +123,7 @@ msi2: msi-controller@1570e08 {
 		};
 
 		ifc: ifc@1530000 {
-			compatible = "fsl,ifc", "simple-bus";
+			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
 			status = "disabled";
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/6] arm64: dts: remove "simple-bus" compatible from ifc node
  2022-03-22  2:28 [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Li Yang
@ 2022-03-22  2:28 ` Li Yang
  2022-03-22  2:28 ` [PATCH 4/6] ARM: dts: update ifc node name to be memory-controller Li Yang
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: Li Yang @ 2022-03-22  2:28 UTC (permalink / raw)
  To: Shawn Guo, Li Yang, Rob Herring
  Cc: linux-arm-kernel, devicetree, linux-kernel

The binding of ifc device has been updated.  Update dts to match
accordingly.

Signed-off-by: Li Yang <leoyang.li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index 01b01e320411..996d45bc6d91 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -390,7 +390,7 @@ dcfg: dcfg@1ee0000 {
 		};
 
 		ifc: ifc@1530000 {
-			compatible = "fsl,ifc", "simple-bus";
+			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <0 43 0x4>;
 		};
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
index 687fea6d8afa..cdfb0513e081 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
@@ -281,7 +281,7 @@ ddr: memory-controller@1080000 {
 		};
 
 		ifc: ifc@1530000 {
-			compatible = "fsl,ifc", "simple-bus";
+			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>;
 			status = "disabled";
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
index 3ed1f2c51cad..c8f76898384b 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
@@ -397,7 +397,7 @@ gpio3: gpio@2330000 {
 		};
 
 		ifc: ifc@2240000 {
-			compatible = "fsl,ifc", "simple-bus";
+			compatible = "fsl,ifc";
 			reg = <0x0 0x2240000 0x0 0x20000>;
 			interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>;
 			little-endian;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index 3cb9c21d2775..ed872273c3af 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -1037,7 +1037,7 @@ i2c3: i2c@2030000 {
 		};
 
 		ifc: ifc@2240000 {
-			compatible = "fsl,ifc", "simple-bus";
+			compatible = "fsl,ifc";
 			reg = <0x0 0x2240000 0x0 0x20000>;
 			interrupts = <0 21 0x4>; /* Level high type */
 			little-endian;
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 4/6] ARM: dts: update ifc node name to be memory-controller
  2022-03-22  2:28 [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Li Yang
  2022-03-22  2:28 ` [PATCH 2/6] arm64: dts: " Li Yang
@ 2022-03-22  2:28 ` Li Yang
  2022-03-22  2:28 ` [PATCH 5/6] arm64: dts: freescale: " Li Yang
  2022-04-10  7:27 ` [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Shawn Guo
  3 siblings, 0 replies; 7+ messages in thread
From: Li Yang @ 2022-03-22  2:28 UTC (permalink / raw)
  To: Shawn Guo, Li Yang, Rob Herring
  Cc: linux-arm-kernel, devicetree, linux-kernel

Update the node name to be align with latest binding.

Signed-off-by: Li Yang <leoyang.li@nxp.com>
---
 arch/arm/boot/dts/ls1021a.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi
index 52835d3dfa1b..dca1bfbd028e 100644
--- a/arch/arm/boot/dts/ls1021a.dtsi
+++ b/arch/arm/boot/dts/ls1021a.dtsi
@@ -122,7 +122,7 @@ msi2: msi-controller@1570e08 {
 			interrupts = <GIC_SPI 180 IRQ_TYPE_LEVEL_HIGH>;
 		};
 
-		ifc: ifc@1530000 {
+		ifc: memory-controller@1530000 {
 			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 5/6] arm64: dts: freescale: update ifc node name to be memory-controller
  2022-03-22  2:28 [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Li Yang
  2022-03-22  2:28 ` [PATCH 2/6] arm64: dts: " Li Yang
  2022-03-22  2:28 ` [PATCH 4/6] ARM: dts: update ifc node name to be memory-controller Li Yang
@ 2022-03-22  2:28 ` Li Yang
  2022-04-10  7:27 ` [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Shawn Guo
  3 siblings, 0 replies; 7+ messages in thread
From: Li Yang @ 2022-03-22  2:28 UTC (permalink / raw)
  To: Shawn Guo, Li Yang, Rob Herring
  Cc: linux-arm-kernel, devicetree, linux-kernel

Update the node name to be align with updated DT binding.  But be
noted that u-boot for ls1088a used the ifc node name to disable ifc-nor
node when the SoC is configured to use QSPI.  The u-boot has been
updated to use the latest name but the change could break
compatibility with older u-boot for ls1088a.

Signed-off-by: Li Yang <leoyang.li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 2 +-
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index 996d45bc6d91..7cd1260b0690 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -389,7 +389,7 @@ dcfg: dcfg@1ee0000 {
 			big-endian;
 		};
 
-		ifc: ifc@1530000 {
+		ifc: memory-controller@1530000 {
 			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <0 43 0x4>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
index cdfb0513e081..df7104477e3e 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
@@ -280,7 +280,7 @@ ddr: memory-controller@1080000 {
 			big-endian;
 		};
 
-		ifc: ifc@1530000 {
+		ifc: memory-controller@1530000 {
 			compatible = "fsl,ifc";
 			reg = <0x0 0x1530000 0x0 0x10000>;
 			interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
index c8f76898384b..de57c2c2cf3b 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
@@ -396,7 +396,7 @@ gpio3: gpio@2330000 {
 			#interrupt-cells = <2>;
 		};
 
-		ifc: ifc@2240000 {
+		ifc: memory-controller@2240000 {
 			compatible = "fsl,ifc";
 			reg = <0x0 0x2240000 0x0 0x20000>;
 			interrupts = <0 21 IRQ_TYPE_LEVEL_HIGH>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index ed872273c3af..fe9b6191ff1b 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -1036,7 +1036,7 @@ i2c3: i2c@2030000 {
 					    QORIQ_CLK_PLL_DIV(4)>;
 		};
 
-		ifc: ifc@2240000 {
+		ifc: memory-controller@2240000 {
 			compatible = "fsl,ifc";
 			reg = <0x0 0x2240000 0x0 0x20000>;
 			interrupts = <0 21 0x4>; /* Level high type */
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node
  2022-03-22  2:28 [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Li Yang
                   ` (2 preceding siblings ...)
  2022-03-22  2:28 ` [PATCH 5/6] arm64: dts: freescale: " Li Yang
@ 2022-04-10  7:27 ` Shawn Guo
  2022-04-27 21:26   ` Leo Li
  3 siblings, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2022-04-10  7:27 UTC (permalink / raw)
  To: Li Yang; +Cc: Rob Herring, linux-arm-kernel, devicetree, linux-kernel

On Mon, Mar 21, 2022 at 09:28:13PM -0500, Li Yang wrote:
> The binding of ifc device has been updated.  Update dts to match
> accordingly.
> 
> Signed-off-by: Li Yang <leoyang.li@nxp.com>

I only received patches #1, #2, #4 and #5, while it claims there are 6
patches in the series.

Shawn

> ---
>  arch/arm/boot/dts/ls1021a.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi
> index 2e69d6eab4d1..52835d3dfa1b 100644
> --- a/arch/arm/boot/dts/ls1021a.dtsi
> +++ b/arch/arm/boot/dts/ls1021a.dtsi
> @@ -123,7 +123,7 @@ msi2: msi-controller@1570e08 {
>  		};
>  
>  		ifc: ifc@1530000 {
> -			compatible = "fsl,ifc", "simple-bus";
> +			compatible = "fsl,ifc";
>  			reg = <0x0 0x1530000 0x0 0x10000>;
>  			interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
>  			status = "disabled";
> -- 
> 2.25.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node
  2022-04-10  7:27 ` [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Shawn Guo
@ 2022-04-27 21:26   ` Leo Li
  2022-05-05  0:50     ` Shawn Guo
  0 siblings, 1 reply; 7+ messages in thread
From: Leo Li @ 2022-04-27 21:26 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, linux-arm-kernel@lists.infradead.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org



> -----Original Message-----
> From: Shawn Guo <shawnguo@kernel.org>
> Sent: Sunday, April 10, 2022 2:28 AM
> To: Leo Li <leoyang.li@nxp.com>
> Cc: Rob Herring <robh+dt@kernel.org>; linux-arm-kernel@lists.infradead.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible
> from ifc node
> 
> On Mon, Mar 21, 2022 at 09:28:13PM -0500, Li Yang wrote:
> > The binding of ifc device has been updated.  Update dts to match
> > accordingly.
> >
> > Signed-off-by: Li Yang <leoyang.li@nxp.com>
> 
> I only received patches #1, #2, #4 and #5, while it claims there are 6 patches
> in the series.

I used the get_maintainers.pl script to generate the recipients.  The patch #3 and #6 are for powerpc, that's probably why it missed you.  Please help to review the current patches.  If they need a rebase to the latest baseline I will resend the whole series and include you for all the patches.

Regards,
Leo
> 
> Shawn
> 
> > ---
> >  arch/arm/boot/dts/ls1021a.dtsi | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/ls1021a.dtsi
> > b/arch/arm/boot/dts/ls1021a.dtsi index 2e69d6eab4d1..52835d3dfa1b
> > 100644
> > --- a/arch/arm/boot/dts/ls1021a.dtsi
> > +++ b/arch/arm/boot/dts/ls1021a.dtsi
> > @@ -123,7 +123,7 @@ msi2: msi-controller@1570e08 {
> >  		};
> >
> >  		ifc: ifc@1530000 {
> > -			compatible = "fsl,ifc", "simple-bus";
> > +			compatible = "fsl,ifc";
> >  			reg = <0x0 0x1530000 0x0 0x10000>;
> >  			interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
> >  			status = "disabled";
> > --
> > 2.25.1
> >

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node
  2022-04-27 21:26   ` Leo Li
@ 2022-05-05  0:50     ` Shawn Guo
  0 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2022-05-05  0:50 UTC (permalink / raw)
  To: Leo Li
  Cc: Rob Herring, linux-arm-kernel@lists.infradead.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org

On Wed, Apr 27, 2022 at 09:26:22PM +0000, Leo Li wrote:
> 
> 
> > -----Original Message-----
> > From: Shawn Guo <shawnguo@kernel.org>
> > Sent: Sunday, April 10, 2022 2:28 AM
> > To: Leo Li <leoyang.li@nxp.com>
> > Cc: Rob Herring <robh+dt@kernel.org>; linux-arm-kernel@lists.infradead.org;
> > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible
> > from ifc node
> > 
> > On Mon, Mar 21, 2022 at 09:28:13PM -0500, Li Yang wrote:
> > > The binding of ifc device has been updated.  Update dts to match
> > > accordingly.
> > >
> > > Signed-off-by: Li Yang <leoyang.li@nxp.com>
> > 
> > I only received patches #1, #2, #4 and #5, while it claims there are 6 patches
> > in the series.
> 
> I used the get_maintainers.pl script to generate the recipients.  The patch #3 and #6 are for powerpc, that's probably why it missed you.  Please help to review the current patches.  If they need a rebase to the latest baseline I will resend the whole series and include you for all the patches.
> 

Applied all 4 patches you sent me, thanks!

Shawn

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-05-05  0:51 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-03-22  2:28 [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Li Yang
2022-03-22  2:28 ` [PATCH 2/6] arm64: dts: " Li Yang
2022-03-22  2:28 ` [PATCH 4/6] ARM: dts: update ifc node name to be memory-controller Li Yang
2022-03-22  2:28 ` [PATCH 5/6] arm64: dts: freescale: " Li Yang
2022-04-10  7:27 ` [PATCH 1/6] ARM: dts: ls1021a: remove "simple-bus" compatible from ifc node Shawn Guo
2022-04-27 21:26   ` Leo Li
2022-05-05  0:50     ` Shawn Guo

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).