* [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port"
@ 2020-10-15 3:55 Jisheng Zhang
2020-10-15 3:56 ` [PATCH 1/8] ARM: dts: berlin: Remove compatible string from dw apb gpio port Jisheng Zhang
` (8 more replies)
0 siblings, 9 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:55 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The "snps,dw-apb-gpio-port" is never used.
Jisheng Zhang (8):
ARM: dts: berlin: Remove compatible string from dw apb gpio port
ARM: dts: socfpga: Remove compatible string from dw apb gpio port
arm64: dts: apm: remove compatible string from dw apb gpio port
arm64: dts: bitmain: Remove compatible string from dw apb gpio port
arm64: dts: hip05: Remove compatible string from dw apb gpio port
arm: dts: socfpga: Remove compatible string from dw apb gpio port
arm64: dts: synaptics: Remove compatible string from dw apb gpio port
dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
.../devicetree/bindings/gpio/snps,dw-apb-gpio.yaml | 5 -----
arch/arm/boot/dts/berlin2.dtsi | 6 ------
arch/arm/boot/dts/berlin2cd.dtsi | 6 ------
arch/arm/boot/dts/berlin2q.dtsi | 6 ------
arch/arm/boot/dts/socfpga.dtsi | 3 ---
arch/arm/boot/dts/socfpga_arria10.dtsi | 3 ---
arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi | 2 --
arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 1 -
arch/arm64/boot/dts/apm/apm-storm.dtsi | 1 -
arch/arm64/boot/dts/bitmain/bm1880.dtsi | 3 ---
arch/arm64/boot/dts/hisilicon/hip05.dtsi | 2 --
arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 2 --
arch/arm64/boot/dts/synaptics/as370.dtsi | 2 --
arch/arm64/boot/dts/synaptics/berlin4ct.dtsi | 6 ------
14 files changed, 48 deletions(-)
--
2.28.0
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/8] ARM: dts: berlin: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
@ 2020-10-15 3:56 ` Jisheng Zhang
2020-10-15 3:56 ` [PATCH 2/8] ARM: dts: socfpga: " Jisheng Zhang
` (7 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:56 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm/boot/dts/berlin2.dtsi | 6 ------
arch/arm/boot/dts/berlin2cd.dtsi | 6 ------
arch/arm/boot/dts/berlin2q.dtsi | 6 ------
3 files changed, 18 deletions(-)
diff --git a/arch/arm/boot/dts/berlin2.dtsi b/arch/arm/boot/dts/berlin2.dtsi
index 6194857f8a02..02c9b8b5b633 100644
--- a/arch/arm/boot/dts/berlin2.dtsi
+++ b/arch/arm/boot/dts/berlin2.dtsi
@@ -188,7 +188,6 @@ gpio0: gpio@400 {
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -206,7 +205,6 @@ gpio1: gpio@800 {
#size-cells = <0>;
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -224,7 +222,6 @@ gpio2: gpio@c00 {
#size-cells = <0>;
portc: gpio-port@2 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -242,7 +239,6 @@ gpio3: gpio@1000 {
#size-cells = <0>;
portd: gpio-port@3 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -443,7 +439,6 @@ sm_gpio1: gpio@5000 {
#size-cells = <0>;
portf: gpio-port@5 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -458,7 +453,6 @@ sm_gpio0: gpio@c000 {
#size-cells = <0>;
porte: gpio-port@4 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
diff --git a/arch/arm/boot/dts/berlin2cd.dtsi b/arch/arm/boot/dts/berlin2cd.dtsi
index 6f30d7eb3b41..f7698ab97abf 100644
--- a/arch/arm/boot/dts/berlin2cd.dtsi
+++ b/arch/arm/boot/dts/berlin2cd.dtsi
@@ -178,7 +178,6 @@ gpio0: gpio@400 {
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -196,7 +195,6 @@ gpio1: gpio@800 {
#size-cells = <0>;
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -214,7 +212,6 @@ gpio2: gpio@c00 {
#size-cells = <0>;
portc: gpio-port@2 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -232,7 +229,6 @@ gpio3: gpio@1000 {
#size-cells = <0>;
portd: gpio-port@3 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -470,7 +466,6 @@ sm_gpio1: gpio@5000 {
#size-cells = <0>;
portf: gpio-port@5 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
@@ -515,7 +510,6 @@ sm_gpio0: gpio@c000 {
#size-cells = <0>;
porte: gpio-port@4 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
diff --git a/arch/arm/boot/dts/berlin2q.dtsi b/arch/arm/boot/dts/berlin2q.dtsi
index b6a0acac6836..831505b47e27 100644
--- a/arch/arm/boot/dts/berlin2q.dtsi
+++ b/arch/arm/boot/dts/berlin2q.dtsi
@@ -249,7 +249,6 @@ gpio0: gpio@400 {
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -267,7 +266,6 @@ gpio1: gpio@800 {
#size-cells = <0>;
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -285,7 +283,6 @@ gpio2: gpio@c00 {
#size-cells = <0>;
portc: gpio-port@2 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -303,7 +300,6 @@ gpio3: gpio@1000 {
#size-cells = <0>;
portd: gpio-port@3 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -549,7 +545,6 @@ sm_gpio1: gpio@5000 {
#size-cells = <0>;
portf: gpio-port@5 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -610,7 +605,6 @@ sm_gpio0: gpio@c000 {
#size-cells = <0>;
porte: gpio-port@4 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 2/8] ARM: dts: socfpga: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
2020-10-15 3:56 ` [PATCH 1/8] ARM: dts: berlin: Remove compatible string from dw apb gpio port Jisheng Zhang
@ 2020-10-15 3:56 ` Jisheng Zhang
2020-10-15 3:56 ` [PATCH 3/8] arm64: dts: apm: remove " Jisheng Zhang
` (6 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:56 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm/boot/dts/socfpga.dtsi | 3 ---
arch/arm/boot/dts/socfpga_arria10.dtsi | 3 ---
arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi | 2 --
3 files changed, 8 deletions(-)
diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index 0b021eef0b53..2c06f1fa3d26 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -609,7 +609,6 @@ gpio0: gpio@ff708000 {
status = "disabled";
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <29>;
@@ -630,7 +629,6 @@ gpio1: gpio@ff709000 {
status = "disabled";
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <29>;
@@ -651,7 +649,6 @@ gpio2: gpio@ff70a000 {
status = "disabled";
portc: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <27>;
diff --git a/arch/arm/boot/dts/socfpga_arria10.dtsi b/arch/arm/boot/dts/socfpga_arria10.dtsi
index 0013ec3463c4..a52132d3c62f 100644
--- a/arch/arm/boot/dts/socfpga_arria10.dtsi
+++ b/arch/arm/boot/dts/socfpga_arria10.dtsi
@@ -488,7 +488,6 @@ gpio0: gpio@ffc02900 {
status = "disabled";
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <29>;
@@ -508,7 +507,6 @@ gpio1: gpio@ffc02a00 {
status = "disabled";
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <29>;
@@ -528,7 +526,6 @@ gpio2: gpio@ffc02b00 {
status = "disabled";
portc: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <27>;
diff --git a/arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi b/arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi
index a6fb01c7ab34..5da1a69d6e1d 100644
--- a/arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi
+++ b/arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi
@@ -200,7 +200,6 @@ gpio0: gpio@ffc03200 {
status = "disabled";
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <24>;
@@ -220,7 +219,6 @@ gpio1: gpio@ffc03300 {
status = "disabled";
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <24>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 3/8] arm64: dts: apm: remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
2020-10-15 3:56 ` [PATCH 1/8] ARM: dts: berlin: Remove compatible string from dw apb gpio port Jisheng Zhang
2020-10-15 3:56 ` [PATCH 2/8] ARM: dts: socfpga: " Jisheng Zhang
@ 2020-10-15 3:56 ` Jisheng Zhang
2020-10-15 3:57 ` [PATCH 4/8] arm64: dts: bitmain: Remove " Jisheng Zhang
` (5 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:56 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 1 -
arch/arm64/boot/dts/apm/apm-storm.dtsi | 1 -
2 files changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi b/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
index 3feb1881bbc2..cc4265cc13f6 100644
--- a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
+++ b/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
@@ -714,7 +714,6 @@ dwgpio: gpio@1c024000 {
#size-cells = <0>;
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
snps,nr-gpios = <32>;
reg = <0>;
diff --git a/arch/arm64/boot/dts/apm/apm-storm.dtsi b/arch/arm64/boot/dts/apm/apm-storm.dtsi
index 8c802d87e751..64b86b9d1b37 100644
--- a/arch/arm64/boot/dts/apm/apm-storm.dtsi
+++ b/arch/arm64/boot/dts/apm/apm-storm.dtsi
@@ -830,7 +830,6 @@ dwgpio: gpio@1c024000 {
#size-cells = <0>;
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
snps,nr-gpios = <32>;
reg = <0>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 4/8] arm64: dts: bitmain: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (2 preceding siblings ...)
2020-10-15 3:56 ` [PATCH 3/8] arm64: dts: apm: remove " Jisheng Zhang
@ 2020-10-15 3:57 ` Jisheng Zhang
2020-10-15 3:57 ` [PATCH 5/8] arm64: dts: hip05: " Jisheng Zhang
` (4 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:57 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm64/boot/dts/bitmain/bm1880.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/bitmain/bm1880.dtsi b/arch/arm64/boot/dts/bitmain/bm1880.dtsi
index fa6e6905f588..f80373358bba 100644
--- a/arch/arm64/boot/dts/bitmain/bm1880.dtsi
+++ b/arch/arm64/boot/dts/bitmain/bm1880.dtsi
@@ -124,7 +124,6 @@ gpio0: gpio@50027000 {
reg = <0x0 0x50027000 0x0 0x400>;
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -142,7 +141,6 @@ gpio1: gpio@50027400 {
reg = <0x0 0x50027400 0x0 0x400>;
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -160,7 +158,6 @@ gpio2: gpio@50027800 {
reg = <0x0 0x50027800 0x0 0x400>;
portc: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <8>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 5/8] arm64: dts: hip05: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (3 preceding siblings ...)
2020-10-15 3:57 ` [PATCH 4/8] arm64: dts: bitmain: Remove " Jisheng Zhang
@ 2020-10-15 3:57 ` Jisheng Zhang
2020-10-15 3:58 ` [PATCH 6/8] arm: dts: socfpga: " Jisheng Zhang
` (3 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:57 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm64/boot/dts/hisilicon/hip05.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/hisilicon/hip05.dtsi b/arch/arm64/boot/dts/hisilicon/hip05.dtsi
index bc49955360db..390225f8979c 100644
--- a/arch/arm64/boot/dts/hisilicon/hip05.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hip05.dtsi
@@ -332,7 +332,6 @@ peri_gpio0: gpio@802e0000 {
status = "disabled";
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -351,7 +350,6 @@ peri_gpio1: gpio@802f0000 {
status = "disabled";
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 6/8] arm: dts: socfpga: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (4 preceding siblings ...)
2020-10-15 3:57 ` [PATCH 5/8] arm64: dts: hip05: " Jisheng Zhang
@ 2020-10-15 3:58 ` Jisheng Zhang
2020-10-15 3:58 ` [PATCH 7/8] arm64: dts: synaptics: " Jisheng Zhang
` (2 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:58 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
index 9d7f19e97df7..757c53ffe345 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
@@ -199,7 +199,6 @@ gpio0: gpio@ffc03200 {
status = "disabled";
porta: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <24>;
@@ -219,7 +218,6 @@ gpio1: gpio@ffc03300 {
status = "disabled";
portb: gpio-controller@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <24>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 7/8] arm64: dts: synaptics: Remove compatible string from dw apb gpio port
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (5 preceding siblings ...)
2020-10-15 3:58 ` [PATCH 6/8] arm: dts: socfpga: " Jisheng Zhang
@ 2020-10-15 3:58 ` Jisheng Zhang
2020-10-15 3:59 ` [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
2020-10-15 11:40 ` [PATCH 0/8] " Serge Semin
8 siblings, 0 replies; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:58 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
arch/arm64/boot/dts/synaptics/as370.dtsi | 2 --
arch/arm64/boot/dts/synaptics/berlin4ct.dtsi | 6 ------
2 files changed, 8 deletions(-)
diff --git a/arch/arm64/boot/dts/synaptics/as370.dtsi b/arch/arm64/boot/dts/synaptics/as370.dtsi
index addeb0efc616..f29b21f930b9 100644
--- a/arch/arm64/boot/dts/synaptics/as370.dtsi
+++ b/arch/arm64/boot/dts/synaptics/as370.dtsi
@@ -140,7 +140,6 @@ gpio0: gpio@1800 {
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -158,7 +157,6 @@ gpio1: gpio@2000 {
#size-cells = <0>;
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
diff --git a/arch/arm64/boot/dts/synaptics/berlin4ct.dtsi b/arch/arm64/boot/dts/synaptics/berlin4ct.dtsi
index 15625b99e336..4b563c0e015e 100644
--- a/arch/arm64/boot/dts/synaptics/berlin4ct.dtsi
+++ b/arch/arm64/boot/dts/synaptics/berlin4ct.dtsi
@@ -137,7 +137,6 @@ gpio0: gpio@400 {
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -155,7 +154,6 @@ gpio1: gpio@800 {
#size-cells = <0>;
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -173,7 +171,6 @@ gpio2: gpio@c00 {
#size-cells = <0>;
portc: gpio-port@2 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -191,7 +188,6 @@ gpio3: gpio@1000 {
#size-cells = <0>;
portd: gpio-port@3 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -266,7 +262,6 @@ sm_gpio0: gpio@8000 {
#size-cells = <0>;
porte: gpio-port@4 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
@@ -281,7 +276,6 @@ sm_gpio1: gpio@9000 {
#size-cells = <0>;
portf: gpio-port@5 {
- compatible = "snps,dw-apb-gpio-port";
gpio-controller;
#gpio-cells = <2>;
snps,nr-gpios = <32>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (6 preceding siblings ...)
2020-10-15 3:58 ` [PATCH 7/8] arm64: dts: synaptics: " Jisheng Zhang
@ 2020-10-15 3:59 ` Jisheng Zhang
2020-10-15 12:04 ` Serge Semin
2020-10-15 11:40 ` [PATCH 0/8] " Serge Semin
8 siblings, 1 reply; 13+ messages in thread
From: Jisheng Zhang @ 2020-10-15 3:59 UTC (permalink / raw)
To: Hoan Tran, Serge Semin, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel
The compatible string is never used.
Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
---
Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml | 5 -----
1 file changed, 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
index 1240f6289249..76e7b07d592f 100644
--- a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
@@ -50,9 +50,6 @@ patternProperties:
"^gpio-(port|controller)@[0-9a-f]+$":
type: object
properties:
- compatible:
- const: snps,dw-apb-gpio-port
-
reg:
maxItems: 1
@@ -111,7 +108,6 @@ examples:
#size-cells = <0>;
porta: gpio-port@0 {
- compatible = "snps,dw-apb-gpio-port";
reg = <0>;
gpio-controller;
#gpio-cells = <2>;
@@ -123,7 +119,6 @@ examples:
};
portb: gpio-port@1 {
- compatible = "snps,dw-apb-gpio-port";
reg = <1>;
gpio-controller;
#gpio-cells = <2>;
--
2.28.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port"
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
` (7 preceding siblings ...)
2020-10-15 3:59 ` [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
@ 2020-10-15 11:40 ` Serge Semin
8 siblings, 0 replies; 13+ messages in thread
From: Serge Semin @ 2020-10-15 11:40 UTC (permalink / raw)
To: Jisheng Zhang
Cc: Hoan Tran, Linus Walleij, Bartosz Golaszewski, Rob Herring,
Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu, linux-gpio, devicetree,
linux-kernel, linux-arm-kernel
Hello Jisheng,
Thanks for sending this cleanup and for the work you've done in its framework.
The compatible property is indeed redundant in the DW APB GPIO bindings, because
any sub-node is considered as one of the device port from the set {A,B,C,D} and I
don't see why would we need to be there for anything else. The port index is
initialized by the reg property, which I've just realized lack of a proper
constraints.
Anyway see my comment to the DT binding file. The rest of the series is up to
the corresponding subsystem maintainers.
-Sergey
On Thu, Oct 15, 2020 at 11:55:24AM +0800, Jisheng Zhang wrote:
> The "snps,dw-apb-gpio-port" is never used.
>
> Jisheng Zhang (8):
> ARM: dts: berlin: Remove compatible string from dw apb gpio port
> ARM: dts: socfpga: Remove compatible string from dw apb gpio port
> arm64: dts: apm: remove compatible string from dw apb gpio port
> arm64: dts: bitmain: Remove compatible string from dw apb gpio port
> arm64: dts: hip05: Remove compatible string from dw apb gpio port
> arm: dts: socfpga: Remove compatible string from dw apb gpio port
> arm64: dts: synaptics: Remove compatible string from dw apb gpio port
> dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
>
> .../devicetree/bindings/gpio/snps,dw-apb-gpio.yaml | 5 -----
> arch/arm/boot/dts/berlin2.dtsi | 6 ------
> arch/arm/boot/dts/berlin2cd.dtsi | 6 ------
> arch/arm/boot/dts/berlin2q.dtsi | 6 ------
> arch/arm/boot/dts/socfpga.dtsi | 3 ---
> arch/arm/boot/dts/socfpga_arria10.dtsi | 3 ---
> arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi | 2 --
> arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 1 -
> arch/arm64/boot/dts/apm/apm-storm.dtsi | 1 -
> arch/arm64/boot/dts/bitmain/bm1880.dtsi | 3 ---
> arch/arm64/boot/dts/hisilicon/hip05.dtsi | 2 --
> arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 2 --
> arch/arm64/boot/dts/synaptics/as370.dtsi | 2 --
> arch/arm64/boot/dts/synaptics/berlin4ct.dtsi | 6 ------
> 14 files changed, 48 deletions(-)
>
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
2020-10-15 3:59 ` [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
@ 2020-10-15 12:04 ` Serge Semin
2020-10-16 14:43 ` Andy Shevchenko
0 siblings, 1 reply; 13+ messages in thread
From: Serge Semin @ 2020-10-15 12:04 UTC (permalink / raw)
To: Jisheng Zhang
Cc: Hoan Tran, Linus Walleij, Bartosz Golaszewski, Rob Herring,
Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu, linux-gpio, devicetree,
linux-kernel, linux-arm-kernel
On Thu, Oct 15, 2020 at 11:59:03AM +0800, Jisheng Zhang wrote:
> The compatible string is never used.
Before sending v2 could you run "make dt_binding_check" for the DT schema
modified in this patch? Like this:
make -j8 ARCH=mips CROSS_COMPILE=mipsel-baikal-linux- dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
>
> Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
> ---
> Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> index 1240f6289249..76e7b07d592f 100644
> --- a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> @@ -50,9 +50,6 @@ patternProperties:
> "^gpio-(port|controller)@[0-9a-f]+$":
> type: object
> properties:
> - compatible:
> - const: snps,dw-apb-gpio-port
> -
Yep, you've removed the "compatible" property but forgot to do the same from the
"required:" property of the sub-nodes schema. That's why the suggested
make-command above would have failed during the DT schema examples validation if
you ran it. Please, fix it and resend the series.
> reg:
> maxItems: 1
Yeah. Here I've forgotten to add the "reg" property constraints. Since certainly
the controller won't have more than four sub-nodes by design, could you add one
in an additional patch of the series since you'll need to resend the patchset anyway?
Like this:
reg:
- maxItems: 1
+ minimum: 0
+ maximum: 3
-Sergey
>
> @@ -111,7 +108,6 @@ examples:
> #size-cells = <0>;
>
> porta: gpio-port@0 {
> - compatible = "snps,dw-apb-gpio-port";
> reg = <0>;
> gpio-controller;
> #gpio-cells = <2>;
> @@ -123,7 +119,6 @@ examples:
> };
>
> portb: gpio-port@1 {
> - compatible = "snps,dw-apb-gpio-port";
> reg = <1>;
> gpio-controller;
> #gpio-cells = <2>;
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
2020-10-15 12:04 ` Serge Semin
@ 2020-10-16 14:43 ` Andy Shevchenko
2020-10-16 16:24 ` Serge Semin
0 siblings, 1 reply; 13+ messages in thread
From: Andy Shevchenko @ 2020-10-16 14:43 UTC (permalink / raw)
To: Serge Semin
Cc: Jisheng Zhang, Hoan Tran, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu, open list:GPIO SUBSYSTEM,
devicetree, Linux Kernel Mailing List, linux-arm Mailing List
On Thu, Oct 15, 2020 at 3:33 PM Serge Semin <fancer.lancer@gmail.com> wrote:
> On Thu, Oct 15, 2020 at 11:59:03AM +0800, Jisheng Zhang wrote:
> > The compatible string is never used.
>
> Before sending v2 could you run "make dt_binding_check" for the DT schema
> modified in this patch? Like this:
>
> make -j8 ARCH=mips CROSS_COMPILE=mipsel-baikal-linux- dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
Do you really need to install all cross compilation tools for that?
I think it should simple work as
% make dt_binding_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port"
2020-10-16 14:43 ` Andy Shevchenko
@ 2020-10-16 16:24 ` Serge Semin
0 siblings, 0 replies; 13+ messages in thread
From: Serge Semin @ 2020-10-16 16:24 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Jisheng Zhang, Hoan Tran, Linus Walleij, Bartosz Golaszewski,
Rob Herring, Sebastian Hesselbarth, Dinh Nguyen, Khuong Dinh,
Manivannan Sadhasivam, Wei Xu, open list:GPIO SUBSYSTEM,
devicetree, Linux Kernel Mailing List, linux-arm Mailing List
On Fri, Oct 16, 2020 at 05:43:08PM +0300, Andy Shevchenko wrote:
> On Thu, Oct 15, 2020 at 3:33 PM Serge Semin <fancer.lancer@gmail.com> wrote:
> > On Thu, Oct 15, 2020 at 11:59:03AM +0800, Jisheng Zhang wrote:
> > > The compatible string is never used.
> >
> > Before sending v2 could you run "make dt_binding_check" for the DT schema
> > modified in this patch? Like this:
> >
> > make -j8 ARCH=mips CROSS_COMPILE=mipsel-baikal-linux- dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
>
> Do you really need to install all cross compilation tools for that?
> I think it should simple work as
>
> % make dt_binding_check
> DT_SCHEMA_FILES=Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
Of course you don't. I just cited my normal command. Your version should be
enough to perform the bindings file validation. Note that there are host
tools/libs requirements, which need to be satisfied to make that command
working. See the next file for details:
Documentation/devicetree/writing-schema.rst
-Sergey
>
> --
> With Best Regards,
> Andy Shevchenko
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-10-16 16:24 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-15 3:55 [PATCH 0/8] remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
2020-10-15 3:56 ` [PATCH 1/8] ARM: dts: berlin: Remove compatible string from dw apb gpio port Jisheng Zhang
2020-10-15 3:56 ` [PATCH 2/8] ARM: dts: socfpga: " Jisheng Zhang
2020-10-15 3:56 ` [PATCH 3/8] arm64: dts: apm: remove " Jisheng Zhang
2020-10-15 3:57 ` [PATCH 4/8] arm64: dts: bitmain: Remove " Jisheng Zhang
2020-10-15 3:57 ` [PATCH 5/8] arm64: dts: hip05: " Jisheng Zhang
2020-10-15 3:58 ` [PATCH 6/8] arm: dts: socfpga: " Jisheng Zhang
2020-10-15 3:58 ` [PATCH 7/8] arm64: dts: synaptics: " Jisheng Zhang
2020-10-15 3:59 ` [PATCH 8/8] dt-bindings: gpio: dw-apb: remove never-used "snps,dw-apb-gpio-port" Jisheng Zhang
2020-10-15 12:04 ` Serge Semin
2020-10-16 14:43 ` Andy Shevchenko
2020-10-16 16:24 ` Serge Semin
2020-10-15 11:40 ` [PATCH 0/8] " Serge Semin
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).