From: <Tudor.Ambarus@microchip.com> To: <krzysztof.kozlowski@linaro.org>, <lee.jones@linaro.org>, <robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>, <Nicolas.Ferre@microchip.com>, <alexandre.belloni@bootlin.com>, <Claudiu.Beznea@microchip.com>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH] ARM: dts: at91: use generic node name for dataflash Date: Tue, 12 Apr 2022 11:21:09 +0000 [thread overview] Message-ID: <ec974b3d-f220-a54b-6d00-99ea07afc20a@microchip.com> (raw) In-Reply-To: <20220412105013.249793-1-krzysztof.kozlowski@linaro.org> On 4/12/22 13:50, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > The node names should be generic, so use "flash" for dataflash nodes and > for cfi-flash. > > Suggested-by: Tudor Ambarus <tudor.ambarus@microchip.com> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com> thanks, ta > --- > Documentation/devicetree/bindings/mfd/atmel-flexcom.txt | 2 +- > arch/arm/boot/dts/at91rm9200ek.dts | 4 ++-- > arch/arm/boot/dts/at91sam9260ek.dts | 2 +- > arch/arm/boot/dts/at91sam9261ek.dts | 2 +- > arch/arm/boot/dts/at91sam9263ek.dts | 2 +- > arch/arm/boot/dts/at91sam9g20ek_common.dtsi | 2 +- > arch/arm/boot/dts/at91sam9m10g45ek.dts | 2 +- > arch/arm/boot/dts/at91sam9rlek.dts | 2 +- > arch/arm/boot/dts/usb_a9263.dts | 2 +- > 9 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt b/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > index 692300117c64..9d837535637b 100644 > --- a/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > +++ b/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > @@ -54,7 +54,7 @@ flexcom@f8034000 { > clock-names = "spi_clk"; > atmel,fifo-size = <32>; > > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at25f512b"; > reg = <0>; > spi-max-frequency = <20000000>; > diff --git a/arch/arm/boot/dts/at91rm9200ek.dts b/arch/arm/boot/dts/at91rm9200ek.dts > index e1ef4e44e663..4624a6f076f8 100644 > --- a/arch/arm/boot/dts/at91rm9200ek.dts > +++ b/arch/arm/boot/dts/at91rm9200ek.dts > @@ -73,7 +73,7 @@ &pinctrl_uart1_dcd > spi0: spi@fffe0000 { > status = "okay"; > cs-gpios = <&pioA 3 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <15000000>; > reg = <0>; > @@ -94,7 +94,7 @@ usb0: ohci@300000 { > status = "okay"; > }; > > - nor_flash@10000000 { > + flash@10000000 { > compatible = "cfi-flash"; > reg = <0x10000000 0x800000>; > linux,mtd-name = "physmap-flash.0"; > diff --git a/arch/arm/boot/dts/at91sam9260ek.dts b/arch/arm/boot/dts/at91sam9260ek.dts > index ce96345d28a3..6381088ba24f 100644 > --- a/arch/arm/boot/dts/at91sam9260ek.dts > +++ b/arch/arm/boot/dts/at91sam9260ek.dts > @@ -92,7 +92,7 @@ macb0: ethernet@fffc4000 { > > spi0: spi@fffc8000 { > cs-gpios = <0>, <&pioC 11 0>, <0>, <0>; > - mtd_dataflash@1 { > + flash@1 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <1>; > diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts > index beed819609e8..8f11c0b7d76d 100644 > --- a/arch/arm/boot/dts/at91sam9261ek.dts > +++ b/arch/arm/boot/dts/at91sam9261ek.dts > @@ -145,7 +145,7 @@ spi0: spi@fffc8000 { > cs-gpios = <&pioA 3 0>, <0>, <&pioA 28 0>, <0>; > status = "okay"; > > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > reg = <0>; > spi-max-frequency = <15000000>; > diff --git a/arch/arm/boot/dts/at91sam9263ek.dts b/arch/arm/boot/dts/at91sam9263ek.dts > index 71f60576761a..42e734020235 100644 > --- a/arch/arm/boot/dts/at91sam9263ek.dts > +++ b/arch/arm/boot/dts/at91sam9263ek.dts > @@ -95,7 +95,7 @@ pinctrl_board_mmc0: mmc0-board { > spi0: spi@fffa4000 { > status = "okay"; > cs-gpios = <&pioA 5 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > index 87bb39060e8b..74b90dc58cbf 100644 > --- a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > +++ b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > @@ -110,7 +110,7 @@ ssc0: ssc@fffbc000 { > > spi0: spi@fffc8000 { > cs-gpios = <0>, <&pioC 11 0>, <0>, <0>; > - mtd_dataflash@1 { > + flash@1 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <1>; > diff --git a/arch/arm/boot/dts/at91sam9m10g45ek.dts b/arch/arm/boot/dts/at91sam9m10g45ek.dts > index b6256a20fbc7..e5db198a87a8 100644 > --- a/arch/arm/boot/dts/at91sam9m10g45ek.dts > +++ b/arch/arm/boot/dts/at91sam9m10g45ek.dts > @@ -167,7 +167,7 @@ pinctrl_pwm_leds: pwm-led { > spi0: spi@fffa4000{ > status = "okay"; > cs-gpios = <&pioB 3 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <13000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/at91sam9rlek.dts b/arch/arm/boot/dts/at91sam9rlek.dts > index 62981b39c815..d74b8d9d84aa 100644 > --- a/arch/arm/boot/dts/at91sam9rlek.dts > +++ b/arch/arm/boot/dts/at91sam9rlek.dts > @@ -180,7 +180,7 @@ usb0: gadget@fffd4000 { > spi0: spi@fffcc000 { > status = "okay"; > cs-gpios = <&pioA 28 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <15000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/usb_a9263.dts b/arch/arm/boot/dts/usb_a9263.dts > index 8a0cfbfd0c45..b6cb9cdf8197 100644 > --- a/arch/arm/boot/dts/usb_a9263.dts > +++ b/arch/arm/boot/dts/usb_a9263.dts > @@ -60,7 +60,7 @@ usb1: gadget@fff78000 { > spi0: spi@fffa4000 { > cs-gpios = <&pioB 15 GPIO_ACTIVE_HIGH>; > status = "okay"; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > reg = <0>; > spi-max-frequency = <15000000>; > -- > 2.32.0 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: <Tudor.Ambarus@microchip.com> To: <krzysztof.kozlowski@linaro.org>, <lee.jones@linaro.org>, <robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>, <Nicolas.Ferre@microchip.com>, <alexandre.belloni@bootlin.com>, <Claudiu.Beznea@microchip.com>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH] ARM: dts: at91: use generic node name for dataflash Date: Tue, 12 Apr 2022 11:21:09 +0000 [thread overview] Message-ID: <ec974b3d-f220-a54b-6d00-99ea07afc20a@microchip.com> (raw) In-Reply-To: <20220412105013.249793-1-krzysztof.kozlowski@linaro.org> On 4/12/22 13:50, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > The node names should be generic, so use "flash" for dataflash nodes and > for cfi-flash. > > Suggested-by: Tudor Ambarus <tudor.ambarus@microchip.com> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com> thanks, ta > --- > Documentation/devicetree/bindings/mfd/atmel-flexcom.txt | 2 +- > arch/arm/boot/dts/at91rm9200ek.dts | 4 ++-- > arch/arm/boot/dts/at91sam9260ek.dts | 2 +- > arch/arm/boot/dts/at91sam9261ek.dts | 2 +- > arch/arm/boot/dts/at91sam9263ek.dts | 2 +- > arch/arm/boot/dts/at91sam9g20ek_common.dtsi | 2 +- > arch/arm/boot/dts/at91sam9m10g45ek.dts | 2 +- > arch/arm/boot/dts/at91sam9rlek.dts | 2 +- > arch/arm/boot/dts/usb_a9263.dts | 2 +- > 9 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt b/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > index 692300117c64..9d837535637b 100644 > --- a/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > +++ b/Documentation/devicetree/bindings/mfd/atmel-flexcom.txt > @@ -54,7 +54,7 @@ flexcom@f8034000 { > clock-names = "spi_clk"; > atmel,fifo-size = <32>; > > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at25f512b"; > reg = <0>; > spi-max-frequency = <20000000>; > diff --git a/arch/arm/boot/dts/at91rm9200ek.dts b/arch/arm/boot/dts/at91rm9200ek.dts > index e1ef4e44e663..4624a6f076f8 100644 > --- a/arch/arm/boot/dts/at91rm9200ek.dts > +++ b/arch/arm/boot/dts/at91rm9200ek.dts > @@ -73,7 +73,7 @@ &pinctrl_uart1_dcd > spi0: spi@fffe0000 { > status = "okay"; > cs-gpios = <&pioA 3 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <15000000>; > reg = <0>; > @@ -94,7 +94,7 @@ usb0: ohci@300000 { > status = "okay"; > }; > > - nor_flash@10000000 { > + flash@10000000 { > compatible = "cfi-flash"; > reg = <0x10000000 0x800000>; > linux,mtd-name = "physmap-flash.0"; > diff --git a/arch/arm/boot/dts/at91sam9260ek.dts b/arch/arm/boot/dts/at91sam9260ek.dts > index ce96345d28a3..6381088ba24f 100644 > --- a/arch/arm/boot/dts/at91sam9260ek.dts > +++ b/arch/arm/boot/dts/at91sam9260ek.dts > @@ -92,7 +92,7 @@ macb0: ethernet@fffc4000 { > > spi0: spi@fffc8000 { > cs-gpios = <0>, <&pioC 11 0>, <0>, <0>; > - mtd_dataflash@1 { > + flash@1 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <1>; > diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts > index beed819609e8..8f11c0b7d76d 100644 > --- a/arch/arm/boot/dts/at91sam9261ek.dts > +++ b/arch/arm/boot/dts/at91sam9261ek.dts > @@ -145,7 +145,7 @@ spi0: spi@fffc8000 { > cs-gpios = <&pioA 3 0>, <0>, <&pioA 28 0>, <0>; > status = "okay"; > > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > reg = <0>; > spi-max-frequency = <15000000>; > diff --git a/arch/arm/boot/dts/at91sam9263ek.dts b/arch/arm/boot/dts/at91sam9263ek.dts > index 71f60576761a..42e734020235 100644 > --- a/arch/arm/boot/dts/at91sam9263ek.dts > +++ b/arch/arm/boot/dts/at91sam9263ek.dts > @@ -95,7 +95,7 @@ pinctrl_board_mmc0: mmc0-board { > spi0: spi@fffa4000 { > status = "okay"; > cs-gpios = <&pioA 5 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > index 87bb39060e8b..74b90dc58cbf 100644 > --- a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > +++ b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi > @@ -110,7 +110,7 @@ ssc0: ssc@fffbc000 { > > spi0: spi@fffc8000 { > cs-gpios = <0>, <&pioC 11 0>, <0>, <0>; > - mtd_dataflash@1 { > + flash@1 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <50000000>; > reg = <1>; > diff --git a/arch/arm/boot/dts/at91sam9m10g45ek.dts b/arch/arm/boot/dts/at91sam9m10g45ek.dts > index b6256a20fbc7..e5db198a87a8 100644 > --- a/arch/arm/boot/dts/at91sam9m10g45ek.dts > +++ b/arch/arm/boot/dts/at91sam9m10g45ek.dts > @@ -167,7 +167,7 @@ pinctrl_pwm_leds: pwm-led { > spi0: spi@fffa4000{ > status = "okay"; > cs-gpios = <&pioB 3 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <13000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/at91sam9rlek.dts b/arch/arm/boot/dts/at91sam9rlek.dts > index 62981b39c815..d74b8d9d84aa 100644 > --- a/arch/arm/boot/dts/at91sam9rlek.dts > +++ b/arch/arm/boot/dts/at91sam9rlek.dts > @@ -180,7 +180,7 @@ usb0: gadget@fffd4000 { > spi0: spi@fffcc000 { > status = "okay"; > cs-gpios = <&pioA 28 0>, <0>, <0>, <0>; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > spi-max-frequency = <15000000>; > reg = <0>; > diff --git a/arch/arm/boot/dts/usb_a9263.dts b/arch/arm/boot/dts/usb_a9263.dts > index 8a0cfbfd0c45..b6cb9cdf8197 100644 > --- a/arch/arm/boot/dts/usb_a9263.dts > +++ b/arch/arm/boot/dts/usb_a9263.dts > @@ -60,7 +60,7 @@ usb1: gadget@fff78000 { > spi0: spi@fffa4000 { > cs-gpios = <&pioB 15 GPIO_ACTIVE_HIGH>; > status = "okay"; > - mtd_dataflash@0 { > + flash@0 { > compatible = "atmel,at45", "atmel,dataflash"; > reg = <0>; > spi-max-frequency = <15000000>; > -- > 2.32.0 >
next prev parent reply other threads:[~2022-04-12 11:34 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-12 10:50 [PATCH] ARM: dts: at91: use generic node name for dataflash Krzysztof Kozlowski 2022-04-12 10:50 ` Krzysztof Kozlowski 2022-04-12 11:21 ` Tudor.Ambarus [this message] 2022-04-12 11:21 ` Tudor.Ambarus 2022-04-13 9:24 ` Nicolas Ferre 2022-04-13 9:24 ` Nicolas Ferre
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=ec974b3d-f220-a54b-6d00-99ea07afc20a@microchip.com \ --to=tudor.ambarus@microchip.com \ --cc=Claudiu.Beznea@microchip.com \ --cc=Nicolas.Ferre@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=devicetree@vger.kernel.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=krzysztof.kozlowski@linaro.org \ --cc=lee.jones@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=robh+dt@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: linkBe 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.