[U-Boot] [PATCH v2 1/7] tegra: Rework Tamonten support

[Allen Martin]

On Fri, Jun 08, 2012 at 02:07:25PM -0700, Stephen Warren wrote:
> 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.

The problem is this:

> >>  ifneq ($(OBJTREE),$(SRCTREE))

is true, but these directories:

> >> +$(shell mkdir -p $(obj)../common $(obj)../../nvidia/common)

don't exist because $(obj) is not under $(OBJTREE) in the SPL build,
it's under $(SPLTREE)

-Allen
-- 
nvpublic