[PATCH v11 15/18] sifive: fu540: Add sample SD gpt partition layout

[Pragnesh Patel]

Hi Bin/Jagan,

>-----Original Message-----
>From: Bin Meng <bmeng.cn at gmail.com>
>Sent: 20 May 2020 19:30
>To: Pragnesh Patel <pragnesh.patel at sifive.com>
>Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Atish Patra
><atish.patra at wdc.com>; Palmer Dabbelt <palmerdabbelt at google.com>; Paul
>Walmsley <paul.walmsley at sifive.com>; Jagan Teki
><jagan at amarulasolutions.com>; Anup Patel <anup.patel at wdc.com>; Sagar
>Kadam <sagar.kadam at sifive.com>; Rick Chen <rick at andestech.com>; Palmer
>Dabbelt <palmer at dabbelt.com>
>Subject: Re: [PATCH v11 15/18] sifive: fu540: Add sample SD gpt partition
>layout
>
>[External Email] Do not click links or attachments unless you recognize the
>sender and know the content is safe
>
>Hi Pragnesh,
>
>On Tue, May 19, 2020 at 3:05 PM Pragnesh Patel
><pragnesh.patel at sifive.com> wrote:
>>
>> From: Jagan Teki <jagan at amarulasolutions.com>
>>
>> This is a sample GPT partition layout for SD card, right now three
>> important partitions are added to make the system bootable.
>>
>> partition layout:
>>
>> Part    Start LBA       End LBA         Name
>>         Attributes
>>         Type GUID
>>         Partition GUID
>>   1     0x00000022      0x00000821      "loader1"
>>         attrs:  0x0000000000000000
>>         type:   5b193300-fc78-40cd-8002-e86c45580b47
>>         guid:   cbcbef44-e627-42bc-b134-93b6f3784b8c
>>   2     0x00000822      0x00002821      "loader2"
>>         attrs:  0x0000000000000000
>>         type:   2e54b353-1271-4842-806f-e436d6af6985
>>         guid:   f54eba28-d8de-4852-978d-1a673777e2ae
>>   3     0x00002822      0x00020821      "rootfs"
>>         attrs:  0x0000000000000004
>>         type:   0fc63daf-8483-4772-8e79-3d69d8477de4
>>         type:   linux
>>         guid:   9561df46-8d55-4799-a83b-cfee9ef6ff93
>>
>> Note:
>> - loader1 would be fsbl or spl
>> - loader2 would be U-Boot or U-Boot proper
>>
>> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
>> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
>> Reviewed-by: Pragnesh Patel <pragnesh.patel at sifive.com>
>> ---
>>  board/sifive/fu540/Kconfig     |  2 ++
>>  include/configs/sifive-fu540.h | 13 +++++++++++++
>>  2 files changed, 15 insertions(+)
>>
>
>I see Jagan posted patches for SPI flash booting, and the layout is different
>from MMC.
>
>Can we use the same layout here, ie: by reserving spaces for the U-Boot
>environment on MMC?
>
>See
>http://patchwork.ozlabs.org/project/uboot/patch/20200519192340.16624-7-
>jagan at amarulasolutions.com/
>
>    0 - 34 : reserved for GPT header
>   35 - 39 : unused
>   40 - 2087 : loader1 (SPL, FSBL)
> 2088 - 10279 : loader2 (U-Boot proper, U-Boot)
>10280 - 10535 : environment
>10536 - 65494 : rootfs
>65528 - 65536 : distro script

I am okay with this. @Jagan Teki do you have any comment on this ?

>
>
>REgards,
>Bin