[U-Boot] fw_env bug for multi-sector environments on NOR flash
Fei, Yiyang
YiYang.Fei at barco.com
Thu Aug 25 20:28:02 CEST 2011
Hi,
I am storing u-boot environment variables on a NOR flash using two
sectors. I encountered an error when writing changes using fw_setenv.
/etc # cat fw_env.config
# Configuration file for fw_(printenv/saveenv) utility.
# Up to two entries are valid, in this case the redundand
# environment sector is assumed present.
# MTD device name Device offset Env. size Flash sector
size Number of sectors
/dev/mtd12 0x0000 0x20000 0x10000
2
/etc # fw_setenv test test
End of range reached, aborting
Error: can't write fw_env to flash
The reason for the error is that although both sectors are written in
one pass, the loop for the write is executed twice because the
"processed" variable is incremented by the incorrect amount. The
following change is needed to fix this issue for NOR multi-sector
environments.
diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
index ed6b53f..e6b2cae 100644
--- a/tools/env/fw_env.c
+++ b/tools/env/fw_env.c
@@ -880,9 +880,8 @@ static int flash_write_buf (int dev, int fd, void
*buf, size_t count,
ioctl (fd, MEMLOCK, &erase);
- processed += blocklen;
- block_seek = 0;
- blockstart += blocklen;
+ processed += erasesize;
+ blockstart += erasesize;
}
if (write_total > count)
Are there any objections to committing this change?
Yiyang Fei
More information about the U-Boot
mailing list