* [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
@ 2019-12-30 8:58 Peng Fan
2020-01-04 0:36 ` Rob Herring
0 siblings, 1 reply; 4+ messages in thread
From: Peng Fan @ 2019-12-30 8:58 UTC (permalink / raw)
To: robh+dt, mark.rutland, shawnguo, kernel, festevam
Cc: Aisheng Dong, devicetree, Peng Fan, Alice Guo, linux-kernel,
dl-linux-imx, linux-arm-kernel
From: Peng Fan <peng.fan@nxp.com>
There is no binding doc for "fsl,aips-bus", "simple-bus" is enough
for aips usage, so drop it.
Scirpt:
sed -i 's/compatible = "fsl,aips-bus", "simple-bus";/compatible = "simple-bus";/'
arch/arm/boot/dts/imx*
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm/boot/dts/imx25.dtsi | 4 ++--
arch/arm/boot/dts/imx31.dtsi | 4 ++--
arch/arm/boot/dts/imx50.dtsi | 4 ++--
arch/arm/boot/dts/imx51.dtsi | 4 ++--
arch/arm/boot/dts/imx53.dtsi | 4 ++--
arch/arm/boot/dts/imx6qdl.dtsi | 4 ++--
arch/arm/boot/dts/imx6sl.dtsi | 4 ++--
arch/arm/boot/dts/imx6sll.dtsi | 4 ++--
arch/arm/boot/dts/imx6sx.dtsi | 6 +++---
arch/arm/boot/dts/imx6ul.dtsi | 4 ++--
arch/arm/boot/dts/imx6ull.dtsi | 2 +-
arch/arm/boot/dts/imx7s.dtsi | 6 +++---
12 files changed, 25 insertions(+), 25 deletions(-)
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 40b95a290bd6..3b7a0b249d80 100644
--- a/arch/arm/boot/dts/imx25.dtsi
+++ b/arch/arm/boot/dts/imx25.dtsi
@@ -76,7 +76,7 @@
ranges;
aips@43f00000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x43f00000 0x100000>;
@@ -333,7 +333,7 @@
};
aips@53f00000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x53f00000 0x100000>;
diff --git a/arch/arm/boot/dts/imx31.dtsi b/arch/arm/boot/dts/imx31.dtsi
index 6b62f0745b82..b0e7e3bf8a1a 100644
--- a/arch/arm/boot/dts/imx31.dtsi
+++ b/arch/arm/boot/dts/imx31.dtsi
@@ -64,7 +64,7 @@
};
aips@43f00000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x43f00000 0x100000>;
@@ -226,7 +226,7 @@
};
aips@53f00000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x53f00000 0x100000>;
diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
index 0bfe7c91d0eb..961de09b571d 100644
--- a/arch/arm/boot/dts/imx50.dtsi
+++ b/arch/arm/boot/dts/imx50.dtsi
@@ -102,7 +102,7 @@
ranges;
aips@50000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x50000000 0x10000000>;
@@ -390,7 +390,7 @@
};
aips@60000000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x60000000 0x10000000>;
diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
index dea86b98e9c3..86708688371b 100644
--- a/arch/arm/boot/dts/imx51.dtsi
+++ b/arch/arm/boot/dts/imx51.dtsi
@@ -159,7 +159,7 @@
};
aips@70000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x70000000 0x10000000>;
@@ -441,7 +441,7 @@
};
aips@80000000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x80000000 0x10000000>;
diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
index ed341cfd9d09..f46a83c7d2c4 100644
--- a/arch/arm/boot/dts/imx53.dtsi
+++ b/arch/arm/boot/dts/imx53.dtsi
@@ -223,7 +223,7 @@
};
aips@50000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x50000000 0x10000000>;
@@ -655,7 +655,7 @@
};
aips@60000000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x60000000 0x10000000>;
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index e6b4b8525f98..9b7635e9cf3c 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -295,7 +295,7 @@
};
aips-bus@2000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
@@ -936,7 +936,7 @@
};
aips-bus@2100000 { /* AIPS2 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02100000 0x100000>;
diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
index 59c54e6ad09a..4b4f22217dfe 100644
--- a/arch/arm/boot/dts/imx6sl.dtsi
+++ b/arch/arm/boot/dts/imx6sl.dtsi
@@ -144,7 +144,7 @@
};
aips1: aips-bus@2000000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
@@ -787,7 +787,7 @@
};
aips2: aips-bus@2100000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02100000 0x100000>;
diff --git a/arch/arm/boot/dts/imx6sll.dtsi b/arch/arm/boot/dts/imx6sll.dtsi
index a1bc5bb31756..fac8f22255aa 100644
--- a/arch/arm/boot/dts/imx6sll.dtsi
+++ b/arch/arm/boot/dts/imx6sll.dtsi
@@ -145,7 +145,7 @@
};
aips1: aips-bus@2000000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
@@ -664,7 +664,7 @@
};
aips2: aips-bus@2100000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02100000 0x100000>;
diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
index 59bad60a47dc..4499be62c8bb 100644
--- a/arch/arm/boot/dts/imx6sx.dtsi
+++ b/arch/arm/boot/dts/imx6sx.dtsi
@@ -236,7 +236,7 @@
};
aips1: aips-bus@2000000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
@@ -831,7 +831,7 @@
};
aips2: aips-bus@2100000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02100000 0x100000>;
@@ -1189,7 +1189,7 @@
};
aips3: aips-bus@2200000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02200000 0x100000>;
diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
index d9fdca12819b..63d276fc2477 100644
--- a/arch/arm/boot/dts/imx6ul.dtsi
+++ b/arch/arm/boot/dts/imx6ul.dtsi
@@ -205,7 +205,7 @@
};
aips1: aips-bus@2000000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
@@ -772,7 +772,7 @@
};
aips2: aips-bus@2100000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02100000 0x100000>;
diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi
index b7e67d121322..633fa08bc972 100644
--- a/arch/arm/boot/dts/imx6ull.dtsi
+++ b/arch/arm/boot/dts/imx6ull.dtsi
@@ -52,7 +52,7 @@
/ {
soc {
aips3: aips-bus@2200000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02200000 0x100000>;
diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arch/arm/boot/dts/imx7s.dtsi
index 139ab9b98472..552b14be14a1 100644
--- a/arch/arm/boot/dts/imx7s.dtsi
+++ b/arch/arm/boot/dts/imx7s.dtsi
@@ -317,7 +317,7 @@
};
aips1: aips-bus@30000000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x30000000 0x400000>;
@@ -669,7 +669,7 @@
};
aips2: aips-bus@30400000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x30400000 0x400000>;
@@ -809,7 +809,7 @@
};
aips3: aips-bus@30800000 {
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x30800000 0x400000>;
--
2.16.4
_______________________________________________
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] 4+ messages in thread
* Re: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
2019-12-30 8:58 [PATCH] ARM: dts: imx: drop "fsl,aips-bus" Peng Fan
@ 2020-01-04 0:36 ` Rob Herring
2020-01-06 2:38 ` Peng Fan
0 siblings, 1 reply; 4+ messages in thread
From: Rob Herring @ 2020-01-04 0:36 UTC (permalink / raw)
To: Peng Fan
Cc: mark.rutland, Aisheng Dong, devicetree, shawnguo, Alice Guo,
linux-kernel, dl-linux-imx, kernel, festevam, linux-arm-kernel
On Mon, Dec 30, 2019 at 08:58:05AM +0000, Peng Fan wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> There is no binding doc for "fsl,aips-bus", "simple-bus" is enough
> for aips usage, so drop it.
NAK. The AIPS bus has registers, so 'simple-bus' alone is not enough.
What you should do is change 'aips' node names to 'bus'.
>
> Scirpt:
> sed -i 's/compatible = "fsl,aips-bus", "simple-bus";/compatible = "simple-bus";/'
> arch/arm/boot/dts/imx*
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> arch/arm/boot/dts/imx25.dtsi | 4 ++--
> arch/arm/boot/dts/imx31.dtsi | 4 ++--
> arch/arm/boot/dts/imx50.dtsi | 4 ++--
> arch/arm/boot/dts/imx51.dtsi | 4 ++--
> arch/arm/boot/dts/imx53.dtsi | 4 ++--
> arch/arm/boot/dts/imx6qdl.dtsi | 4 ++--
> arch/arm/boot/dts/imx6sl.dtsi | 4 ++--
> arch/arm/boot/dts/imx6sll.dtsi | 4 ++--
> arch/arm/boot/dts/imx6sx.dtsi | 6 +++---
> arch/arm/boot/dts/imx6ul.dtsi | 4 ++--
> arch/arm/boot/dts/imx6ull.dtsi | 2 +-
> arch/arm/boot/dts/imx7s.dtsi | 6 +++---
> 12 files changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
> index 40b95a290bd6..3b7a0b249d80 100644
> --- a/arch/arm/boot/dts/imx25.dtsi
> +++ b/arch/arm/boot/dts/imx25.dtsi
> @@ -76,7 +76,7 @@
> ranges;
>
> aips@43f00000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x43f00000 0x100000>;
> @@ -333,7 +333,7 @@
> };
>
> aips@53f00000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x53f00000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx31.dtsi b/arch/arm/boot/dts/imx31.dtsi
> index 6b62f0745b82..b0e7e3bf8a1a 100644
> --- a/arch/arm/boot/dts/imx31.dtsi
> +++ b/arch/arm/boot/dts/imx31.dtsi
> @@ -64,7 +64,7 @@
> };
>
> aips@43f00000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x43f00000 0x100000>;
> @@ -226,7 +226,7 @@
> };
>
> aips@53f00000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x53f00000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
> index 0bfe7c91d0eb..961de09b571d 100644
> --- a/arch/arm/boot/dts/imx50.dtsi
> +++ b/arch/arm/boot/dts/imx50.dtsi
> @@ -102,7 +102,7 @@
> ranges;
>
> aips@50000000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x50000000 0x10000000>;
> @@ -390,7 +390,7 @@
> };
>
> aips@60000000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x60000000 0x10000000>;
> diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
> index dea86b98e9c3..86708688371b 100644
> --- a/arch/arm/boot/dts/imx51.dtsi
> +++ b/arch/arm/boot/dts/imx51.dtsi
> @@ -159,7 +159,7 @@
> };
>
> aips@70000000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x70000000 0x10000000>;
> @@ -441,7 +441,7 @@
> };
>
> aips@80000000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x80000000 0x10000000>;
> diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
> index ed341cfd9d09..f46a83c7d2c4 100644
> --- a/arch/arm/boot/dts/imx53.dtsi
> +++ b/arch/arm/boot/dts/imx53.dtsi
> @@ -223,7 +223,7 @@
> };
>
> aips@50000000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x50000000 0x10000000>;
> @@ -655,7 +655,7 @@
> };
>
> aips@60000000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x60000000 0x10000000>;
> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> index e6b4b8525f98..9b7635e9cf3c 100644
> --- a/arch/arm/boot/dts/imx6qdl.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> @@ -295,7 +295,7 @@
> };
>
> aips-bus@2000000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
> @@ -936,7 +936,7 @@
> };
>
> aips-bus@2100000 { /* AIPS2 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02100000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
> index 59c54e6ad09a..4b4f22217dfe 100644
> --- a/arch/arm/boot/dts/imx6sl.dtsi
> +++ b/arch/arm/boot/dts/imx6sl.dtsi
> @@ -144,7 +144,7 @@
> };
>
> aips1: aips-bus@2000000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
> @@ -787,7 +787,7 @@
> };
>
> aips2: aips-bus@2100000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02100000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx6sll.dtsi b/arch/arm/boot/dts/imx6sll.dtsi
> index a1bc5bb31756..fac8f22255aa 100644
> --- a/arch/arm/boot/dts/imx6sll.dtsi
> +++ b/arch/arm/boot/dts/imx6sll.dtsi
> @@ -145,7 +145,7 @@
> };
>
> aips1: aips-bus@2000000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
> @@ -664,7 +664,7 @@
> };
>
> aips2: aips-bus@2100000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02100000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> index 59bad60a47dc..4499be62c8bb 100644
> --- a/arch/arm/boot/dts/imx6sx.dtsi
> +++ b/arch/arm/boot/dts/imx6sx.dtsi
> @@ -236,7 +236,7 @@
> };
>
> aips1: aips-bus@2000000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
> @@ -831,7 +831,7 @@
> };
>
> aips2: aips-bus@2100000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02100000 0x100000>;
> @@ -1189,7 +1189,7 @@
> };
>
> aips3: aips-bus@2200000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02200000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
> index d9fdca12819b..63d276fc2477 100644
> --- a/arch/arm/boot/dts/imx6ul.dtsi
> +++ b/arch/arm/boot/dts/imx6ul.dtsi
> @@ -205,7 +205,7 @@
> };
>
> aips1: aips-bus@2000000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
> @@ -772,7 +772,7 @@
> };
>
> aips2: aips-bus@2100000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02100000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi
> index b7e67d121322..633fa08bc972 100644
> --- a/arch/arm/boot/dts/imx6ull.dtsi
> +++ b/arch/arm/boot/dts/imx6ull.dtsi
> @@ -52,7 +52,7 @@
> / {
> soc {
> aips3: aips-bus@2200000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02200000 0x100000>;
> diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arch/arm/boot/dts/imx7s.dtsi
> index 139ab9b98472..552b14be14a1 100644
> --- a/arch/arm/boot/dts/imx7s.dtsi
> +++ b/arch/arm/boot/dts/imx7s.dtsi
> @@ -317,7 +317,7 @@
> };
>
> aips1: aips-bus@30000000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x30000000 0x400000>;
> @@ -669,7 +669,7 @@
> };
>
> aips2: aips-bus@30400000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x30400000 0x400000>;
> @@ -809,7 +809,7 @@
> };
>
> aips3: aips-bus@30800000 {
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x30800000 0x400000>;
> --
> 2.16.4
>
_______________________________________________
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] 4+ messages in thread
* RE: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
2020-01-04 0:36 ` Rob Herring
@ 2020-01-06 2:38 ` Peng Fan
2020-01-14 14:39 ` Rob Herring
0 siblings, 1 reply; 4+ messages in thread
From: Peng Fan @ 2020-01-06 2:38 UTC (permalink / raw)
To: Rob Herring
Cc: mark.rutland, Aisheng Dong, devicetree, shawnguo, Alice Guo,
linux-kernel, dl-linux-imx, kernel, festevam, linux-arm-kernel
Hi Rob,
> Subject: Re: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
>
> On Mon, Dec 30, 2019 at 08:58:05AM +0000, Peng Fan wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > There is no binding doc for "fsl,aips-bus", "simple-bus" is enough for
> > aips usage, so drop it.
>
> NAK. The AIPS bus has registers, so 'simple-bus' alone is not enough.
You mean the "reg" property below, right?
aips-bus@2000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
reg = <0x02000000 0x100000>;
But the reg property is not required, I think it could be removed.
There is no binding doc and driver for "fsl,aips-bus", so I think
It not make sense to have that compatible in dts.
>
> What you should do is change 'aips' node names to 'bus'.
How do you think about below change, with reg removed?
aips-bus@2000000 { /* AIPS1 */
- compatible = "fsl,aips-bus", "simple-bus";
+ compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
- reg = <0x02000000 0x100000>;
Thanks,
Peng.
>
> >
> > Scirpt:
> > sed -i 's/compatible = "fsl,aips-bus", "simple-bus";/compatible =
> "simple-bus";/'
> > arch/arm/boot/dts/imx*
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> > arch/arm/boot/dts/imx25.dtsi | 4 ++--
> > arch/arm/boot/dts/imx31.dtsi | 4 ++--
> > arch/arm/boot/dts/imx50.dtsi | 4 ++--
> > arch/arm/boot/dts/imx51.dtsi | 4 ++--
> > arch/arm/boot/dts/imx53.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6qdl.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6sl.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6sll.dtsi | 4 ++-- arch/arm/boot/dts/imx6sx.dtsi
> > | 6 +++--- arch/arm/boot/dts/imx6ul.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6ull.dtsi | 2 +-
> > arch/arm/boot/dts/imx7s.dtsi | 6 +++---
> > 12 files changed, 25 insertions(+), 25 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/imx25.dtsi
> > b/arch/arm/boot/dts/imx25.dtsi index 40b95a290bd6..3b7a0b249d80
> 100644
> > --- a/arch/arm/boot/dts/imx25.dtsi
> > +++ b/arch/arm/boot/dts/imx25.dtsi
> > @@ -76,7 +76,7 @@
> > ranges;
> >
> > aips@43f00000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x43f00000 0x100000>;
> > @@ -333,7 +333,7 @@
> > };
> >
> > aips@53f00000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x53f00000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx31.dtsi
> > b/arch/arm/boot/dts/imx31.dtsi index 6b62f0745b82..b0e7e3bf8a1a
> 100644
> > --- a/arch/arm/boot/dts/imx31.dtsi
> > +++ b/arch/arm/boot/dts/imx31.dtsi
> > @@ -64,7 +64,7 @@
> > };
> >
> > aips@43f00000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x43f00000 0x100000>;
> > @@ -226,7 +226,7 @@
> > };
> >
> > aips@53f00000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x53f00000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx50.dtsi
> > b/arch/arm/boot/dts/imx50.dtsi index 0bfe7c91d0eb..961de09b571d
> 100644
> > --- a/arch/arm/boot/dts/imx50.dtsi
> > +++ b/arch/arm/boot/dts/imx50.dtsi
> > @@ -102,7 +102,7 @@
> > ranges;
> >
> > aips@50000000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x50000000 0x10000000>;
> > @@ -390,7 +390,7 @@
> > };
> >
> > aips@60000000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x60000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx51.dtsi
> > b/arch/arm/boot/dts/imx51.dtsi index dea86b98e9c3..86708688371b
> 100644
> > --- a/arch/arm/boot/dts/imx51.dtsi
> > +++ b/arch/arm/boot/dts/imx51.dtsi
> > @@ -159,7 +159,7 @@
> > };
> >
> > aips@70000000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x70000000 0x10000000>;
> > @@ -441,7 +441,7 @@
> > };
> >
> > aips@80000000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x80000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx53.dtsi
> > b/arch/arm/boot/dts/imx53.dtsi index ed341cfd9d09..f46a83c7d2c4
> 100644
> > --- a/arch/arm/boot/dts/imx53.dtsi
> > +++ b/arch/arm/boot/dts/imx53.dtsi
> > @@ -223,7 +223,7 @@
> > };
> >
> > aips@50000000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x50000000 0x10000000>;
> > @@ -655,7 +655,7 @@
> > };
> >
> > aips@60000000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x60000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi
> > b/arch/arm/boot/dts/imx6qdl.dtsi index e6b4b8525f98..9b7635e9cf3c
> > 100644
> > --- a/arch/arm/boot/dts/imx6qdl.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> > @@ -295,7 +295,7 @@
> > };
> >
> > aips-bus@2000000 { /* AIPS1 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02000000 0x100000>;
> > @@ -936,7 +936,7 @@
> > };
> >
> > aips-bus@2100000 { /* AIPS2 */
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sl.dtsi
> > b/arch/arm/boot/dts/imx6sl.dtsi index 59c54e6ad09a..4b4f22217dfe
> > 100644
> > --- a/arch/arm/boot/dts/imx6sl.dtsi
> > +++ b/arch/arm/boot/dts/imx6sl.dtsi
> > @@ -144,7 +144,7 @@
> > };
> >
> > aips1: aips-bus@2000000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02000000 0x100000>;
> > @@ -787,7 +787,7 @@
> > };
> >
> > aips2: aips-bus@2100000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sll.dtsi
> > b/arch/arm/boot/dts/imx6sll.dtsi index a1bc5bb31756..fac8f22255aa
> > 100644
> > --- a/arch/arm/boot/dts/imx6sll.dtsi
> > +++ b/arch/arm/boot/dts/imx6sll.dtsi
> > @@ -145,7 +145,7 @@
> > };
> >
> > aips1: aips-bus@2000000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02000000 0x100000>;
> > @@ -664,7 +664,7 @@
> > };
> >
> > aips2: aips-bus@2100000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sx.dtsi
> > b/arch/arm/boot/dts/imx6sx.dtsi index 59bad60a47dc..4499be62c8bb
> > 100644
> > --- a/arch/arm/boot/dts/imx6sx.dtsi
> > +++ b/arch/arm/boot/dts/imx6sx.dtsi
> > @@ -236,7 +236,7 @@
> > };
> >
> > aips1: aips-bus@2000000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02000000 0x100000>;
> > @@ -831,7 +831,7 @@
> > };
> >
> > aips2: aips-bus@2100000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02100000 0x100000>;
> > @@ -1189,7 +1189,7 @@
> > };
> >
> > aips3: aips-bus@2200000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02200000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6ul.dtsi
> > b/arch/arm/boot/dts/imx6ul.dtsi index d9fdca12819b..63d276fc2477
> > 100644
> > --- a/arch/arm/boot/dts/imx6ul.dtsi
> > +++ b/arch/arm/boot/dts/imx6ul.dtsi
> > @@ -205,7 +205,7 @@
> > };
> >
> > aips1: aips-bus@2000000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02000000 0x100000>;
> > @@ -772,7 +772,7 @@
> > };
> >
> > aips2: aips-bus@2100000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6ull.dtsi
> > b/arch/arm/boot/dts/imx6ull.dtsi index b7e67d121322..633fa08bc972
> > 100644
> > --- a/arch/arm/boot/dts/imx6ull.dtsi
> > +++ b/arch/arm/boot/dts/imx6ull.dtsi
> > @@ -52,7 +52,7 @@
> > / {
> > soc {
> > aips3: aips-bus@2200000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x02200000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx7s.dtsi
> > b/arch/arm/boot/dts/imx7s.dtsi index 139ab9b98472..552b14be14a1
> 100644
> > --- a/arch/arm/boot/dts/imx7s.dtsi
> > +++ b/arch/arm/boot/dts/imx7s.dtsi
> > @@ -317,7 +317,7 @@
> > };
> >
> > aips1: aips-bus@30000000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x30000000 0x400000>;
> > @@ -669,7 +669,7 @@
> > };
> >
> > aips2: aips-bus@30400000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x30400000 0x400000>;
> > @@ -809,7 +809,7 @@
> > };
> >
> > aips3: aips-bus@30800000 {
> > - compatible = "fsl,aips-bus", "simple-bus";
> > + compatible = "simple-bus";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > reg = <0x30800000 0x400000>;
> > --
> > 2.16.4
> >
_______________________________________________
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] 4+ messages in thread
* Re: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
2020-01-06 2:38 ` Peng Fan
@ 2020-01-14 14:39 ` Rob Herring
0 siblings, 0 replies; 4+ messages in thread
From: Rob Herring @ 2020-01-14 14:39 UTC (permalink / raw)
To: Peng Fan
Cc: mark.rutland, Aisheng Dong, devicetree, shawnguo, Alice Guo,
linux-kernel, dl-linux-imx, kernel, festevam, linux-arm-kernel
On Sun, Jan 5, 2020 at 8:38 PM Peng Fan <peng.fan@nxp.com> wrote:
>
> Hi Rob,
>
> > Subject: Re: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
> >
> > On Mon, Dec 30, 2019 at 08:58:05AM +0000, Peng Fan wrote:
> > > From: Peng Fan <peng.fan@nxp.com>
> > >
> > > There is no binding doc for "fsl,aips-bus", "simple-bus" is enough for
> > > aips usage, so drop it.
> >
> > NAK. The AIPS bus has registers, so 'simple-bus' alone is not enough.
>
> You mean the "reg" property below, right?
> aips-bus@2000000 { /* AIPS1 */
> - compatible = "fsl,aips-bus", "simple-bus";
> + compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x02000000 0x100000>;
>
> But the reg property is not required, I think it could be removed.
> There is no binding doc and driver for "fsl,aips-bus", so I think
> It not make sense to have that compatible in dts.
Well, there should be a binding doc, but whether or not there's a
driver is irrelevant.
From what I remember, either AIPS or SPBA bus has registers to allow
user mode access or not. Something may need to configure those and it
may want to use DT info to do that. It's not just Linux that you need
to think about.
Rob
_______________________________________________
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] 4+ messages in thread
end of thread, other threads:[~2020-01-14 14:39 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-30 8:58 [PATCH] ARM: dts: imx: drop "fsl,aips-bus" Peng Fan
2020-01-04 0:36 ` Rob Herring
2020-01-06 2:38 ` Peng Fan
2020-01-14 14:39 ` Rob Herring
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).