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

[Pragnesh Patel]

Hi Bin,

>-----Original Message-----
>From: Bin Meng <bmeng.cn at gmail.com>
>Sent: 20 April 2020 20:13
>To: Pragnesh Patel <pragnesh.patel at sifive.com>
>Cc: Jagan Teki <jagan at amarulasolutions.com>; U-Boot-Denx <u-
>boot at lists.denx.de>; Atish Patra <atish.patra at wdc.com>;
>palmerdabbelt at google.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 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.

Will submit v7 soon.

>
>Regards,
>Bin