[U-Boot] [PATCH 1/2] doc: FIT image: clarify usage of "compression" property

[Simon Goldschmidt]

On 30.07.2018 14:44, Wolfgang Denk wrote:
> Dear Simon,
> 
> In message <642b8dae-f941-5255-42c7-3761e12d04cb at de.pepperl-fuchs.com> you wrote:
>>
>> As I'm not that long with U-Boot, can you point me to a rough date of a
>> release that I could check to see if it worked at that time?
> 
> I can only speculate...  The first bigger rework of the cose appears
> to be this patch series:
> 
> 30864 859e92b775   2013-05-14 15:37:25 -0400   image: Move timestamp #ifdefs to header file
> 30863 61a439a873   2013-05-14 15:37:25 -0400   image: Export fit_check_ramdisk()
> 30862 53fbb7e885   2013-05-14 15:37:25 -0400   image: Split FIT code into new image-fit.c
> 30861 604f23dde0   2013-05-14 15:37:25 -0400   image: Move HOSTCC image code to tools/
> 30860 94e5fa46a0   2013-05-14 15:37:25 -0400   image: Split hash node processing into its own function
> 30859 b7260910dc   2013-05-14 15:37:25 -0400   image: Convert fit_image_hash_set_value() to static, and rename
> 30858 b8da836650   2013-05-14 15:37:25 -0400   image: Rename fit_image_check_hashes() to fit_image_verify()
> 30857 ab9efc665a   2013-05-14 15:37:25 -0400   image: Move hash checking into its own function
> 30856 e754da2aee   2013-05-14 15:37:25 -0400   image: Move error! string to common place
> 30855 003efd7da4   2013-05-14 15:37:25 -0400   image: Export fit_conf_get_prop_node()
> 30854 bbb467dc3c   2013-05-14 15:37:25 -0400   image: Rename fit_add_hashes() to fit_add_verification_data()
> 30853 d8b75360ee   2013-05-14 15:37:25 -0400   image: Rename hash printing to fit_image_print_verification_dat      a()
> 30852 35e7b0f179   2013-05-14 15:37:25 -0400   sandbox: image: Add support for booting images in sandbox
> 30851 aa6d6db4d4   2013-05-14 15:37:25 -0400   mkimage: Put FIT loading in function and tidy error handling
> 30850 1fe7d93891   2013-05-14 15:37:25 -0400   image: Remove remaining #ifdefs in image-fit.c
> 30849 87ebee39e9   2013-05-14 15:37:25 -0400   image: Add CONFIG_FIT_SPL_PRINT to control FIT image printing i      n SPL
> 30848 44d3a3066b   2013-05-14 15:37:25 -0400   image: Split libfdt code into image-fdt.c
> 30847 13d06981a9   2013-05-14 15:37:25 -0400   image: Add device tree setup to image library
> 30846 c19d13b030   2013-05-14 15:37:25 -0400   arm: Refactor bootm to reduce #ifdefs
> 30845 6caa195614   2013-05-14 15:37:25 -0400   arm: Use image_setup_linux() instead of local code
> 30844 3e51266a4e   2013-05-14 15:37:25 -0400   powerpc: Use image_setup_linux() instead of local code
> 30843 24507cf50a   2013-05-14 15:37:26 -0400   m68k: Use image_setup_linux() instead of local code
> 30842 2a08b740e3   2013-05-14 15:37:26 -0400   sparc: Use image_setup_linux() instead of local code
> 
> But there have been earlier changes, and many later (heavy) reworks,
> too.

I have checked starting with v1.3.3 (i.e. v2008.05) and even then, the 
compression property has only been handled for kernels. It seems like 
only the kernel has been loaded from FIT back then...

Next I checked v2009.01, where FDT and ramdisk are loaded from FIT, too. 
There is even a check that FDT is uncompressed. For ramdisk, the 
compression property is not checked.

Reading that, I tend to think uncompression has always been like it is now.

Best Regards,
Simon

> 
> Best regards,
> 
> Wolfgang Denk
>