FW: [PATCH v4 00/13] riscv: Switch to use binman to generate u-boot.itb

[Rick Chen]

Hi Bin

> Hi Rick,
>
> On Wed, May 12, 2021 at 11:25 AM Rick Chen <rickchen36 at gmail.com> wrote:
> >
> > HI Bin,
> >
> > >
> > > > Hi Rick,
> > > >
> > > > On Tue, May 11, 2021 at 8:49 AM Rick Chen <rickchen36 at gmail.com> wrote:
> > > > >
> > > > > Hi Bin,
> > > > >
> > > > > > Hi Rick,
> > > > > >
> > > > > > On Mon, May 10, 2021 at 3:22 PM Rick Chen <rickchen36 at gmail.com> wrote:
> > > > > > >
> > > > > > > Hi Bin
> > > > > > >
> > > > > > > > Hi Bin,
> > > > > > > >
> > > > > > > > > From: Bin Meng <bmeng.cn at gmail.com>
> > > > > > > > > Sent: Monday, May 10, 2021 2:58 PM
> > > > > > > > > To: Simon Glass <sjg at chromium.org>; Rick Jian-Zhi Chen(陳建志) <rick at andestech.com>; u-boot at lists.denx.de
> > > > > > > > > Subject: [PATCH v4 00/13] riscv: Switch to use binman to generate u-boot.itb
> > > > > > > > >
> > > > > > > > > This series updates binman to handle creation of u-boot.itb image for RISC-V boards.
> > > > > > > > >
> > > > > > > > > Azure results: PASS
> > > > > > > > > https://dev.azure.com/bmeng/GitHub/_build/results?buildId=363&view=results
> > > > > > > > >
> > > > > > > > > The following tests were performed:
> > > > > > > > > * booting qemu-riscv{32|64}_spl_defconfig on QEMU virt
> > > > > > > > > * booting sifive_unleashed_defconfig on QEMU sifive_u
> > > > > > > > >
> > > > > > > > > AE350 SPL defconfigs are not tested. @Rick, could you please test and report?
> > > > > > > >
> > > > > > > > OK. I will verify it on AE350.
> > > > > > >
> > > > > > > It fail as below messages:
> > > > > > >
> > > > > > > U-Boot SPL 2021.07-rc1-00218-g468b3b3 (May 10 2021 - 15:13:03 +0800)
> > > > > > > Trying to boot from RAM
> > > > > > > alloc space exhausted
> > > > > >
> > > > > > Looks it is running out of memory.
> > > > > >
> > > > > > > Could not get FIT buffer of 499076 bytes
> > > > > > >         check CONFIG_SYS_SPL_MALLOC_SIZE
> > > > > >
> > > > > > Could you please try increasing CONFIG_SYS_SPL_MALLOC_SIZE?
> > > > >
> > > > > I increased CONFIG_SYS_SPL_MALLOC_SIZE, but it is useless.
> > > > > But it boots successfully after increase CONFIG_SPL_SYS_MALLOC_F_LEN larger.
> > > >
> > > > Thanks for testing. I am not sure why AE350 fails to boot because this
> > > > series only changes the way to assemble the bits.
> > > >
> > > > Could you please confirm if without this patch series, AE350 can boot?
> > >
> > > OK.
> >
> > ============================================
> > I have verified AE350 without your patch, it works as below:
> > ============================================
> > U-Boot SPL 2021.07-rc1-00194-g07b5310 (May 12 2021 - 10:59:48 +0800)
> > Trying to boot from RAM
> >
> > U-Boot 2021.07-rc1-00194-g07b5310 (May 12 2021 - 10:59:48 +0800)
> >
> > DRAM:  1 GiB
> > Flash: 64 MiB
> > MMC:   mmc at f0e00000: 0
> > Loading Environment from SPIFlash... SF: Detected mx25u1635e with page
> > size 256 Bytes, erase size 4 KiB, total 2 MiB
> > OK
> > In:    serial at f0300000
> > Out:   serial at f0300000
> > Err:   serial at f0300000
> > Net:   no alias for ethernet0
> >
> > Warning: mac at e0100000 (eth0) using random MAC address - 26:00:fa:12:76:ad
> > eth0: mac at e0100000
> > Hit any key to stop autoboot:  0
> > RISC-V #
> >
> > =========================
> > With your patch, it fail as below:
> > =========================
> >
> > U-Boot SPL 2021.07-rc1-00207-g28a2d21 (May 12 2021 - 11:09:11 +0800)
> > Trying to boot from RAM
> > alloc space exhausted
> > Could not get FIT buffer of 499076 bytes
> >         check CONFIG_SYS_SPL_MALLOC_SIZE
> > No device tree specified in SPL image
> >
> > =======================================================
> > After increase CONFIG_SPL_SYS_MALLOC_F_LEN, it works as below
> > =======================================================
> > U-Boot SPL 2021.07-rc1-00207-g28a2d21 (May 12 2021 - 11:11:00 +0800)
> > Trying to boot from RAM
> >
> >
> > U-Boot 2021.07-rc1-00207-g28a2d21 (May 12 2021 - 11:11:00 +0800)
> >
> > DRAM:  1 GiB
> > Flash: 64 MiB
> > MMC:   mmc at f0e00000: 0
> > Loading Environment from SPIFlash... SF: Detected mx25u1635e with page
> > size 256 Bytes, erase size 4 KiB, total 2 MiB
> > OK
> > In:    serial at f0300000
> > Out:   serial at f0300000
> > Err:   serial at f0300000
> > Net:   no alias for ethernet0
> >
> > Warning: mac at e0100000 (eth0) using random MAC address - e6:58:7e:7c:5f:49
> > eth0: mac at e0100000
> > Hit any key to stop autoboot:  0
> > RISC-V #
> >
> >
> > I found that it need larger heap size when spl try to get fit image
> > with using binman to generate u-boot.itb instead of
> > USE_SPL_FIT_GENERATOR.
> > But it is OK. I will send a patch for AE350 later.
>
> A patch for AE350 to increase CONFIG_SPL_SYS_MALLOC_F_LEN needs to be
> applied before this series.
>
> Would you please send the AE350 patch, and get this series applied?

OK, I will send the AE350 patch later.

Thanks,
Rick

>
> Regards,
> Bin