Hi Bin/Jagan, >-----Original Message----- >From: Bin Meng <bmeng...@gmail.com> >Sent: 20 May 2020 19:30 >To: Pragnesh Patel <pragnesh.pa...@sifive.com> >Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Atish Patra ><atish.pa...@wdc.com>; Palmer Dabbelt <palmerdabb...@google.com>; Paul >Walmsley <paul.walms...@sifive.com>; Jagan Teki ><ja...@amarulasolutions.com>; Anup Patel <anup.pa...@wdc.com>; Sagar >Kadam <sagar.ka...@sifive.com>; Rick Chen <r...@andestech.com>; Palmer >Dabbelt <pal...@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.pa...@sifive.com> wrote: >> >> From: Jagan Teki <ja...@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 <ja...@amarulasolutions.com> >> Reviewed-by: Bin Meng <bmeng...@gmail.com> >> Reviewed-by: Pragnesh Patel <pragnesh.pa...@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- >ja...@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
