Hi Pragnesh, On Wed, Apr 29, 2020 at 10:04 PM Pragnesh Patel <pragnesh.pa...@sifive.com> wrote: > > Hi Bin, > > >-----Original Message----- > >From: Bin Meng <bmeng...@gmail.com> > >Sent: 29 April 2020 19:28 > >To: Jagan Teki <ja...@amarulasolutions.com> > >Cc: Pragnesh Patel <pragnesh.pa...@sifive.com>; Rick Chen > ><r...@andestech.com>; U-Boot Mailing List <u-boot@lists.denx.de>; Sagar > >Kadam <sagar.ka...@sifive.com>; Paul Walmsley > ><paul.walms...@sifive.com>; Troy Benjegerdes > ><troy.benjeger...@sifive.com>; Palmer Dabbelt > ><palmerdabb...@google.com>; Anup Patel <anup.pa...@wdc.com>; Atish > >Patra <atish.pa...@wdc.com>; linux-amarula <linux- > >amar...@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 <ja...@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 <ja...@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.
Sorry, I was referring to another patch which is similar to this one: https://lists.denx.de/pipermail/u-boot/2020-April/407627.html Regards, Bin
