[PATCH v3 07/10] configs: fu540: Add config file for U-boot SPL

[Pragnesh Patel]

>-----Original Message-----
>From: Anup Patel <anup at brainfault.org>
>Sent: 27 January 2020 13:09
>To: Pragnesh Patel <pragnesh.patel at sifive.com>
>Cc: Jagan Teki <jagan at amarulasolutions.com>; Atish Patra
><atish.patra at wdc.com>; palmerdabbelt at google.com; U-Boot-Denx <u-
>boot at lists.denx.de>
>Subject: Re: [PATCH v3 07/10] configs: fu540: Add config file for U-boot SPL
>
>On Mon, Jan 27, 2020 at 12:20 PM Pragnesh Patel
><pragnesh.patel at sifive.com> wrote:
>>
>>
>> >-----Original Message-----
>> >From: Jagan Teki <jagan at amarulasolutions.com>
>> >Sent: 24 January 2020 12:21
>> >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 ( Sifive)
>> ><paul.walmsley at sifive.com>; Troy Benjegerdes ( Sifive)
>> ><troy.benjegerdes at sifive.com>; Anup Patel <anup.patel at wdc.com>; Sagar
>> >Kadam <sagar.kadam at sifive.com>
>> >Subject: Re: [PATCH v3 07/10] configs: fu540: Add config file for
>> >U-boot SPL
>> >
>> >On Fri, Jan 24, 2020 at 11:21 AM Pragnesh Patel
>> ><pragnesh.patel at sifive.com>
>> >wrote:
>> >>
>> >> With sifive_fu540_spl_defconfig:
>> >>
>> >> U-Boot SPL will be loaded by ZSBL from SD card (replace fsbl.bin
>> >> with
>> >> u-boot-spl.bin) and runs in L2 LIM in machine mode and then load
>> >> FIT image u-boot.itb from 1st partition of SD card (replace
>> >> fw_payload.bin with u-boot.itb) into RAM.
>> >>
>> >> U-boot SPL expects u-boot.itb FIT image in the 1st partition of SD
>> >> card irrespective of GUID
>> >>
>> >> Signed-off-by: Pragnesh Patel <pragnesh.patel at sifive.com>
>> >> ---
>> >
>> >I'm thinking we don't need any separate config for SPL. Say if we
>> >have single config that creates both SPL and U-Boot proper and give
>> >feasibility to the user to use
>> >1) SPL + U-Boot proper
>> >2) SiFive SPL + U-Boot proper
>>
>> Sifive SPL ?
>
>I think what Jagan meant was:
>
>1) U-Boot SPL + OpenSBI FW_DYNAMIC + U-Boot proper
>2) FSBL + OpenSBI FW_PAYLOAD + U-Boot proper
>
>So "SiFive SPL" is actually "FSBL"

Ok, Thanks for the clarification.

>
>Regards,
>Anup