[U-Boot] [PATCH v2 5/6] apalis/colibri_t30: move environment location
Stephen Warren
swarren at wwwdotorg.org
Wed Sep 28 20:11:13 CEST 2016
On 09/28/2016 03:35 AM, Marcel Ziswiler wrote:
> Now with the config block handling in place move the U-Boot environment
> location before the config block at the end of 1st "boot sector" as
> deployed during production using our downstream BSP.
> diff --git a/include/configs/apalis_t30.h b/include/configs/apalis_t30.h
> -/* Environment in eMMC, at the end of 2nd "boot sector" */
> +/* Environment in eMMC, before config block at the end of 1st "boot sector" */
> #define CONFIG_ENV_IS_IN_MMC
> -#define CONFIG_ENV_OFFSET (-CONFIG_ENV_SIZE)
> +#define CONFIG_ENV_OFFSET (-CONFIG_ENV_SIZE + \
> + CONFIG_TDX_CFG_BLOCK_OFFSET)
I'm not convinced that ENV_OFFSET actually points where the description
says it does. (-CONFIG_ENV_SIZE + CONFIG_TDX_CFG_BLOCK_OFFSET) is (-8192
+ -512). The original comment states that an ENV_OFFSET of -8192 locates
the environment at the end of a sector, so shifting it 512 bytes earlier
in the flash can't possible now align it with a different sector? Should
ENV_OFFSET be shifted down by a whole sector size by this patch, i.e.
should +CONFIG_TXT_CFG_BLOCK_OFFSET be rounded up to a sector size to
match the comment?
More information about the U-Boot
mailing list