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

[Pragnesh Patel]

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.

>
>Jagan.