[U-Boot] [PATCH 5/9] sunxi: allow FIT image for 32-bit builds as well

Alexander Graf agraf at suse.de
Sat Feb 23 15:12:22 UTC 2019



On 23.02.19 02:10, André Przywara wrote:
> On 22/02/2019 08:06, Alexander Graf wrote:
>>
>>
>> On 21.02.19 02:30, Andre Przywara wrote:
>>> Even though we introduced FIT image support for the SPL to cover the
>>> 64-bit SoCs, there is no technical limitation to those parts.
>>>
>>> Change the Makefile stanza to always create a FIT image if the
>>> particular config either provided a script or an .its file.
>>>
>>> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
>>> ---
>>>  Makefile | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/Makefile b/Makefile
>>> index d6e7c69f40..3a560e29bf 100644
>>> --- a/Makefile
>>> +++ b/Makefile
>>> @@ -1350,7 +1350,7 @@ u-boot-x86-16bit.bin: u-boot FORCE
>>>  endif
>>>  
>>>  ifneq ($(CONFIG_ARCH_SUNXI),)
>>> -ifeq ($(CONFIG_ARM64),)
>>> +ifeq ($(CONFIG_SPL_FIT_SOURCE)$(CONFIG_SPL_FIT_GENERATOR),)
>>
>> The commit message sounds as if this should be an "ifneq"?
> 
> If find those ifeq(something,) constructs always confusing, but I think
> it's right here: The line in the if-clause creates the *legacy* U-Boot
> image, so the two strings must be both empty for it to match and to
> execute those commands. If only one of them is "y", the comparison with
> the empty string will fail and the else clause will be called, which
> creates the FIT image.
> 
> Or did I trick myself?

No, I just got confused myself, sorry :). All good.

Reviewed-by: Alexander Graf <agraf at suse.de>


Alex


More information about the U-Boot mailing list