Hi Jagan, >-----Original Message----- >From: Jagan Teki <ja...@amarulasolutions.com> >Sent: 07 April 2020 15:02 >To: Pragnesh Patel <pragnesh.pa...@sifive.com> >Cc: U-Boot-Denx <u-boot@lists.denx.de>; Atish Patra ><atish.pa...@wdc.com>; palmerdabb...@google.com; Bin Meng ><bmeng...@gmail.com>; Paul Walmsley <paul.walms...@sifive.com>; Troy >Benjegerdes <troy.benjeger...@sifive.com>; Anup Patel ><anup.pa...@wdc.com>; Sagar Kadam <sagar.ka...@sifive.com>; Rick Chen ><r...@andestech.com> >Subject: Re: [PATCH v6 00/17] RISC-V SiFive FU540 support SPL > >[External Email] Do not click links or attachments unless you recognize the >sender and know the content is safe > >Hi Pragnesh, > >On Sun, Mar 29, 2020 at 10:36 PM Pragnesh Patel ><pragnesh.pa...@sifive.com> wrote: >> >> This series add support for SPL to FU540.U-Boot SPL can boot from >> L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and >> U-Boot proper from MMC devices. >> >> How to test this patch: >> 1) Go to OpenSBI-dir : make PLATFORM=sifive/fu540 O=build_dir >> I=install_dir FW_DYNAMIC=y install >> 2) cp install_dir/platform/sifive/fu540/firmware/fw_dynamic.bin >> <u-boot-dir>/ >> 3) Change to u-boot-dir >> 4) make sifive_fu540_defconfig >> 5) make all >> 6) ZSBL loads the U-boot SPL(u-boot-spl.bin) from a partition with >> GUID type 5B193300-FC78-40CD-8002-E86C45580B47 >> >> sudo dd if=spl/u-boot-spl.bin of=/dev/sdc4 bs=1M >> >> 7) U-boot SPL expects a u-boot FIT image(u-boot.itb) from 1st >partition(/dev/sdc1) >> of SD card irrespective of GUID >> >> sudo dd if=u-boot.itb of=/dev/sdc1 bs=1M > >Seems like SPL (sdc4) at 4th partition and U-Boot proper at first partition >which seems an unconventional way to create partition layouts based on the >booting stage hierarchy. > >I did send the changes on top of our changes [1] which indeed follow the >conventional partition layout. So feel free to incorporate your series while >sending the next version. > >So from next version, you can drop >CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION and write >images as, > >$ sudo dd if=u-boot.itb of=/dev/sda seek=4096 $ sudo dd if=spl/u-boot- >spl.bin of=/dev/sda seek=2048 > >[1] https://patchwork.ozlabs.org/project/uboot/list/?series=168967
Thanks Jagan, will update in v7. > >Jagan.