On Fri, May 10, 2024 at 01:38:34PM +0200, [email protected] wrote:

> From: Jasper Orschulko <[email protected]>
> 
> The A/B update strategy of the env's has a gap in the first 2 calls of 
> saveenv.
> The env's are stored twice on the first memory area if:
> gd->env_valid == ENV_INVALID.
> 
> u-boot=> saveenv
> Saving Environment to MMC... Writing to MMC(1)... OK
> u-boot=> saveenv
> Saving Environment to MMC... Writing to MMC(1)... OK  <-- !!!
> u-boot=> saveenv
> Saving Environment to MMC... Writing to redundant MMC(1)... OK
> u-boot=> saveenv
> Saving Environment to MMC... Writing to MMC(1)... OK
> 
> Signed-off-by: Michael Glembotzki <[email protected]>
> Signed-off-by: Jasper Orschulko <[email protected]>

Sorry for the extreme delay here. I was hoping for more feedback from
others and then it slipped my mind. And then we had a long thread
addressing this same problem for env/sf.c. So I've applied this now
(with a slight reword to note the env/sf.c fix) and posted a patch to
address env/fat.c and env/ubi.c.

Applied to u-boot/master, thanks!

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to