]> git.dujemihanovic.xyz Git - u-boot.git/commit
Merge patch series "env: mmc: fix use of two separate partitions with proper type...
authorTom Rini <trini@konsulko.com>
Tue, 1 Oct 2024 14:46:01 +0000 (08:46 -0600)
committerTom Rini <trini@konsulko.com>
Tue, 1 Oct 2024 14:46:01 +0000 (08:46 -0600)
commit667a671ccab9e6cfe04c654e2c935ea0a070f58b
tree7f54188cc9959fd29d01ef14a7b862960aa882e2
parent6f52feee2867f138af3d81a90a02a2b222e3366d
parentc1131aca9adfb2861dd34ba0e45f593874adcd81
Merge patch series "env: mmc: fix use of two separate partitions with proper type GUID"

Rasmus Villemoes <rasmus.villemoes@prevas.dk> says:

I always define a disk layout with two separate partitions for the two
copies of the U-Boot environment and, being the one who introduced the
type GUID for such partitions, of course also set those partitions'
type GUID appropriately.

This has worked just fine, but, it turns out, only because I've never
had CONFIG_PARTITION_TYPE_GUID enabled on any of my boards; I've
always just set the offsets of the two partitions via the config
variables CONFIG_ENV_OFFSET(,_REDUND).

I didn't even know that env/mmc.c had learnt to look for the env
partition based on the type GUID, or that that would overrule the
ENV_OFFSET config variables, until I experienced weird random
corruption while doing bringup for an stm32 board, where
PARTITION_TYPE_GUID is automatically set because it is select'ed by
CMD_STM32PROG.

These patches try to fix the code to fit my scheme, while not changing
anything for existing setups that use the two-copies-one-partition
scheme, other than complaining loudly if the system is misconfigured
and avoiding such random corruption of neighbouring partitions.