2022-04-07 20:19:04

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

The node names should be generic and SPI NOR dtschema expects "flash".

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/mips/boot/dts/brcm/bcm97358svmb.dts | 2 +-
arch/mips/boot/dts/brcm/bcm97360svmb.dts | 2 +-
arch/mips/boot/dts/brcm/bcm97425svmb.dts | 2 +-
arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts | 2 +-
arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts | 2 +-
arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts | 2 +-
6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/mips/boot/dts/brcm/bcm97358svmb.dts b/arch/mips/boot/dts/brcm/bcm97358svmb.dts
index 522f2c40d6e6..c17fc14d4899 100644
--- a/arch/mips/boot/dts/brcm/bcm97358svmb.dts
+++ b/arch/mips/boot/dts/brcm/bcm97358svmb.dts
@@ -78,7 +78,7 @@ &nand {
&qspi {
status = "okay";

- m25p80@0 {
+ flash@0 {
compatible = "m25p80";
reg = <0>;
spi-max-frequency = <40000000>;
diff --git a/arch/mips/boot/dts/brcm/bcm97360svmb.dts b/arch/mips/boot/dts/brcm/bcm97360svmb.dts
index 01f215b08dba..c9b76f41e7a6 100644
--- a/arch/mips/boot/dts/brcm/bcm97360svmb.dts
+++ b/arch/mips/boot/dts/brcm/bcm97360svmb.dts
@@ -81,7 +81,7 @@ &sdhci0 {
&qspi {
status = "okay";

- m25p80@0 {
+ flash@0 {
compatible = "m25p80";
reg = <0>;
spi-max-frequency = <40000000>;
diff --git a/arch/mips/boot/dts/brcm/bcm97425svmb.dts b/arch/mips/boot/dts/brcm/bcm97425svmb.dts
index f38934934349..289a57b912ef 100644
--- a/arch/mips/boot/dts/brcm/bcm97425svmb.dts
+++ b/arch/mips/boot/dts/brcm/bcm97425svmb.dts
@@ -116,7 +116,7 @@ &sdhci1 {
&qspi {
status = "okay";

- m25p80@0 {
+ flash@0 {
compatible = "m25p80";
reg = <0>;
spi-max-frequency = <40000000>;
diff --git a/arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts b/arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts
index 6069b33cf09f..826e91b840a3 100644
--- a/arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts
+++ b/arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts
@@ -129,7 +129,7 @@ &spi {
pinctrl-names = "default";
pinctrl-0 = <&pinmux_spi_spi>, <&pinmux_spi_cs1_cs>;

- m25p80@0 {
+ flash@0 {
compatible = "jedec,spi-nor";
reg = <0>;
spi-max-frequency = <40000000>;
diff --git a/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts b/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts
index 5892bcf71595..db961d61cfde 100644
--- a/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts
+++ b/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts
@@ -60,7 +60,7 @@ &sdhci {
&spi0 {
status = "okay";

- m25p80@0 {
+ flash@0 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "jedec,spi-nor";
diff --git a/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts b/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts
index a7fce8de6147..224383a30ce3 100644
--- a/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts
+++ b/arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts
@@ -44,7 +44,7 @@ &sdhci {
&spi0 {
status = "okay";

- m25p80@0 {
+ flash@0 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "jedec,spi-nor";
--
2.32.0


2022-04-07 20:53:52

by Florian Fainelli

[permalink] [raw]
Subject: Re: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

On 4/7/22 07:33, Krzysztof Kozlowski wrote:
> The node names should be generic and SPI NOR dtschema expects "flash".
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/mips/boot/dts/brcm/bcm97358svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97360svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97425svmb.dts | 2 +-

For the above:

Acked-by: Florian Fainelli <[email protected]>
--
Florian

2022-04-08 07:13:20

by Arınç ÜNAL

[permalink] [raw]
Subject: Re: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

> On 7 Apr 2022, at 17:33, Krzysztof Kozlowski <[email protected]> wrote:
>
> The node names should be generic and SPI NOR dtschema expects "flash".
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

For ralink mt7621:

Acked-by: Arınç ÜNAL <[email protected]>

Thanks.
Arınç

2022-04-08 07:51:30

by Sergio Paracuellos

[permalink] [raw]
Subject: Re: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

On Thu, Apr 7, 2022 at 4:33 PM Krzysztof Kozlowski
<[email protected]> wrote:
>
> The node names should be generic and SPI NOR dtschema expects "flash".
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts | 2 +-
> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)

Acked-by: Sergio Paracuellos <[email protected]>

Thanks,
Sergio Paracuellos

2022-04-26 15:46:41

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

On 07/04/2022 16:33, Krzysztof Kozlowski wrote:
> The node names should be generic and SPI NOR dtschema expects "flash".
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/mips/boot/dts/brcm/bcm97358svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97360svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97425svmb.dts | 2 +-
> arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts | 2 +-
> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts | 2 +-
> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)

Hi all,

Anyone from MIPS folks willing to take it? I got three acks, so maybe
that means I should handle it?


Best regards,
Krzysztof

2022-04-26 21:31:22

by Thomas Bogendoerfer

[permalink] [raw]
Subject: Re: [PATCH] MIPS: dts: align SPI NOR node name with dtschema

On Thu, Apr 07, 2022 at 04:33:28PM +0200, Krzysztof Kozlowski wrote:
> The node names should be generic and SPI NOR dtschema expects "flash".
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/mips/boot/dts/brcm/bcm97358svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97360svmb.dts | 2 +-
> arch/mips/boot/dts/brcm/bcm97425svmb.dts | 2 +-
> arch/mips/boot/dts/ralink/gardena_smart_gateway_mt7688.dts | 2 +-
> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc1.dts | 2 +-
> arch/mips/boot/dts/ralink/mt7621-gnubee-gb-pc2.dts | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)

applied to mips-next.

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]