> 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.
Reviewed-by: Rick Chen <r...@andestech.com> > > Regards, > Bin
