On Mon, Dec 02, 2019 at 08:40:28PM -0300, Fabio Estevam wrote: > U-Boot binary has grown in such a way that it goes beyond the reserved > area for the environment variables. > > Running "saveenv" followed by a "reset" causes U-Boot to hang because > of this overlap. > > Fix this problem by increasing the CONFIG_ENV_OFFSET size. > > Also, in order to prevent this same problem in the future, use > CONFIG_BOARD_SIZE_LIMIT, which will detect the overlap in build-time. > > CONFIG_BOARD_SIZE_LIMIT does not accept math expressions, so declare > CONFIG_ENV_OFFSET with its direct value instead. > > Signed-off-by: Fabio Estevam <feste...@gmail.com> > Acked-by: Pierre-Jean Texier <pjtex...@koncepto.io> > Tested-by: Pierre-Jean Texier <pjtex...@koncepto.io> > Acked-by: Joris Offouga <offougajo...@gmail.com> > Tested-by: Joris Offouga <offougajo...@gmail.com> > --- > Changes since v1: > - Rebased against master as CONFIG_ENV_OFFSET has moved to Kconfig > > configs/warp7_defconfig | 2 +- > include/configs/warp7.h | 13 +++++++++++++ > 2 files changed, 14 insertions(+), 1 deletion(-)
Is there a reason we can't use CONFIG_SYS_THUMB_BUILD on these platforms? It's not default y as I seem to recall some thumb problems with very early ARMv7 cores, but it should almost certainly be set here (and a large number of other i.MX platforms). I'd much rather see that than moving the environment offset. Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
