[U-Boot] [PATCH] tools/mkimage: use lseek rather than fstat for file size for -l option

Peter Korsgaard jacmet at sunsite.dk
Wed Dec 3 13:07:31 CET 2008


>>>>> "Wolfgang" == Wolfgang Denk <wd at denx.de> writes:

Hi,

 >> Exactly, hence my other patch (tools/mkimage: ignore trailing
 >> garbage) to only error out if the size is smaller than the size
 >> defined in the header.

 Wolfgang> This doesn't make sense to me. If we have a way to check
 Wolfgang> the image, we want to check everything, including correct
 Wolfgang> image length.

I provided a reason for the change in the patch:

"E.G. if the uImage was stored in flash and hence padded to the flash
sector size"

I can add a warning if there's trailing garbage if you prefer, but I
think the use case is valid.

 >> Ok, I kept it like that to keep the patch as minimal as possible -
 >> I'll change it and resend.

 Wolfgang> I think it makes little sense as my feeling is the changes make
 Wolfgang> mkimage checking not better, just worse.

Ok, fixed in the v2 patch I recently sent.

-- 
Bye, Peter Korsgaard


More information about the U-Boot mailing list