diff options
author | Eric Ruei | 2016-08-09 10:31:41 -0500 |
---|---|---|
committer | Anand Balagopalakrishnan | 2016-08-26 12:37:43 -0500 |
commit | a5a0fe1799cabfa1278321b667263cf3fa2cf32f (patch) | |
tree | fa1b582249fe76085cbb9d9ce82cdebf6f9a5651 | |
parent | c13efb766a8fc680374ae7affc6df535f823bacb (diff) | |
download | omap5-sgx-ddk-linux-a5a0fe1799cabfa1278321b667263cf3fa2cf32f.tar.gz omap5-sgx-ddk-linux-a5a0fe1799cabfa1278321b667263cf3fa2cf32f.tar.xz omap5-sgx-ddk-linux-a5a0fe1799cabfa1278321b667263cf3fa2cf32f.zip |
km: Resolve Linux build warnings
Compiler warnings were observed at DDK Kernel build at PSDK. They are
caused by the double prefixes of $CC as $(CROSS_COMPILE)$(CROSS_COMPILE)gcc
because $CC is already set to (CROSS_COMPILE)gcc at Rules.make at PSDK.
This problem can be resolved by adding a "string contains" check to make
the $CC prefix operation conditional. The proposed fixes have been submitted
to IMG and approved by IMG engineers as ticket 101146
Please note that we only made minimum changes to resolve the compiler warnings and
there is no intention to enhance the IMG make procedures to handle all similar cases.
Change-Id: I54d27af799863fff5662de2050a93e2498c2ed82
Signed-off-by: Eric Ruei <e-ruei1@ti.com>
Reviewed-on: https://gerrit.ext.ti.com/gerrit/omap/14288
Reviewed-by: Anand Balagopalakrishnan <anandb@ti.com>
-rw-r--r-- | eurasia_km/eurasiacon/build/linux2/commands.mk | 4 | ||||
-rw-r--r-- | eurasia_km/eurasiacon/build/linux2/config/core.mk | 2 |
2 files changed, 6 insertions, 0 deletions
diff --git a/eurasia_km/eurasiacon/build/linux2/commands.mk b/eurasia_km/eurasiacon/build/linux2/commands.mk index cb92174..a5cef58 100644 --- a/eurasia_km/eurasiacon/build/linux2/commands.mk +++ b/eurasia_km/eurasiacon/build/linux2/commands.mk | |||
@@ -224,7 +224,11 @@ override CXX := $(if $(V),,@)$(CCACHE) $(CXX) \ | |||
224 | -target $(patsubst %-,%,$(CROSS_COMPILE)) \ | 224 | -target $(patsubst %-,%,$(CROSS_COMPILE)) \ |
225 | -B$(dir $(shell which $(CROSS_COMPILE)gcc)) | 225 | -B$(dir $(shell which $(CROSS_COMPILE)gcc)) |
226 | else | 226 | else |
227 | ifeq (,$(findstring $(CROSS_COMPILE), $(CC))) | ||
227 | override CC := $(if $(V),,@)$(CCACHE) $(CROSS_COMPILE)$(CC) | 228 | override CC := $(if $(V),,@)$(CCACHE) $(CROSS_COMPILE)$(CC) |
229 | else | ||
230 | override CC := $(if $(V),,@)$(CCACHE) $(CC) | ||
231 | endif | ||
228 | override CXX := $(if $(V),,@)$(CCACHE) $(CROSS_COMPILE)$(CXX) | 232 | override CXX := $(if $(V),,@)$(CCACHE) $(CROSS_COMPILE)$(CXX) |
229 | endif | 233 | endif |
230 | else | 234 | else |
diff --git a/eurasia_km/eurasiacon/build/linux2/config/core.mk b/eurasia_km/eurasiacon/build/linux2/config/core.mk index 30ba321..08ff716 100644 --- a/eurasia_km/eurasiacon/build/linux2/config/core.mk +++ b/eurasia_km/eurasiacon/build/linux2/config/core.mk | |||
@@ -228,8 +228,10 @@ _CLANG := \ | |||
228 | ifeq ($(_CLANG),true) | 228 | ifeq ($(_CLANG),true) |
229 | _CC := $(_CC) -target $(patsubst %-,%,$(CROSS_COMPILE)) | 229 | _CC := $(_CC) -target $(patsubst %-,%,$(CROSS_COMPILE)) |
230 | else | 230 | else |
231 | ifeq (,$(findstring $(CROSS_COMPILE), $(_CC))) | ||
231 | _CC := $(CROSS_COMPILE)$(_CC) | 232 | _CC := $(CROSS_COMPILE)$(_CC) |
232 | endif | 233 | endif |
234 | endif | ||
233 | HOST_CC ?= gcc | 235 | HOST_CC ?= gcc |
234 | 236 | ||
235 | -include ../config/user-defs.mk | 237 | -include ../config/user-defs.mk |