From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Fri, 08 Jun 2012 15:07:25 -0600
Subject: [U-Boot] [PATCH v2 1/7] tegra: Rework Tamonten support
In-Reply-To: <20120608200139.GN13311@nvidia.com>
References: <1337953588-20696-1-git-send-email-thierry.reding@avionic-design.de>
	<20120608200139.GN13311@nvidia.com>
Message-ID: <4FD2698D.7030505@wwwdotorg.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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.