[Buildroot] [PATCH 1/1] boot/uboot: add config option for uboot environment padding byte

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon Dec 11 07:15:15 UTC 2017


Hello,

On Mon, 11 Dec 2017 07:48:06 +0100, Johannes Schmitz wrote:

> this is a good question. So far I just know that reading the environment
> only worked for me with a 0x00 padding. Just to be sure it was not a
> combined problem of size and padding I can try again with the mkenvimage
> default of 0xFF and the correct size.

Yes, it would be good to investigate some more. The fact that the CRC
was not matching the original environment CRC is fine, because the
padding was different. So the padding being different cannot on its own
explain why it was not working for you.

> In any case the padding option should be added to expose the full
> mkenvimage capabilities, however if you're right I might have to edit the
> help text again and remove the statement about the CRC calculation.

Yes, that's my thought: the help text you're proposing is potentially
wrong.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


More information about the buildroot mailing list