[External] Solve the problem of bad CRC when using u-boot-aspeed-sdk

[George Liu]

On Wed, Dec 22, 2021 at 6:24 PM Lei Yu <yulei.sh at bytedance.com> wrote:
>
> On Wed, Dec 22, 2021 at 4:05 PM George Liu <liuxiwei1013 at gmail.com> wrote:
> >
> > Hi, everyone:
> >
> > The 64M flash used by Inspur's fp5280g2 machine has recently found
> > that it cannot be saved when setting the u-boot environment variable,
> > and then a warning message appears when rebooting:
> >
> > ```
> > Loading Environment from SPI Flash... SF: Detected mx66l51235l with
> > page size 256 Bytes, erase size 64 KiB, total 64 MiB
> > *** Warning-bad CRC, using default environment
> > ```
> >
> > Since fp5280g2 includes `openbmc-flash-layout-64.dtsi`, the initial problem is:
> > 1. The value of CONFIG_ENV_SIZE and CONFIG_ENV_OFFSET[1] of u-boot is
> > inconsistent with the size of u-boot env of Kernel's
> > openbmc-flash-layout-64.dtsi[2].
> > 2. The `Env. size` of the fw_env_ast2600_nor.config[3] file also needs
> > to be updated
> >
> > After modifying the above problem and test by:
> > 1. Use `setenv` to add a new environment variable in u-boot and reboot
> > BMC, and then use `fw_printenv` to see this value synchronously.
> > 2. Use `fw_setenv` to add a new environment variable in BMC and reboot
> > BMC into u-boot, and then use `printenv` to see this value
> > synchronously.
> >
> > We pushed a new patch to meta-inspur and it worked fine.
> > https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/49916
>
> Good to know the issue is fixed.
> Could you kindly re-work the patches to make it applicable for all
> systems using 64MiB static layout?

Sure, We will re-test and re-push it, Thanks for your reply :)

>
>
> --
> BRs,
> Lei YU