Hi Fabio, Pierre-Jean

Le 21/10/2019 à 19:12, Pierre-Jean Texier a écrit :
Fabio,

Le 21/10/2019 à 18:50, Fabio Estevam a écrit :
Hi Pierre-Jean,

On Mon, Oct 21, 2019 at 1:43 PM Pierre-Jean Texier <pjtex...@koncepto.io> wrote:

On top-of master 32ded50 ("spl: mmc: make eMMC HW boot
partition configuration optional ")
I am not able to reproduce the described behavior.

I used u-boot-imx tree and this is what I get:

ls -al u-boot-dtb.imx
-rw-r--r-- 1 fabio fabio 547840 out 21 13:45 u-boot-dtb.imx

Same for me, but I can't reproduce this issue :/
Same for me


Original CONFIG_ENV_OFFSET is:
#define CONFIG_ENV_OFFSET (8 * SZ_64K)

CONFIG_ENV_OFFSET = 8 * 64 * 1024 = 524288

So the u-boot-dtb.imx is overlapping the env region.

Indeed.


Maybe your compiler is producing a smaller binary and you are not
getting the issue.

Anyway, it is safer to increase CONFIG_ENV_OFFSET and use
CONFIG_BOARD_SIZE_LIMIT to detect such problem in build-time.


You're right, I agree with that ;)
I agree with you ;)

So,

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>

Best regards,

Joris



Thanks !

Regards

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to