Dear Aneesh V,
In message 4E16D9DA.4080803@ti.com you wrote:
+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.
Hm... instead of
COBJS-$(CONFIG_NORMAL_UBOOT) += fileA.o
we could use
COBJS-$(if $(CONFIG_UBOOT_SPL_BUILD),,y)
directly.
Note that Daniel's patches never referenced CONFIG_NORMAL_UBOOT anywhere, so I think we can remove it for now and decide when it turns out we really need it.
Best regards,
Wolfgang Denk