[PATCH v6 00/17] RISC-V SiFive FU540 support SPL

[Bin Meng]

Hi Pragnesh,

On Wed, Apr 8, 2020 at 12:42 PM Pragnesh Patel
<pragnesh.patel at sifive.com> wrote:
>
> Hi Jagan,
>
> >-----Original Message-----
> >From: Jagan Teki <jagan at amarulasolutions.com>
> >Sent: 07 April 2020 15:02
> >To: Pragnesh Patel <pragnesh.patel at sifive.com>
> >Cc: U-Boot-Denx <u-boot at lists.denx.de>; Atish Patra
> ><atish.patra at wdc.com>; palmerdabbelt at google.com; Bin Meng
> ><bmeng.cn at gmail.com>; Paul Walmsley <paul.walmsley at sifive.com>; Troy
> >Benjegerdes <troy.benjegerdes at sifive.com>; Anup Patel
> ><anup.patel at wdc.com>; Sagar Kadam <sagar.kadam at sifive.com>; Rick Chen
> ><rick at 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.patel at 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.

I've reviewed the v6 patches. Since Jagan also sent several series
related to SPI and booting from SPI that is dependent on this series,
would you please post a v7 soon? We need get this SPL support merged
as soon as possible. Thank you.

Regards,
Bin