On 11/10/2023 20.37, Tom Rini wrote: > On Thu, Sep 28, 2023 at 10:02:57AM +0200, Rasmus Villemoes wrote: > >> The man page correctly said that -B was ignored without -E, while the >> `mkimage -h` output suggested otherwise. Now that -B can actually be >> used by itself, update the man page. >> >> While at it, also amend the `mkimage -h` line to mention the >> connection with -E. >> >> The FDT header is a fixed 40 bytes, so its size cannot (and is not) >> modified, while its alignment is a property of the address in RAM one >> loads the FIT to, so not something mkimage can affect in any way. (In >> the file itself, the header is of course at offset 0, which has all >> possible alignments already.) >> >> Reported-by: Sean Anderson <sean...@gmail.com> >> Signed-off-by: Rasmus Villemoes <rasmus.villem...@prevas.dk> >> Reviewed-by: Simon Glass <s...@chromium.org> > > Applied to u-boot/master, thanks! >
Thanks, but I'm afraid that was premature. The original series which this was a fixup/followup for hasn't been applied, and Sean had reservations. I'm leaving it to Simon or Tom or whoever has final say to decide if they should eventually go in, but it would probably be good to get a verdict soonish (it really shouldn't be too controversial), and if it's a no, this should just be reverted. Rasmus