[PATCH 0/2] spi: remove "spidev" nodes from DTs

[PATCH 0/2] spi: remove "spidev" nodes from DTs

[Wolfram Sang]

There were two DTs left specifying "spidev" directly. Remove them.

Wolfram Sang (2):
  ARM: dts: stm32: argon: remove spidev node
  powerpc/82xx: remove spidev node from mgcoge

 arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi | 6 ------
 arch/powerpc/boot/dts/mgcoge.dts                 | 7 -------
 2 files changed, 13 deletions(-)

-- 
2.35.1

[PATCH 1/2] ARM: dts: stm32: argon: remove spidev node

[Wolfram Sang]

Commit 956b200a846e ("spi: spidev: Warn loudly if instantiated from DT
as "spidev"") states that there should not be spidev nodes in DTs.
Remove this non-HW description. There won't be a regression because it
won't bind since 2015 anyhow.

Fixes: 16e3e44c5b87 ("ARM: dts: stm32: Add support for the emtrion emSBC-Argon")
Cc: Reinhold Mueller <reinhold.mueller@emtrion.com>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Please take it via your platform tree.

 arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi b/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
index ac53ee3c496b..30156b7546ed 100644
--- a/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
+++ b/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
@@ -435,12 +435,6 @@ &spi1 {
 	pinctrl-0 = <&spi1_pins_a>;
 	cs-gpios = <&gpioz 3 0>;
 	status = "disabled";
-
-	spidev@0  {
-		compatible = "spidev";
-		reg = <0>;
-		spi-max-frequency = <100000>;
-	};
 };
 
 &timers1 {
-- 
2.35.1

Re: [PATCH 1/2] ARM: dts: stm32: argon: remove spidev node

[Krzysztof Kozlowski]

On 24/08/2022 11:21, Wolfram Sang wrote:
> Commit 956b200a846e ("spi: spidev: Warn loudly if instantiated from DT
> as "spidev"") states that there should not be spidev nodes in DTs.
> Remove this non-HW description. There won't be a regression because it
> won't bind since 2015 anyhow.
> 
> Fixes: 16e3e44c5b87 ("ARM: dts: stm32: Add support for the emtrion emSBC-Argon")
> Cc: Reinhold Mueller <reinhold.mueller@emtrion.com>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

Re: [PATCH 1/2] ARM: dts: stm32: argon: remove spidev node

[Alexandre TORGUE]

On 8/24/22 10:21, Wolfram Sang wrote:
> Commit 956b200a846e ("spi: spidev: Warn loudly if instantiated from DT
> as "spidev"") states that there should not be spidev nodes in DTs.
> Remove this non-HW description. There won't be a regression because it
> won't bind since 2015 anyhow.
> 
> Fixes: 16e3e44c5b87 ("ARM: dts: stm32: Add support for the emtrion emSBC-Argon")
> Cc: Reinhold Mueller <reinhold.mueller@emtrion.com>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> 
> Please take it via your platform tree.
> 
>   arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi | 6 ------
>   1 file changed, 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi b/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
> index ac53ee3c496b..30156b7546ed 100644
> --- a/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi
> @@ -435,12 +435,6 @@ &spi1 {
>   	pinctrl-0 = <&spi1_pins_a>;
>   	cs-gpios = <&gpioz 3 0>;
>   	status = "disabled";
> -
> -	spidev@0  {
> -		compatible = "spidev";
> -		reg = <0>;
> -		spi-max-frequency = <100000>;
> -	};
>   };
>   
>   &timers1 {

Applied on stm32-next.

Thanks.
Alex

[PATCH 2/2] powerpc/82xx: remove spidev node from mgcoge

[Wolfram Sang]

Commit 956b200a846e ("spi: spidev: Warn loudly if instantiated from DT
as "spidev"") states that there should not be spidev nodes in DTs.
Remove this non-HW description. There won't be a regression because it
won't bind since 2015 anyhow.

Fixes: 5d1d67e361ea ("powerpc/82xx: add SPI support for mgcoge")
Cc: Heiko Schocher <hs@denx.de>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Please take it via your platform tree.

 arch/powerpc/boot/dts/mgcoge.dts | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/arch/powerpc/boot/dts/mgcoge.dts b/arch/powerpc/boot/dts/mgcoge.dts
index 7de068991bde..9cefed207234 100644
--- a/arch/powerpc/boot/dts/mgcoge.dts
+++ b/arch/powerpc/boot/dts/mgcoge.dts
@@ -225,13 +225,6 @@ spi@11aa0 {
 				interrupts = <2 8>;
 				interrupt-parent = <&PIC>;
 				cs-gpios = < &cpm2_pio_d 19 0>;
-				#address-cells = <1>;
-				#size-cells = <0>;
-				ds3106@1 {
-					compatible = "gen,spidev";
-					reg = <0>;
-					spi-max-frequency = <8000000>;
-				};
 			};
 
 		};
-- 
2.35.1

Re: [PATCH 2/2] powerpc/82xx: remove spidev node from mgcoge

[Krzysztof Kozlowski]

On 24/08/2022 11:21, Wolfram Sang wrote:
> Commit 956b200a846e ("spi: spidev: Warn loudly if instantiated from DT
> as "spidev"") states that there should not be spidev nodes in DTs.
> Remove this non-HW description. There won't be a regression because it
> won't bind since 2015 anyhow.
> 
> Fixes: 5d1d67e361ea ("powerpc/82xx: add SPI support for mgcoge")
> Cc: Heiko Schocher <hs@denx.de>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

Re: [PATCH 0/2] spi: remove "spidev" nodes from DTs

[Michael Ellerman]

On Wed, 24 Aug 2022 10:21:27 +0200, Wolfram Sang wrote:
> There were two DTs left specifying "spidev" directly. Remove them.
> 
> Wolfram Sang (2):
>   ARM: dts: stm32: argon: remove spidev node
>   powerpc/82xx: remove spidev node from mgcoge
> 
> arch/arm/boot/dts/stm32mp157c-emstamp-argon.dtsi | 6 ------
>  arch/powerpc/boot/dts/mgcoge.dts                 | 7 -------
>  2 files changed, 13 deletions(-)
> 
> [...]

Patch 2 applied to powerpc/next.

[2/2] powerpc/82xx: remove spidev node from mgcoge
      https://git.kernel.org/powerpc/c/fd20b60aea6a37788f2f761af405b41c6c34473b

cheers