Hi Rick, On Sat, Jun 12, 2021 at 9:30 PM Rick Chen <rickche...@gmail.com> wrote: > > HI Bin > > > Hi Rick, > > > > On Wed, Jun 9, 2021 at 3:06 PM Rick Chen <rickche...@gmail.com> wrote: > > > > > > Hi Bin, > > > > > > > From: Bin Meng <bmeng...@gmail.com> > > > > Sent: Friday, June 04, 2021 1:51 PM > > > > To: Rick Jian-Zhi Chen(陳建志) <r...@andestech.com>; Leo Yu-Chi Liang(梁育齊) > > > > <ycli...@andestech.com>; U-Boot Mailing List <u-boot@lists.denx.de> > > > > Subject: [PATCH 5/5] riscv: ae350: dts: Add missing "u-boot,dm-spl" for > > > > SPL config > > > > > > > > At present the AE350 SPL defconfig is using OF_PRIOR_STAGE. The > > > > intention was to use gdb to load device tree before running U-Boot > > > > SPL/proper from RAM. When we switch to OF_SEPARATE we will have to use > > > > our own DT but without "u-boot,dm-spl" in several essential nodes, SPL > > > > does not boot. > > > > > > Can you describe how do you verify and provide the steps about that > > > SPL boot fail that I can duplicate it. :) > > > > $ make ae350_rv64_spl_defconfig; make -j > > $ make menuconfig (change OF_PRIOR_STAGE to OF_EMBED or OF_SEPARATE) > > > > Load u-boot.bin to RAM > > It can boot with OF_EMBED. > But it compile fail with choosing OF_EMBED at the first time, fail > messages as below: > > binman: [Errno 2] No such file or directory: 'u-boot.dtb' > Makefile:1084: recipe for target 'all' failed > make: *** [all] Error 1
Yes, this is a known issue of the binman conversion for SPL. OF_EMBED is a debugging purpose hence I am inclined to leave it as is. Regards, Bin