Makefile: remove BUILD_TAG from KBUILD_CFLAGS
authorStephen Warren <swarren@nvidia.com>
Wed, 10 Feb 2016 22:16:19 +0000 (15:16 -0700)
committerTom Rini <trini@konsulko.com>
Mon, 15 Feb 2016 17:04:47 +0000 (12:04 -0500)
commit15b51878bec835c315c80529a0c4d147d681c8e9
treea0d67e87792995867f5452196798fc2e0ee4df87
parenta094c9211e98f51668d84430b5fe081faa9d4a32
Makefile: remove BUILD_TAG from KBUILD_CFLAGS

If BUILD_TAG is part of KBUILD_CFLAGS, then any time the value changes,
all files get rebuilt. In a continuous integration environment, the value
will change every build. This wastes time, assuming that incremental
builds would otherwise occur.

To solve this, remove BUILD_TAG from KBUILD_CFLAGS and add it to CFLAGS
for just the one file that uses it. This does have the disadvantage that
if any other files want to use the flag, we'll need to duplicate this
custom CFLAGS setup logic. However, it seems unlikely we'll need this.

An alternative would be to add BUILD_TAG to the "local version" and remove
the special case code from display_options.c. However, that would affect
the format of the U-Boot signon message, which may negatively affect
people looking for specific data there. The approach of using
file-specific CFLAGS was suggested by Masahiro Yamada.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Acked-by: Simon Glass <sjg@chromium.org>
Makefile
lib/Makefile