* [PATCH] ARM: dts: Remove "spidev" nodes
@ 2021-12-17 22:12 Rob Herring
2021-12-18 15:53 ` Nicolas Ferre
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Rob Herring @ 2021-12-17 22:12 UTC (permalink / raw)
To: soc, Nicolas Ferre, Alexandre Belloni, Ludovic Desroches,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Benoît Cousson, Tony Lindgren, Andy Gross,
Bjorn Andersson, Viresh Kumar, Shiraz Hashim
Cc: Mark Brown, devicetree, linux-arm-kernel, linux-kernel,
linux-omap, linux-arm-msm
"spidev" is not a real device, but a Linux implementation detail. It has
never been documented either. The kernel has WARNed on the use of it for
over 6 years. Time to remove its usage from the tree.
Cc: Mark Brown <broonie@kernel.org>
Signed-off-by: Rob Herring <robh@kernel.org>
---
Arnd, Olof, Can you please apply this directly.
arch/arm/boot/dts/at91-q5xr5.dts | 18 ----------
arch/arm/boot/dts/at91-wb50n.dts | 6 ----
arch/arm/boot/dts/imx28-tx28.dts | 36 -------------------
arch/arm/boot/dts/imx53-tx53.dtsi | 11 ------
arch/arm/boot/dts/imx6qdl-tx6.dtsi | 12 -------
.../boot/dts/imx6ul-imx6ull-opos6uldev.dtsi | 12 -------
arch/arm/boot/dts/imx6ul-prti6g.dts | 6 ----
arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 24 -------------
arch/arm/boot/dts/omap3-tao3530.dtsi | 14 --------
.../dts/qcom-mdm9615-wp8548-mangoh-green.dts | 2 +-
arch/arm/boot/dts/spear1310-evb.dts | 16 ---------
arch/arm/boot/dts/spear1340-evb.dts | 16 ---------
12 files changed, 1 insertion(+), 172 deletions(-)
diff --git a/arch/arm/boot/dts/at91-q5xr5.dts b/arch/arm/boot/dts/at91-q5xr5.dts
index 5827383b181b..47a00062f01f 100644
--- a/arch/arm/boot/dts/at91-q5xr5.dts
+++ b/arch/arm/boot/dts/at91-q5xr5.dts
@@ -147,12 +147,6 @@ uboot@8000 {
reg = <0x8000 0x3E000>;
};
};
-
- spidev@1 {
- compatible = "spidev";
- spi-max-frequency = <2000000>;
- reg = <1>;
- };
};
&spi1 {
@@ -160,18 +154,6 @@ &spi1 {
pinctrl-0 = <&pinctrl_spi1 &pinctrl_spi1_npcs0 &pinctrl_spi1_npcs1>;
cs-gpios = <&pioB 3 GPIO_ACTIVE_HIGH>, <&pioC 5 GPIO_ACTIVE_LOW>, <0>, <0>;
status = "okay";
-
- spidev@0 {
- compatible = "spidev";
- spi-max-frequency = <2000000>;
- reg = <0>;
- };
-
- spidev@1 {
- compatible = "spidev";
- spi-max-frequency = <2000000>;
- reg = <1>;
- };
};
&usart0 {
diff --git a/arch/arm/boot/dts/at91-wb50n.dts b/arch/arm/boot/dts/at91-wb50n.dts
index a5e45bb95c04..89f0f717f7ed 100644
--- a/arch/arm/boot/dts/at91-wb50n.dts
+++ b/arch/arm/boot/dts/at91-wb50n.dts
@@ -90,12 +90,6 @@ &i2c0 {
&spi1 {
status = "okay";
-
- spidev@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <8000000>;
- };
};
&usb0 {
diff --git a/arch/arm/boot/dts/imx28-tx28.dts b/arch/arm/boot/dts/imx28-tx28.dts
index 164254c28f8e..9e5651c7ea6b 100644
--- a/arch/arm/boot/dts/imx28-tx28.dts
+++ b/arch/arm/boot/dts/imx28-tx28.dts
@@ -239,24 +239,6 @@ &gpio3 9 GPIO_ACTIVE_LOW
>;
/* enable this and disable ssp3 below, if you need full duplex SPI transfer */
status = "disabled";
-
- spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <57600000>;
- };
-
- spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <57600000>;
- };
-
- spi@2 {
- compatible = "spidev";
- reg = <2>;
- spi-max-frequency = <57600000>;
- };
};
};
@@ -700,24 +682,6 @@ &ssp3 {
pinctrl-0 = <&spi3_pins_a>;
clock-frequency = <57600000>;
status = "okay";
-
- spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <57600000>;
- };
-
- spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <57600000>;
- };
-
- spi@2 {
- compatible = "spidev";
- reg = <2>;
- spi-max-frequency = <57600000>;
- };
};
&usb0 {
diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi
index 7c9730f3f820..81c2726a328a 100644
--- a/arch/arm/boot/dts/imx53-tx53.dtsi
+++ b/arch/arm/boot/dts/imx53-tx53.dtsi
@@ -192,17 +192,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH
&gpio3 19 GPIO_ACTIVE_HIGH
>;
- spidev0: spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <54000000>;
- };
-
- spidev1: spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <54000000>;
- };
};
&esdhc1 {
diff --git a/arch/arm/boot/dts/imx6qdl-tx6.dtsi b/arch/arm/boot/dts/imx6qdl-tx6.dtsi
index 362e65ccaa78..bcc5bbcce769 100644
--- a/arch/arm/boot/dts/imx6qdl-tx6.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-tx6.dtsi
@@ -259,18 +259,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH
&gpio3 19 GPIO_ACTIVE_HIGH
>;
status = "disabled";
-
- spidev0: spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <54000000>;
- };
-
- spidev1: spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <54000000>;
- };
};
&fec {
diff --git a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
index 935a77d717a6..18cac19aa9b0 100644
--- a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
+++ b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
@@ -114,18 +114,6 @@ &ecspi4 {
pinctrl-0 = <&pinctrl_ecspi4>;
cs-gpios = <&gpio4 9 GPIO_ACTIVE_LOW>, <&gpio4 3 GPIO_ACTIVE_LOW>;
status = "okay";
-
- spidev0: spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <5000000>;
- };
-
- spidev1: spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <5000000>;
- };
};
&i2c1 {
diff --git a/arch/arm/boot/dts/imx6ul-prti6g.dts b/arch/arm/boot/dts/imx6ul-prti6g.dts
index d62015701d0a..c18390f238e1 100644
--- a/arch/arm/boot/dts/imx6ul-prti6g.dts
+++ b/arch/arm/boot/dts/imx6ul-prti6g.dts
@@ -78,12 +78,6 @@ &ecspi2 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_ecspi2>;
status = "okay";
-
- spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <1000000>;
- };
};
&fec1 {
diff --git a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
index 938a32ced88d..c485d058e079 100644
--- a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
+++ b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
@@ -227,18 +227,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH
&gpio1 10 GPIO_ACTIVE_HIGH
>;
status = "disabled";
-
- spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <660000>;
- };
-
- spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <660000>;
- };
};
sound {
@@ -290,18 +278,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH
&gpio1 10 GPIO_ACTIVE_HIGH
>;
status = "disabled";
-
- spidev0: spi@0 {
- compatible = "spidev";
- reg = <0>;
- spi-max-frequency = <60000000>;
- };
-
- spidev1: spi@1 {
- compatible = "spidev";
- reg = <1>;
- spi-max-frequency = <60000000>;
- };
};
&fec1 {
diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi
index 580bfa1931c8..7f440d11f7e7 100644
--- a/arch/arm/boot/dts/omap3-tao3530.dtsi
+++ b/arch/arm/boot/dts/omap3-tao3530.dtsi
@@ -197,25 +197,11 @@ &i2c3 {
&mcspi1 {
pinctrl-names = "default";
pinctrl-0 = <&mcspi1_pins>;
-
- spidev@0 {
- compatible = "spidev";
- spi-max-frequency = <48000000>;
- reg = <0>;
- spi-cpha;
- };
};
&mcspi3 {
pinctrl-names = "default";
pinctrl-0 = <&mcspi3_pins>;
-
- spidev@0 {
- compatible = "spidev";
- spi-max-frequency = <48000000>;
- reg = <0>;
- spi-cpha;
- };
};
#include "twl4030.dtsi"
diff --git a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
index 942e3a2cac35..0827de5426c1 100644
--- a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
+++ b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
@@ -103,7 +103,7 @@ pins {
&gsbi3_spi {
spi@0 {
- compatible = "swir,mangoh-iotport-spi", "spidev";
+ compatible = "swir,mangoh-iotport-spi";
spi-max-frequency = <24000000>;
reg = <0>;
};
diff --git a/arch/arm/boot/dts/spear1310-evb.dts b/arch/arm/boot/dts/spear1310-evb.dts
index 8fcb6be6e7c7..4cbadcb41084 100644
--- a/arch/arm/boot/dts/spear1310-evb.dts
+++ b/arch/arm/boot/dts/spear1310-evb.dts
@@ -395,22 +395,6 @@ m25p80@1 {
pl022,wait-state = <0>;
pl022,duplex = <0>;
};
-
- spidev@2 {
- compatible = "spidev";
- reg = <2>;
- spi-max-frequency = <25000000>;
- spi-cpha;
- pl022,hierarchy = <0>;
- pl022,interface = <0>;
- pl022,slave-tx-disable;
- pl022,com-mode = <0x2>;
- pl022,rx-level-trig = <0>;
- pl022,tx-level-trig = <0>;
- pl022,ctrl-len = <0x11>;
- pl022,wait-state = <0>;
- pl022,duplex = <0>;
- };
};
wdt@ec800620 {
diff --git a/arch/arm/boot/dts/spear1340-evb.dts b/arch/arm/boot/dts/spear1340-evb.dts
index f70ff56d4542..fd194ebeedc9 100644
--- a/arch/arm/boot/dts/spear1340-evb.dts
+++ b/arch/arm/boot/dts/spear1340-evb.dts
@@ -489,22 +489,6 @@ stmpe_touchscreen {
ts,i-drive = <1>;
};
};
-
- spidev@2 {
- compatible = "spidev";
- reg = <2>;
- spi-max-frequency = <25000000>;
- spi-cpha;
- pl022,hierarchy = <0>;
- pl022,interface = <0>;
- pl022,slave-tx-disable;
- pl022,com-mode = <0x2>;
- pl022,rx-level-trig = <0>;
- pl022,tx-level-trig = <0>;
- pl022,ctrl-len = <0x11>;
- pl022,wait-state = <0>;
- pl022,duplex = <0>;
- };
};
timer@ec800600 {
--
2.32.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: Remove "spidev" nodes
2021-12-17 22:12 [PATCH] ARM: dts: Remove "spidev" nodes Rob Herring
@ 2021-12-18 15:53 ` Nicolas Ferre
2021-12-20 4:20 ` Viresh Kumar
[not found] ` <YcB3ZhbCZGmPNk5s@sirena.org.uk>
2 siblings, 0 replies; 4+ messages in thread
From: Nicolas Ferre @ 2021-12-18 15:53 UTC (permalink / raw)
To: Rob Herring, soc, Alexandre Belloni, Ludovic Desroches,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Benoît Cousson, Tony Lindgren, Andy Gross,
Bjorn Andersson, Viresh Kumar, Shiraz Hashim
Cc: Mark Brown, devicetree, linux-arm-kernel, linux-kernel,
linux-omap, linux-arm-msm
On 17/12/2021 at 23:12, Rob Herring wrote:
> "spidev" is not a real device, but a Linux implementation detail. It has
> never been documented either. The kernel has WARNed on the use of it for
> over 6 years. Time to remove its usage from the tree.
>
> Cc: Mark Brown <broonie@kernel.org>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Arnd, Olof, Can you please apply this directly.
>
> arch/arm/boot/dts/at91-q5xr5.dts | 18 ----------
> arch/arm/boot/dts/at91-wb50n.dts | 6 ----
For at91:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Sorry for having re-introduced this use recently.
Best regards,
Nicolas
> arch/arm/boot/dts/imx28-tx28.dts | 36 -------------------
> arch/arm/boot/dts/imx53-tx53.dtsi | 11 ------
> arch/arm/boot/dts/imx6qdl-tx6.dtsi | 12 -------
> .../boot/dts/imx6ul-imx6ull-opos6uldev.dtsi | 12 -------
> arch/arm/boot/dts/imx6ul-prti6g.dts | 6 ----
> arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 24 -------------
> arch/arm/boot/dts/omap3-tao3530.dtsi | 14 --------
> .../dts/qcom-mdm9615-wp8548-mangoh-green.dts | 2 +-
> arch/arm/boot/dts/spear1310-evb.dts | 16 ---------
> arch/arm/boot/dts/spear1340-evb.dts | 16 ---------
> 12 files changed, 1 insertion(+), 172 deletions(-)
>
> diff --git a/arch/arm/boot/dts/at91-q5xr5.dts b/arch/arm/boot/dts/at91-q5xr5.dts
> index 5827383b181b..47a00062f01f 100644
> --- a/arch/arm/boot/dts/at91-q5xr5.dts
> +++ b/arch/arm/boot/dts/at91-q5xr5.dts
> @@ -147,12 +147,6 @@ uboot@8000 {
> reg = <0x8000 0x3E000>;
> };
> };
> -
> - spidev@1 {
> - compatible = "spidev";
> - spi-max-frequency = <2000000>;
> - reg = <1>;
> - };
> };
>
> &spi1 {
> @@ -160,18 +154,6 @@ &spi1 {
> pinctrl-0 = <&pinctrl_spi1 &pinctrl_spi1_npcs0 &pinctrl_spi1_npcs1>;
> cs-gpios = <&pioB 3 GPIO_ACTIVE_HIGH>, <&pioC 5 GPIO_ACTIVE_LOW>, <0>, <0>;
> status = "okay";
> -
> - spidev@0 {
> - compatible = "spidev";
> - spi-max-frequency = <2000000>;
> - reg = <0>;
> - };
> -
> - spidev@1 {
> - compatible = "spidev";
> - spi-max-frequency = <2000000>;
> - reg = <1>;
> - };
> };
>
> &usart0 {
> diff --git a/arch/arm/boot/dts/at91-wb50n.dts b/arch/arm/boot/dts/at91-wb50n.dts
> index a5e45bb95c04..89f0f717f7ed 100644
> --- a/arch/arm/boot/dts/at91-wb50n.dts
> +++ b/arch/arm/boot/dts/at91-wb50n.dts
> @@ -90,12 +90,6 @@ &i2c0 {
>
> &spi1 {
> status = "okay";
> -
> - spidev@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <8000000>;
> - };
> };
>
> &usb0 {
> diff --git a/arch/arm/boot/dts/imx28-tx28.dts b/arch/arm/boot/dts/imx28-tx28.dts
> index 164254c28f8e..9e5651c7ea6b 100644
> --- a/arch/arm/boot/dts/imx28-tx28.dts
> +++ b/arch/arm/boot/dts/imx28-tx28.dts
> @@ -239,24 +239,6 @@ &gpio3 9 GPIO_ACTIVE_LOW
> >;
> /* enable this and disable ssp3 below, if you need full duplex SPI transfer */
> status = "disabled";
> -
> - spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <57600000>;
> - };
> -
> - spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <57600000>;
> - };
> -
> - spi@2 {
> - compatible = "spidev";
> - reg = <2>;
> - spi-max-frequency = <57600000>;
> - };
> };
> };
>
> @@ -700,24 +682,6 @@ &ssp3 {
> pinctrl-0 = <&spi3_pins_a>;
> clock-frequency = <57600000>;
> status = "okay";
> -
> - spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <57600000>;
> - };
> -
> - spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <57600000>;
> - };
> -
> - spi@2 {
> - compatible = "spidev";
> - reg = <2>;
> - spi-max-frequency = <57600000>;
> - };
> };
>
> &usb0 {
> diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi
> index 7c9730f3f820..81c2726a328a 100644
> --- a/arch/arm/boot/dts/imx53-tx53.dtsi
> +++ b/arch/arm/boot/dts/imx53-tx53.dtsi
> @@ -192,17 +192,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH
> &gpio3 19 GPIO_ACTIVE_HIGH
> >;
>
> - spidev0: spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <54000000>;
> - };
> -
> - spidev1: spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <54000000>;
> - };
> };
>
> &esdhc1 {
> diff --git a/arch/arm/boot/dts/imx6qdl-tx6.dtsi b/arch/arm/boot/dts/imx6qdl-tx6.dtsi
> index 362e65ccaa78..bcc5bbcce769 100644
> --- a/arch/arm/boot/dts/imx6qdl-tx6.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-tx6.dtsi
> @@ -259,18 +259,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH
> &gpio3 19 GPIO_ACTIVE_HIGH
> >;
> status = "disabled";
> -
> - spidev0: spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <54000000>;
> - };
> -
> - spidev1: spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <54000000>;
> - };
> };
>
> &fec {
> diff --git a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
> index 935a77d717a6..18cac19aa9b0 100644
> --- a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
> +++ b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi
> @@ -114,18 +114,6 @@ &ecspi4 {
> pinctrl-0 = <&pinctrl_ecspi4>;
> cs-gpios = <&gpio4 9 GPIO_ACTIVE_LOW>, <&gpio4 3 GPIO_ACTIVE_LOW>;
> status = "okay";
> -
> - spidev0: spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <5000000>;
> - };
> -
> - spidev1: spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <5000000>;
> - };
> };
>
> &i2c1 {
> diff --git a/arch/arm/boot/dts/imx6ul-prti6g.dts b/arch/arm/boot/dts/imx6ul-prti6g.dts
> index d62015701d0a..c18390f238e1 100644
> --- a/arch/arm/boot/dts/imx6ul-prti6g.dts
> +++ b/arch/arm/boot/dts/imx6ul-prti6g.dts
> @@ -78,12 +78,6 @@ &ecspi2 {
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_ecspi2>;
> status = "okay";
> -
> - spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <1000000>;
> - };
> };
>
> &fec1 {
> diff --git a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
> index 938a32ced88d..c485d058e079 100644
> --- a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
> +++ b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi
> @@ -227,18 +227,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH
> &gpio1 10 GPIO_ACTIVE_HIGH
> >;
> status = "disabled";
> -
> - spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <660000>;
> - };
> -
> - spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <660000>;
> - };
> };
>
> sound {
> @@ -290,18 +278,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH
> &gpio1 10 GPIO_ACTIVE_HIGH
> >;
> status = "disabled";
> -
> - spidev0: spi@0 {
> - compatible = "spidev";
> - reg = <0>;
> - spi-max-frequency = <60000000>;
> - };
> -
> - spidev1: spi@1 {
> - compatible = "spidev";
> - reg = <1>;
> - spi-max-frequency = <60000000>;
> - };
> };
>
> &fec1 {
> diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi
> index 580bfa1931c8..7f440d11f7e7 100644
> --- a/arch/arm/boot/dts/omap3-tao3530.dtsi
> +++ b/arch/arm/boot/dts/omap3-tao3530.dtsi
> @@ -197,25 +197,11 @@ &i2c3 {
> &mcspi1 {
> pinctrl-names = "default";
> pinctrl-0 = <&mcspi1_pins>;
> -
> - spidev@0 {
> - compatible = "spidev";
> - spi-max-frequency = <48000000>;
> - reg = <0>;
> - spi-cpha;
> - };
> };
>
> &mcspi3 {
> pinctrl-names = "default";
> pinctrl-0 = <&mcspi3_pins>;
> -
> - spidev@0 {
> - compatible = "spidev";
> - spi-max-frequency = <48000000>;
> - reg = <0>;
> - spi-cpha;
> - };
> };
>
> #include "twl4030.dtsi"
> diff --git a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
> index 942e3a2cac35..0827de5426c1 100644
> --- a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
> +++ b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts
> @@ -103,7 +103,7 @@ pins {
>
> &gsbi3_spi {
> spi@0 {
> - compatible = "swir,mangoh-iotport-spi", "spidev";
> + compatible = "swir,mangoh-iotport-spi";
> spi-max-frequency = <24000000>;
> reg = <0>;
> };
> diff --git a/arch/arm/boot/dts/spear1310-evb.dts b/arch/arm/boot/dts/spear1310-evb.dts
> index 8fcb6be6e7c7..4cbadcb41084 100644
> --- a/arch/arm/boot/dts/spear1310-evb.dts
> +++ b/arch/arm/boot/dts/spear1310-evb.dts
> @@ -395,22 +395,6 @@ m25p80@1 {
> pl022,wait-state = <0>;
> pl022,duplex = <0>;
> };
> -
> - spidev@2 {
> - compatible = "spidev";
> - reg = <2>;
> - spi-max-frequency = <25000000>;
> - spi-cpha;
> - pl022,hierarchy = <0>;
> - pl022,interface = <0>;
> - pl022,slave-tx-disable;
> - pl022,com-mode = <0x2>;
> - pl022,rx-level-trig = <0>;
> - pl022,tx-level-trig = <0>;
> - pl022,ctrl-len = <0x11>;
> - pl022,wait-state = <0>;
> - pl022,duplex = <0>;
> - };
> };
>
> wdt@ec800620 {
> diff --git a/arch/arm/boot/dts/spear1340-evb.dts b/arch/arm/boot/dts/spear1340-evb.dts
> index f70ff56d4542..fd194ebeedc9 100644
> --- a/arch/arm/boot/dts/spear1340-evb.dts
> +++ b/arch/arm/boot/dts/spear1340-evb.dts
> @@ -489,22 +489,6 @@ stmpe_touchscreen {
> ts,i-drive = <1>;
> };
> };
> -
> - spidev@2 {
> - compatible = "spidev";
> - reg = <2>;
> - spi-max-frequency = <25000000>;
> - spi-cpha;
> - pl022,hierarchy = <0>;
> - pl022,interface = <0>;
> - pl022,slave-tx-disable;
> - pl022,com-mode = <0x2>;
> - pl022,rx-level-trig = <0>;
> - pl022,tx-level-trig = <0>;
> - pl022,ctrl-len = <0x11>;
> - pl022,wait-state = <0>;
> - pl022,duplex = <0>;
> - };
> };
>
> timer@ec800600 {
> --
> 2.32.0
>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: Remove "spidev" nodes
2021-12-17 22:12 [PATCH] ARM: dts: Remove "spidev" nodes Rob Herring
2021-12-18 15:53 ` Nicolas Ferre
@ 2021-12-20 4:20 ` Viresh Kumar
[not found] ` <YcB3ZhbCZGmPNk5s@sirena.org.uk>
2 siblings, 0 replies; 4+ messages in thread
From: Viresh Kumar @ 2021-12-20 4:20 UTC (permalink / raw)
To: Rob Herring
Cc: soc, Nicolas Ferre, Alexandre Belloni, Ludovic Desroches,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Benoît Cousson, Tony Lindgren, Andy Gross,
Bjorn Andersson, Viresh Kumar, Shiraz Hashim, Mark Brown,
devicetree, linux-arm-kernel, linux-kernel, linux-omap,
linux-arm-msm
On 17-12-21, 16:12, Rob Herring wrote:
> "spidev" is not a real device, but a Linux implementation detail. It has
> never been documented either. The kernel has WARNed on the use of it for
> over 6 years. Time to remove its usage from the tree.
>
> Cc: Mark Brown <broonie@kernel.org>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Arnd, Olof, Can you please apply this directly.
>
> arch/arm/boot/dts/spear1310-evb.dts | 16 ---------
> arch/arm/boot/dts/spear1340-evb.dts | 16 ---------
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
viresh
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: Remove "spidev" nodes
[not found] ` <YcB3ZhbCZGmPNk5s@sirena.org.uk>
@ 2021-12-20 19:22 ` Alexandre Belloni
0 siblings, 0 replies; 4+ messages in thread
From: Alexandre Belloni @ 2021-12-20 19:22 UTC (permalink / raw)
To: Mark Brown
Cc: Rob Herring, soc, Nicolas Ferre, Ludovic Desroches, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Benoît Cousson, Tony Lindgren, Andy Gross,
Bjorn Andersson, Viresh Kumar, Shiraz Hashim, devicetree,
linux-arm-kernel, linux-kernel, linux-omap, linux-arm-msm
Hello Mark,
On 20/12/2021 12:30:30+0000, Mark Brown wrote:
> On Fri, Dec 17, 2021 at 04:12:32PM -0600, Rob Herring wrote:
> > "spidev" is not a real device, but a Linux implementation detail. It has
> > never been documented either. The kernel has WARNed on the use of it for
> > over 6 years. Time to remove its usage from the tree.
>
> Reviwed-by: Mark Brown <broonie@kernel.org>
You have a typo there so I'm not sure b4 will be able to pick that up
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-12-20 19:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-17 22:12 [PATCH] ARM: dts: Remove "spidev" nodes Rob Herring
2021-12-18 15:53 ` Nicolas Ferre
2021-12-20 4:20 ` Viresh Kumar
[not found] ` <YcB3ZhbCZGmPNk5s@sirena.org.uk>
2021-12-20 19:22 ` Alexandre Belloni
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).