[PATCH v2 2/3] sifive: fu540: Add U-Boot proper sector start

[Pragnesh Patel]

Hi Bin,

>-----Original Message-----
>From: Bin Meng <bmeng.cn at gmail.com>
>Sent: 29 April 2020 19:28
>To: Jagan Teki <jagan at amarulasolutions.com>
>Cc: Pragnesh Patel <pragnesh.patel at sifive.com>; Rick Chen
><rick at andestech.com>; U-Boot Mailing List <u-boot at lists.denx.de>; Sagar
>Kadam <sagar.kadam at sifive.com>; Paul Walmsley
><paul.walmsley at sifive.com>; Troy Benjegerdes
><troy.benjegerdes at sifive.com>; Palmer Dabbelt
><palmerdabbelt at google.com>; Anup Patel <anup.patel at wdc.com>; Atish
>Patra <atish.patra at wdc.com>; linux-amarula <linux-
>amarula at amarulasolutions.com>
>Subject: Re: [PATCH v2 2/3] sifive: fu540: Add U-Boot proper sector start
>
>[External Email] Do not click links or attachments unless you recognize the
>sender and know the content is safe
>
>Hi Jagan,
>
>On Mon, Apr 20, 2020 at 9:37 PM Jagan Teki <jagan at amarulasolutions.com>
>wrote:
>>
>> Add U-Boot proper sector start offset for SiFive FU540.
>> This value is based on the partition layout supported by SiFive FU540.
>>
>> u-boot.itb need to write on this specific offset so-that the SPL will
>> retrieve it from here and load.
>>
>> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
>> ---
>> Changes for v2:
>> - none
>>
>>  common/spl/Kconfig | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/spl/Kconfig b/common/spl/Kconfig index
>> 07eee56219..1ebb30344a 100644
>> --- a/common/spl/Kconfig
>> +++ b/common/spl/Kconfig
>> @@ -308,7 +308,7 @@ config
>SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
>>                      ARCH_MX6 || ARCH_MX7 || \
>>                      ARCH_ROCKCHIP || ARCH_MVEBU ||  ARCH_SOCFPGA || \
>>                      ARCH_AT91 || ARCH_ZYNQ || ARCH_KEYSTONE || OMAP34XX ||
>\
>> -                    OMAP44XX || OMAP54XX || AM33XX || AM43XX
>> +                    OMAP44XX || OMAP54XX || AM33XX || AM43XX ||
>> + TARGET_SIFIVE_FU540
>>         help
>>           Use sector number for specifying U-Boot location on MMC/SD in
>>           raw mode.
>> @@ -325,6 +325,7 @@ config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
>>         default 0x300 if ARCH_ZYNQ || ARCH_KEYSTONE || OMAP34XX ||
>OMAP44XX || \
>>                          OMAP54XX || AM33XX || AM43XX || ARCH_K3
>>         default 0x4000 if ARCH_ROCKCHIP
>> +       default 0x822 if TARGET_SIFIVE_FU540
>>         help
>>           Address on the MMC to load U-Boot from, when the MMC is being
>used
>>           in raw mode. Units: MMC sectors (1 sector = 512 bytes).
>> --
>
>It looks my comments in v1 patch is not addressed?

@Bin Meng I did not see any comment from your side, please help me if I was wrong.

>
>Regards,
>Bin