[PATCH] ARM: dts: nuvoton: Fix flash layout

IS20 Ofer Eilon ofer.eilon at nuvoton.com
Tue Feb 23 01:10:52 AEDT 2021


Hi Avi,

It seems an old version of uboot  u-boot_2019.01.7.5.bin  from igps below:

> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNuvoton-Israel%2Figps%2Ftree%2Fmaster%2FImageGeneration%2Fversions&data=04%7C01%7Cofer.eilon%40nuvoton.com%7Ce56881b8491d42e5ee4c08d8d71bacd4%7Ca3f24931d4034b4a94f17d83ac638e07%7C0%7C0%7C637495861162860437%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=0%2BNzEv%2FSX9QTg0XumchRrU61uGbZ3CZXrtspXu2560I%3D&reserved=0

Please use latest from uboot.bin github.

Regards,
Ofer


-----Original Message-----
From: Avi Fishman <avifishman70 at gmail.com>
Sent: Monday, February 22, 2021 12:21 PM
To: Anton Kachalov <gmouse at google.com>
Cc: Tomer Maimon <tmaimon77 at gmail.com>; Benjamin Fair <benjaminfair at google.com>; Tali Perry <tali.perry1 at gmail.com>; Patrick Venture <venture at google.com>; Nancy Yuen <yuenn at google.com>; Rob Herring <robh+dt at kernel.org>; OpenBMC Maillist <openbmc at lists.ozlabs.org>; devicetree <devicetree at vger.kernel.org>; Linux Kernel Mailing List <linux-kernel at vger.kernel.org>; IS20 Ofer Eilon <ofer.eilon at nuvoton.com>
Subject: Re: [PATCH] ARM: dts: nuvoton: Fix flash layout

Ofer,

Can you check why u-boot doesn't work with SD cards?

On Mon, Feb 22, 2021 at 11:27 AM Anton Kachalov <gmouse at google.com> wrote:
>
> Hi, Tom.
>
> Yes, I'm using it for testing on real hardware.
>
> BTW. Recent u-boot doesn't work with SD cards. The card doesn't
> detect. The last working version was this one:
>
> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> ub.com%2FNuvoton-Israel%2Fnuvoton-info%2Ftree%2Fmaster%2Fnpcm7xx-poleg
> %2Fevaluation-board%2Fsw_deliverables%2Fnpcm7xx_v2.3&data=04%7C01%
> 7Cofer.eilon%40nuvoton.com%7Ce56881b8491d42e5ee4c08d8d71bacd4%7Ca3f249
> 31d4034b4a94f17d83ac638e07%7C0%7C0%7C637495861162860437%7CUnknown%7CTW
> FpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6
> Mn0%3D%7C1000&sdata=f4t41g3CQaFTQNfwwNVBrIwQScndIGcfRTms0yrTn5o%3D
> &reserved=0
>
> However, u-boot from igps repo:
>
> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> ub.com%2FNuvoton-Israel%2Figps%2Ftree%2Fmaster%2FImageGeneration%2Fver
> sions&data=04%7C01%7Cofer.eilon%40nuvoton.com%7Ce56881b8491d42e5ee
> 4c08d8d71bacd4%7Ca3f24931d4034b4a94f17d83ac638e07%7C0%7C0%7C6374958611
> 62860437%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL
> CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=0%2BNzEv%2FSX9QTg0Xumch
> RrU61uGbZ3CZXrtspXu2560I%3D&reserved=0
>
> Has issues too. It doesn't allow me to read more than 4k bytes once at
> a time. Thus, to flash the stuff I have manually read chunks from the
> SD-card: fat load doesn't work at all and I write that data in raw
> partition.
>
> On Sun, 21 Feb 2021 at 17:40, Tomer Maimon <tmaimon77 at gmail.com> wrote:
> >
> > Hi Benjamin and Anton,
> >
> > Sorry for the late reply,
> >
> > The EVB FIU0-CS0 partitioning is used for testing the EVB and this is why it is different than the OpenBMC flash layout.
> >
> >
> >
> > Are you using the NPCM7XX EVB for OpenBMC? if yes we can consider to modify the flash partition to OpenBMC use.
> >
> >
> > On Thu, 18 Feb 2021 at 19:11, Benjamin Fair <benjaminfair at google.com> wrote:
> >>
> >> On Thu, 18 Feb 2021 at 04:42, <gmouse at google.com> wrote:
> >> >
> >> > From: "Anton D. Kachalov" <gmouse at google.com>
> >> >
> >> > This change satisfy OpenBMC requirements for flash layout.
> >> >
> >> > Signed-off-by: Anton D. Kachalov <gmouse at google.com>
> >> > ---
> >> >  arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 28
> >> > +++++++----------------
> >> >  1 file changed, 8 insertions(+), 20 deletions(-)
> >> >
> >> > diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> >> > b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> >> > index bd1eb6ee380f..741c1fee8552 100644
> >> > --- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> >> > +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> >> > @@ -182,8 +182,8 @@ bbuboot2 at 80000 {
> >> >                                 reg = <0x0080000 0x80000>;
> >> >                                 read-only;
> >> >                                 };
> >> > -                       envparam at 100000 {
> >> > -                               label = "env-param";
> >> > +                       ubootenv at 100000 {
> >> > +                               label = "u-boot-env";
> >> >                                 reg = <0x0100000 0x40000>;
> >> >                                 read-only;
> >> >                                 }; @@ -195,25 +195,13 @@
> >> > kernel at 200000 {
> >> >                                 label = "kernel";
> >> >                                 reg = <0x0200000 0x400000>;
> >> >                                 };
> >> > -                       rootfs at 600000 {
> >> > -                               label = "rootfs";
> >> > -                               reg = <0x0600000 0x700000>;
> >> > +                       rofs at 780000 {
> >> > +                               label = "rofs";
> >> > +                               reg = <0x0780000 0x1680000>;
> >> >                                 };
> >> > -                       spare1 at D00000 {
> >> > -                               label = "spare1";
> >> > -                               reg = <0x0D00000 0x200000>;
> >> > -                               };
> >> > -                       spare2 at 0F00000 {
> >> > -                               label = "spare2";
> >> > -                               reg = <0x0F00000 0x200000>;
> >> > -                               };
> >> > -                       spare3 at 1100000 {
> >> > -                               label = "spare3";
> >> > -                               reg = <0x1100000 0x200000>;
> >> > -                               };
> >> > -                       spare4 at 1300000 {
> >> > -                               label = "spare4";
> >> > -                               reg = <0x1300000 0x0>;
> >> > +                       rwfs at 1e00000 {
> >> > +                               label = "rwfs";
> >> > +                               reg = <0x1e00000 0x200000>;
> >> >                         };
> >>
> >> I recommend just including the openbmc-flash-layout.dtsi file here
> >> instead since that contains the common flash layout for most
> >> OpenBMC systems.
> >>
> > Good solution,
> > Do you mean nuvoton-openbmc-flash-layout?
> >>
> >> >                 };
> >> >         };
> >> > --
> >> > 2.30.0.478.g8a0d178c01-goog
> >> >
> >
> >
> > Thanks,
> >
> > Tomer



--
Regards,
Avi
________________________________
________________________________
 The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.


More information about the openbmc mailing list