From: Nicolas Ferre <nicolas.ferre@microchip.com>
To: "Rob Herring" <robh@kernel.org>,
soc@kernel.org,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Ludovic Desroches" <ludovic.desroches@microchip.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Fabio Estevam" <festevam@gmail.com>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Benoît Cousson" <bcousson@baylibre.com>,
"Tony Lindgren" <tony@atomide.com>,
"Andy Gross" <agross@kernel.org>,
"Bjorn Andersson" <bjorn.andersson@linaro.org>,
"Viresh Kumar" <vireshk@kernel.org>,
"Shiraz Hashim" <shiraz.linux.kernel@gmail.com>
Cc: Mark Brown <broonie@kernel.org>, <devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <linux-omap@vger.kernel.org>,
<linux-arm-msm@vger.kernel.org>
Subject: Re: [PATCH] ARM: dts: Remove "spidev" nodes
Date: Sat, 18 Dec 2021 16:53:38 +0100 [thread overview]
Message-ID: <748816cb-e511-8649-29fa-a55ab581246b@microchip.com> (raw)
In-Reply-To: <20211217221232.3664417-1-robh@kernel.org>
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
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@microchip.com>
To: "Rob Herring" <robh@kernel.org>,
soc@kernel.org,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Ludovic Desroches" <ludovic.desroches@microchip.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Fabio Estevam" <festevam@gmail.com>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Benoît Cousson" <bcousson@baylibre.com>,
"Tony Lindgren" <tony@atomide.com>,
"Andy Gross" <agross@kernel.org>,
"Bjorn Andersson" <bjorn.andersson@linaro.org>,
"Viresh Kumar" <vireshk@kernel.org>,
"Shiraz Hashim" <shiraz.linux.kernel@gmail.com>
Cc: Mark Brown <broonie@kernel.org>, <devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <linux-omap@vger.kernel.org>,
<linux-arm-msm@vger.kernel.org>
Subject: Re: [PATCH] ARM: dts: Remove "spidev" nodes
Date: Sat, 18 Dec 2021 16:53:38 +0100 [thread overview]
Message-ID: <748816cb-e511-8649-29fa-a55ab581246b@microchip.com> (raw)
In-Reply-To: <20211217221232.3664417-1-robh@kernel.org>
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-12-18 15:53 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-17 22:12 [PATCH] ARM: dts: Remove "spidev" nodes Rob Herring
2021-12-17 22:12 ` Rob Herring
2021-12-18 15:53 ` Nicolas Ferre [this message]
2021-12-18 15:53 ` Nicolas Ferre
2021-12-20 4:20 ` Viresh Kumar
2021-12-20 4:20 ` Viresh Kumar
2021-12-20 12:30 ` Mark Brown
2021-12-20 12:30 ` Mark Brown
2021-12-20 19:22 ` Alexandre Belloni
2021-12-20 19:22 ` Alexandre Belloni
2021-12-20 20:27 ` Mark Brown
2021-12-20 20:27 ` Mark Brown
2021-12-20 15:40 ` patchwork-bot+linux-soc
2022-01-26 20:43 ` patchwork-bot+linux-arm-msm
-- strict thread matches above, loose matches on Subject: below --
2021-12-20 2:23 kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=748816cb-e511-8649-29fa-a55ab581246b@microchip.com \
--to=nicolas.ferre@microchip.com \
--cc=agross@kernel.org \
--cc=alexandre.belloni@bootlin.com \
--cc=bcousson@baylibre.com \
--cc=bjorn.andersson@linaro.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=ludovic.desroches@microchip.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=shiraz.linux.kernel@gmail.com \
--cc=soc@kernel.org \
--cc=tony@atomide.com \
--cc=vireshk@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.