[PATCH 1/2] dts: synquacer: Drop unused and undocumented SPI properties

'active_clk_edges' and 'chipselect_num' SPI controller properties are unused in u-boot and Linux. They are also not documented in the binding. So drop them.
Signed-off-by: Rob Herring robh@kernel.org --- arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 6b85c05458d4..9f9837b33bef 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -20,8 +20,6 @@ #address-cells = <1>; #size-cells = <0>; status = "okay"; - active_clk_edges; - chipselect_num = <1>;
flash@0 { #address-cells = <1>;

The 'base' GPIO controller property is unused in u-boot and Linux. It is also not documented in the binding. So drop it.
Signed-off-by: Rob Herring robh@kernel.org --- arch/arm/dts/synquacer-sc2a11.dtsi | 1 - 1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 0dd2969b5e3c..7ba1cd1bee70 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -497,7 +497,6 @@ gpio-controller; #gpio-cells = <2>; clocks = <&clk_apb>; - base = <0>; };
exiu: interrupt-controller@510c0000 {

+ Jassi
On Fri, 9 Dec 2022 at 11:45, Rob Herring robh@kernel.org wrote:
The 'base' GPIO controller property is unused in u-boot and Linux. It is also not documented in the binding. So drop it.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11.dtsi | 1 - 1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 0dd2969b5e3c..7ba1cd1bee70 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -497,7 +497,6 @@ gpio-controller; #gpio-cells = <2>; clocks = <&clk_apb>;
base = <0>;
Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org
Regards, Masahisa Kojima
}; exiu: interrupt-controller@510c0000 {
-- 2.35.1

On Thu, 8 Dec 2022 at 23:10, Masahisa Kojima masahisa.kojima@linaro.org wrote:
- Jassi
On Fri, 9 Dec 2022 at 11:45, Rob Herring robh@kernel.org wrote:
The 'base' GPIO controller property is unused in u-boot and Linux. It is also not documented in the binding. So drop it.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11.dtsi | 1 - 1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 0dd2969b5e3c..7ba1cd1bee70 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -497,7 +497,6 @@ gpio-controller; #gpio-cells = <2>; clocks = <&clk_apb>;
base = <0>;
Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org
Acked-by: Jassi Brar jaswinder.singh@linaro.org

On Thu, Dec 08, 2022 at 08:45:09PM -0600, Rob Herring wrote:
The 'base' GPIO controller property is unused in u-boot and Linux. It is also not documented in the binding. So drop it.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11.dtsi | 1 - 1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 0dd2969b5e3c..7ba1cd1bee70 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -497,7 +497,6 @@ gpio-controller; #gpio-cells = <2>; clocks = <&clk_apb>;
base = <0>;
};
exiu: interrupt-controller@510c0000 {
-- 2.35.1
Acked-by: Ilias Apalodimas ilias.apalodimas@linaro.org

On Thu, Dec 08, 2022 at 08:45:09PM -0600, Rob Herring wrote:
The 'base' GPIO controller property is unused in u-boot and Linux. It is also not documented in the binding. So drop it.
Signed-off-by: Rob Herring robh@kernel.org Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org Acked-by: Ilias Apalodimas ilias.apalodimas@linaro.org Acked-by: Jassi Brar jaswinder.singh@linaro.org
Applied to u-boot/next, thanks!

+ Jassi
On Fri, 9 Dec 2022 at 11:45, Rob Herring robh@kernel.org wrote:
'active_clk_edges' and 'chipselect_num' SPI controller properties are unused in u-boot and Linux. They are also not documented in the binding. So drop them.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 6b85c05458d4..9f9837b33bef 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -20,8 +20,6 @@ #address-cells = <1>; #size-cells = <0>; status = "okay";
active_clk_edges;
chipselect_num = <1>;
Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org
Thank you for cleaning up the unused properties.
Regards, Masahisa Kojima
flash@0 { #address-cells = <1>;
-- 2.35.1

On Thu, 8 Dec 2022 at 23:09, Masahisa Kojima masahisa.kojima@linaro.org wrote:
- Jassi
On Fri, 9 Dec 2022 at 11:45, Rob Herring robh@kernel.org wrote:
'active_clk_edges' and 'chipselect_num' SPI controller properties are unused in u-boot and Linux. They are also not documented in the binding. So drop them.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 6b85c05458d4..9f9837b33bef 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -20,8 +20,6 @@ #address-cells = <1>; #size-cells = <0>; status = "okay";
active_clk_edges;
chipselect_num = <1>;
Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org
Acked-by: Jassi Brar jaswinder.singh@linaro.org

On Thu, Dec 08, 2022 at 08:45:08PM -0600, Rob Herring wrote:
'active_clk_edges' and 'chipselect_num' SPI controller properties are unused in u-boot and Linux. They are also not documented in the binding. So drop them.
Signed-off-by: Rob Herring robh@kernel.org
arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 6b85c05458d4..9f9837b33bef 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -20,8 +20,6 @@ #address-cells = <1>; #size-cells = <0>; status = "okay";
active_clk_edges;
chipselect_num = <1>;
flash@0 { #address-cells = <1>;
-- 2.35.1
Acked-by: Ilias Apalodimas ilias.apalodimas@linaro.org

On Thu, Dec 08, 2022 at 08:45:08PM -0600, Rob Herring wrote:
'active_clk_edges' and 'chipselect_num' SPI controller properties are unused in u-boot and Linux. They are also not documented in the binding. So drop them.
Signed-off-by: Rob Herring robh@kernel.org Reviewed-by: Masahisa Kojima masahisa.kojima@linaro.org Acked-by: Ilias Apalodimas ilias.apalodimas@linaro.org Acked-by: Jassi Brar jaswinder.singh@linaro.org
Applied to u-boot/next, thanks!
participants (5)
-
Ilias Apalodimas
-
Jassi Brar
-
Masahisa Kojima
-
Rob Herring
-
Tom Rini