* [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board
@ 2024-06-10 0:34 Tony Dinh
2024-06-10 1:07 ` Dragan Simic
0 siblings, 1 reply; 4+ messages in thread
From: Tony Dinh @ 2024-06-10 0:34 UTC (permalink / raw)
To: U-Boot Mailing List, Stefan Roese
Cc: Tom Rini, Tony Dinh,
This contributor prefers not to receive mails
- Change the spi-max-frequency to 50000000 (50 Mhz). According to the
data sheet[1], the MX25L3205D max frequency is 86 Mhz. Using 50 Mhz in
the DTS to ensure u-boot is consistent with what Linux kernel expected.
- Update GPIO fan to conform to the latest DT binding.
[1] MX25L3205D-MX25L1605D-MX25L6405D-Macronix-MX25L3205DM2I-12G-datasheet.pdf
Signed-off-by: Tony Dinh <mibodhi@gmail.com>
---
arch/arm/dts/armada-385-thecus-n2350.dts | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/dts/armada-385-thecus-n2350.dts b/arch/arm/dts/armada-385-thecus-n2350.dts
index 253cf01130..fdaa444e51 100644
--- a/arch/arm/dts/armada-385-thecus-n2350.dts
+++ b/arch/arm/dts/armada-385-thecus-n2350.dts
@@ -2,7 +2,7 @@
/*
* Device Tree file for Thecus N2350 board
*
- * Copyright (C) 2018-2023 Tony Dinh <mibodhi@gmail.com>
+ * Copyright (C) 2018-2024 Tony Dinh <mibodhi@gmail.com>
* Copyright (C) 2018 Manuel Jung <manuel.jung@hotmail.com>
*/
@@ -143,9 +143,9 @@
fan {
compatible = "gpio-fan";
gpios = <&gpio1 16 GPIO_ACTIVE_HIGH>;
- gpio-fan,speed-map = < 0 0
- 600 1
- 3000 2 >;
+ gpio-fan,speed-map = < 0 0>,
+ <600 1>,
+ <3000 2 >;
pinctrl-0 = <&pmx_fan>;
pinctrl-names = "default";
};
@@ -415,7 +415,7 @@
compatible = "jedec,spi-nor";
reg = <0>;
- spi-max-frequency = <108000000>;
+ spi-max-frequency = <50000000>;
spi-cpha;
partition@0 {
--
2.39.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board
2024-06-10 0:34 [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board Tony Dinh
@ 2024-06-10 1:07 ` Dragan Simic
2024-06-11 21:42 ` Tony Dinh
0 siblings, 1 reply; 4+ messages in thread
From: Dragan Simic @ 2024-06-10 1:07 UTC (permalink / raw)
To: Tony Dinh; +Cc: U-Boot Mailing List, Stefan Roese, Tom Rini
Hello Tony,
Please see a few comments below.
On 2024-06-10 02:34, Tony Dinh wrote:
> - Change the spi-max-frequency to 50000000 (50 Mhz). According to the
> data sheet[1], the MX25L3205D max frequency is 86 Mhz. Using 50 Mhz in
> the DTS to ensure u-boot is consistent with what Linux kernel expected.
> - Update GPIO fan to conform to the latest DT binding.
There's no need for the bullet points, plain prose will do fine instead.
> [1]
> MX25L3205D-MX25L1605D-MX25L6405D-Macronix-MX25L3205DM2I-12G-datasheet.pdf
Unless you can provide a real, working URL for the datasheet, this
reference
should actually be deleted.
> Signed-off-by: Tony Dinh <mibodhi@gmail.com>
> ---
>
> arch/arm/dts/armada-385-thecus-n2350.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/dts/armada-385-thecus-n2350.dts
> b/arch/arm/dts/armada-385-thecus-n2350.dts
> index 253cf01130..fdaa444e51 100644
> --- a/arch/arm/dts/armada-385-thecus-n2350.dts
> +++ b/arch/arm/dts/armada-385-thecus-n2350.dts
> @@ -2,7 +2,7 @@
> /*
> * Device Tree file for Thecus N2350 board
> *
> - * Copyright (C) 2018-2023 Tony Dinh <mibodhi@gmail.com>
> + * Copyright (C) 2018-2024 Tony Dinh <mibodhi@gmail.com>
> * Copyright (C) 2018 Manuel Jung <manuel.jung@hotmail.com>
> */
>
> @@ -143,9 +143,9 @@
> fan {
> compatible = "gpio-fan";
> gpios = <&gpio1 16 GPIO_ACTIVE_HIGH>;
> - gpio-fan,speed-map = < 0 0
> - 600 1
> - 3000 2 >;
> + gpio-fan,speed-map = < 0 0>,
> + <600 1>,
> + <3000 2 >;
> pinctrl-0 = <&pmx_fan>;
> pinctrl-names = "default";
> };
> @@ -415,7 +415,7 @@
> compatible = "jedec,spi-nor";
> reg = <0>;
>
> - spi-max-frequency = <108000000>;
> + spi-max-frequency = <50000000>;
> spi-cpha;
>
> partition@0 {
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board
2024-06-10 1:07 ` Dragan Simic
@ 2024-06-11 21:42 ` Tony Dinh
2024-07-08 14:15 ` Stefan Roese
0 siblings, 1 reply; 4+ messages in thread
From: Tony Dinh @ 2024-06-11 21:42 UTC (permalink / raw)
To: Dragan Simic; +Cc: U-Boot Mailing List, Stefan Roese, Tom Rini
Hi Dragan,
On Sun, Jun 9, 2024 at 6:07 PM Dragan Simic <dsimic@manjaro.org> wrote:
>
> Hello Tony,
>
> Please see a few comments below.
>
> On 2024-06-10 02:34, Tony Dinh wrote:
> > - Change the spi-max-frequency to 50000000 (50 Mhz). According to the
> > data sheet[1], the MX25L3205D max frequency is 86 Mhz. Using 50 Mhz in
> > the DTS to ensure u-boot is consistent with what Linux kernel expected.
> > - Update GPIO fan to conform to the latest DT binding.
>
> There's no need for the bullet points, plain prose will do fine instead.
I like bullet points :) All my patch descriptions have bullet points
when there are 2 or more different items.
>
> > [1]
> > MX25L3205D-MX25L1605D-MX25L6405D-Macronix-MX25L3205DM2I-12G-datasheet.pdf
>
> Unless you can provide a real, working URL for the datasheet, this
> reference
> should actually be deleted.
It was my oversight. That should have been this URL:
https://www.macronix.com/Lists/Datasheet/Attachments/8575/MX25L3205D,%203V,%2032Mb,%20v1.5.pdf
I'll wait for Stefan to review the patch and let him decide whether I
should send in a V2 patch or he will modify the commit description.
Thanks for the comments!
All the best,
Tony
>
> > Signed-off-by: Tony Dinh <mibodhi@gmail.com>
> > ---
> >
> > arch/arm/dts/armada-385-thecus-n2350.dts | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/dts/armada-385-thecus-n2350.dts
> > b/arch/arm/dts/armada-385-thecus-n2350.dts
> > index 253cf01130..fdaa444e51 100644
> > --- a/arch/arm/dts/armada-385-thecus-n2350.dts
> > +++ b/arch/arm/dts/armada-385-thecus-n2350.dts
> > @@ -2,7 +2,7 @@
> > /*
> > * Device Tree file for Thecus N2350 board
> > *
> > - * Copyright (C) 2018-2023 Tony Dinh <mibodhi@gmail.com>
> > + * Copyright (C) 2018-2024 Tony Dinh <mibodhi@gmail.com>
> > * Copyright (C) 2018 Manuel Jung <manuel.jung@hotmail.com>
> > */
> >
> > @@ -143,9 +143,9 @@
> > fan {
> > compatible = "gpio-fan";
> > gpios = <&gpio1 16 GPIO_ACTIVE_HIGH>;
> > - gpio-fan,speed-map = < 0 0
> > - 600 1
> > - 3000 2 >;
> > + gpio-fan,speed-map = < 0 0>,
> > + <600 1>,
> > + <3000 2 >;
> > pinctrl-0 = <&pmx_fan>;
> > pinctrl-names = "default";
> > };
> > @@ -415,7 +415,7 @@
> > compatible = "jedec,spi-nor";
> > reg = <0>;
> >
> > - spi-max-frequency = <108000000>;
> > + spi-max-frequency = <50000000>;
> > spi-cpha;
> >
> > partition@0 {
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board
2024-06-11 21:42 ` Tony Dinh
@ 2024-07-08 14:15 ` Stefan Roese
0 siblings, 0 replies; 4+ messages in thread
From: Stefan Roese @ 2024-07-08 14:15 UTC (permalink / raw)
To: Tony Dinh, Dragan Simic; +Cc: U-Boot Mailing List, Tom Rini
On 6/11/24 23:42, Tony Dinh wrote:
> Hi Dragan,
>
> On Sun, Jun 9, 2024 at 6:07 PM Dragan Simic <dsimic@manjaro.org> wrote:
>>
>> Hello Tony,
>>
>> Please see a few comments below.
>>
>> On 2024-06-10 02:34, Tony Dinh wrote:
>>> - Change the spi-max-frequency to 50000000 (50 Mhz). According to the
>>> data sheet[1], the MX25L3205D max frequency is 86 Mhz. Using 50 Mhz in
>>> the DTS to ensure u-boot is consistent with what Linux kernel expected.
>>> - Update GPIO fan to conform to the latest DT binding.
>>
>> There's no need for the bullet points, plain prose will do fine instead.
>
> I like bullet points :) All my patch descriptions have bullet points
> when there are 2 or more different items.
>
>>
>>> [1]
>>> MX25L3205D-MX25L1605D-MX25L6405D-Macronix-MX25L3205DM2I-12G-datasheet.pdf
>>
>> Unless you can provide a real, working URL for the datasheet, this
>> reference
>> should actually be deleted.
>
> It was my oversight. That should have been this URL:
>
> https://www.macronix.com/Lists/Datasheet/Attachments/8575/MX25L3205D,%203V,%2032Mb,%20v1.5.pdf
>
> I'll wait for Stefan to review the patch and let him decide whether I
> should send in a V2 patch or he will modify the commit description.
I've changed the commit message accordingly.
Applied to u-boot-marvell/master
Thanks,
Stefan
> Thanks for the comments!
>
> All the best,
> Tony
>
>>
>>> Signed-off-by: Tony Dinh <mibodhi@gmail.com>
>>> ---
>>>
>>> arch/arm/dts/armada-385-thecus-n2350.dts | 10 +++++-----
>>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/arch/arm/dts/armada-385-thecus-n2350.dts
>>> b/arch/arm/dts/armada-385-thecus-n2350.dts
>>> index 253cf01130..fdaa444e51 100644
>>> --- a/arch/arm/dts/armada-385-thecus-n2350.dts
>>> +++ b/arch/arm/dts/armada-385-thecus-n2350.dts
>>> @@ -2,7 +2,7 @@
>>> /*
>>> * Device Tree file for Thecus N2350 board
>>> *
>>> - * Copyright (C) 2018-2023 Tony Dinh <mibodhi@gmail.com>
>>> + * Copyright (C) 2018-2024 Tony Dinh <mibodhi@gmail.com>
>>> * Copyright (C) 2018 Manuel Jung <manuel.jung@hotmail.com>
>>> */
>>>
>>> @@ -143,9 +143,9 @@
>>> fan {
>>> compatible = "gpio-fan";
>>> gpios = <&gpio1 16 GPIO_ACTIVE_HIGH>;
>>> - gpio-fan,speed-map = < 0 0
>>> - 600 1
>>> - 3000 2 >;
>>> + gpio-fan,speed-map = < 0 0>,
>>> + <600 1>,
>>> + <3000 2 >;
>>> pinctrl-0 = <&pmx_fan>;
>>> pinctrl-names = "default";
>>> };
>>> @@ -415,7 +415,7 @@
>>> compatible = "jedec,spi-nor";
>>> reg = <0>;
>>>
>>> - spi-max-frequency = <108000000>;
>>> + spi-max-frequency = <50000000>;
>>> spi-cpha;
>>>
>>> partition@0 {
Viele Grüße,
Stefan Roese
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-07-08 14:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-10 0:34 [PATCH] arm: dts: mvebu: Update DTS for Thecus N2350 board Tony Dinh
2024-06-10 1:07 ` Dragan Simic
2024-06-11 21:42 ` Tony Dinh
2024-07-08 14:15 ` Stefan Roese
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.