<div dir="ltr"><div dir="ltr">Hello, Tomer.<br></div><div dir="ltr"><div><br></div><div>Seems like Nuvoton's u-boot expects the uboot-env at different address comparing to openbmc-flash-layout.dtsi:</div><div><br></div><div><a href="https://github.com/Nuvoton-Israel/u-boot/blob/npcm7xx-v2019.01/include/configs/poleg.h#L30">https://github.com/Nuvoton-Israel/u-boot/blob/npcm7xx-v2019.01/include/configs/poleg.h#L30</a><br></div><div><br></div><div>Vs.</div><div><br></div><div><a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/openbmc-flash-layout.dtsi#n13">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/openbmc-flash-layout.dtsi#n13</a><br></div><div><br></div><div>The change is more about making partitions naming the same as expected by OpenBMC.</div><div><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 21 Feb 2021 at 17:40, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hi Benjamin and Anton,<div><br></div><div>Sorry for the late reply,</div><div><br></div><div><p class="MsoNormal" style="margin:0cm 0cm 0.0001pt"><font face="arial, sans-serif">The EVB FIU0-CS0
partitioning is used for testing the EVB and this is why it is
different than the OpenBMC flash layout.</font></p>
<p class="MsoNormal" style="margin:0cm 0cm 0.0001pt"><font face="arial, sans-serif"> </font></p>
<p class="MsoNormal" style="margin:0cm 0cm 0.0001pt"><font face="arial, sans-serif">Are you using the NPCM7XX EVB for
OpenBMC? if yes we can consider to modify the flash partition to OpenBMC use.</font></p></div><div><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 18 Feb 2021 at 19:11, Benjamin Fair <<a href="mailto:benjaminfair@google.com" target="_blank">benjaminfair@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 18 Feb 2021 at 04:42, <<a href="mailto:gmouse@google.com" target="_blank">gmouse@google.com</a>> wrote:<br>
><br>
> From: "Anton D. Kachalov" <<a href="mailto:gmouse@google.com" target="_blank">gmouse@google.com</a>><br>
><br>
> This change satisfy OpenBMC requirements for flash layout.<br>
><br>
> Signed-off-by: Anton D. Kachalov <<a href="mailto:gmouse@google.com" target="_blank">gmouse@google.com</a>><br>
> ---<br>
> arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 28 +++++++----------------<br>
> 1 file changed, 8 insertions(+), 20 deletions(-)<br>
><br>
> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts<br>
> index bd1eb6ee380f..741c1fee8552 100644<br>
> --- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts<br>
> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts<br>
> @@ -182,8 +182,8 @@ bbuboot2@80000 {<br>
> reg = <0x0080000 0x80000>;<br>
> read-only;<br>
> };<br>
> - envparam@100000 {<br>
> - label = "env-param";<br>
> + ubootenv@100000 {<br>
> + label = "u-boot-env";<br>
> reg = <0x0100000 0x40000>;<br>
> read-only;<br>
> };<br>
> @@ -195,25 +195,13 @@ kernel@200000 {<br>
> label = "kernel";<br>
> reg = <0x0200000 0x400000>;<br>
> };<br>
> - rootfs@600000 {<br>
> - label = "rootfs";<br>
> - reg = <0x0600000 0x700000>;<br>
> + rofs@780000 {<br>
> + label = "rofs";<br>
> + reg = <0x0780000 0x1680000>;<br>
> };<br>
> - spare1@D00000 {<br>
> - label = "spare1";<br>
> - reg = <0x0D00000 0x200000>;<br>
> - };<br>
> - spare2@0F00000 {<br>
> - label = "spare2";<br>
> - reg = <0x0F00000 0x200000>;<br>
> - };<br>
> - spare3@1100000 {<br>
> - label = "spare3";<br>
> - reg = <0x1100000 0x200000>;<br>
> - };<br>
> - spare4@1300000 {<br>
> - label = "spare4";<br>
> - reg = <0x1300000 0x0>;<br>
> + rwfs@1e00000 {<br>
> + label = "rwfs";<br>
> + reg = <0x1e00000 0x200000>;<br>
> };<br>
<br>
I recommend just including the openbmc-flash-layout.dtsi file here<br>
instead since that contains the common flash layout for most OpenBMC<br>
systems.<br>
<br></blockquote><div>Good <font face="arial, sans-serif">solution</font>, </div><div>Do you mean nuvoton-openbmc-flash-layout?</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> };<br>
> };<br>
> --<br>
> 2.30.0.478.g8a0d178c01-goog<br>
><br></blockquote><div><br></div><div>Thanks,</div><div><br></div><div>Tomer </div></div></div>
</blockquote></div></div>