[U-Boot] [PATCH v2 2/3] arm: mvebu: clearfog: set uboot image SD card offset

[Baruch Siach]

Hi Stefan,

On Mon, Aug 12, 2019 at 03:14:16PM +0200, Stefan Roese wrote:
> On 06.08.19 08:20, Stefan Roese wrote:
> > On 29.07.19 09:12, Baruch Siach wrote:
> > > Armada 38x ROM skips the first SD card offset when loading SPL. This
> > > affects the location of the main U-Boot image. SPL MMC code now supports
> > > U-Boot image offset based on run-time detection of the boot partition.
> > > Use this feature to make the same generated image support both SD card
> > > and eMMC boot partition.
> > > 
> > > Signed-off-by: Baruch Siach <baruch at tkos.co.il>
> > > ---
> > >    configs/clearfog_defconfig | 2 +-
> > >    1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/configs/clearfog_defconfig b/configs/clearfog_defconfig
> > > index b7b886be4f7a..3609445c1e95 100644
> > > --- a/configs/clearfog_defconfig
> > > +++ b/configs/clearfog_defconfig
> > > @@ -22,7 +22,7 @@ CONFIG_SYS_CONSOLE_INFO_QUIET=y
> > >    # CONFIG_DISPLAY_BOARDINFO is not set
> > >    CONFIG_DISPLAY_BOARDINFO_LATE=y
> > >    CONFIG_SPL_TEXT_BASE=0x40000030
> > > -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x141
> > > +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_DATA_PART_OFFSET=0x1
> > 
> > While looking at this patch, shouldn't this new Kconfig symbol
> > include "SECTOR" for better transparency? And why "DATA"?
> > Perhaps use CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR_OFFSET?

"DATA" is for the main eMMC data hardware partition, as opposed to boot 
partitions. This offset only applied to data partition. The symbol name should 
reflect that, I think.

We could use CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_DATA_PART_SECTOR_OFFSET. Isn't 
that too long?

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -