On 06/08/2012 02:01 PM, Allen Martin wrote:
On Fri, May 25, 2012 at 06:46:22AM -0700, Thierry Reding wrote:
This commit uses the common Tegra board implementation instead of duplicating a lot of the code. In addition, the Plutux and Medcom specific board files can be removed as the MMC/SD setup is common among all Tamonten-based boards.
... diff --git a/board/avionic-design/medcom/Makefile b/board/avionic-design/medcom/Makefile index b0c318c..d96d043 100644 --- a/board/avionic-design/medcom/Makefile +++ b/board/avionic-design/medcom/Makefile @@ -26,12 +26,12 @@ include $(TOPDIR)/config.mk
ifneq ($(OBJTREE),$(SRCTREE)) -$(shell mkdir -p $(obj)../common) +$(shell mkdir -p $(obj)../common $(obj)../../nvidia/common) endif
LIB = $(obj)lib$(BOARD).o
This breaks with my patchset to split out the arm7 code to an SPL because even though SRCDIR and OBJDIR are the same, in the SPL build $(obj) is a subdir of $(SPLTREE) not $(OBJTREE) (not my doing).
How about the following instead which seems more to the point:
ifeq ($(wildcard $(obj)../common),) $(shell mkdir -p $(obj)../common) endif ifeq ($(wildcard $(obj)../../nvidia/common),) $(shell mkdir -p $(obj)../../nvidia/common) endif
Maybe I'm just not reading it right, but isn't that just running the exact same mkdir commands, just splitting it into two commands, and making them optional based on the $(wildcard)? I'm still not clear what the problem is.
Anyway, if this is an issue, then compal/paz00 and compulabl/trimslice will need to be fixed for the SPL changes in the same way.