[U-Boot] [RFC PATCH 1/4] Adapt config.mk for usage in spl/Makefile

[Aneesh V]

Dear Wolfgang,

On Friday 08 July 2011 02:38 PM, Wolfgang Denk wrote:
> Dear Daniel Schwierzeck,
>
> In message<1309883182-12854-2-git-send-email-daniel.schwierzeck at googlemail.com>  you wrote:
>> [aneesh at ti.com:
>> 1. Changed the definition of OBJTREE and the way 'obj' is defined.
>> 2. Defined a flag to identify regular U-Boot build
>> 3. Taking care of autoconf.mk etc for SPL due to changed OBJTREE
>> ]
>>
>> [daniel.schwierzeck at googlemail.com:
>> 1. reverted changes on implicite make rules
>> 2. readded definition of -DCONFIG_UBOOT_SPL_BUILD in CPPFLAGS
>> ]
> ...
>> +# Allow compiling of C and ASM code parts in different way.
>> +# Serves also as a replacement for CONFIG_PRELOADER.
>> +ifneq ($(CONFIG_UBOOT_SPL_BUILD),)
>> +CPPFLAGS += -DCONFIG_UBOOT_SPL_BUILD
>> +endif
>
> I don't understand this comment.
>
>> +CONFIG_NORMAL_UBOOT = $(if $(CONFIG_UBOOT_SPL_BUILD),,y)
>
> Is this not redundant, then?

Yes, it is. But it was handy for conditional inclusion of source files
in the Makefiles, like this:

+# used only by normal u-boot
+COBJS-$(CONFIG_NORMAL_UBOOT)	+= fileA.o
+# used only by SPL
+COBJS-$(CONFIG_UBOOT_SPL_BUILD)	+= fileB.o
+# used by SPL and normal u-boot
+COBJS-y	+= fileC.o

Without CONFIG_NORMAL_UBOOT this becomes a little cumbersome.

best regards,
Aneesh