[U-Boot] [PATCH 3/8 v2] Introduce the Tertiary Program loader

[Scott Wood]

On Mon, 24 Jan 2011 23:29:45 +0100
Wolfgang Denk <wd at denx.de> wrote:

> Dear Haiying Wang,
> 
> In message <1295907459.2051.158.camel at haiying-laptop> you wrote:
> >
> > > It's not the name.  But you use it ina few places here, buth then hard
> > > encode "tpl" in a number of other paces there.  Which means that you
> > > cannot change TPL_BOOT to any other value, or building would break.
> > > So why do we need this variable?
> > It follows the same usage of NAND_SPL.
> 
> Ah. I see. Well, so NAND_SPL needs fixing as well :-(

Agreed, it seems like just hardcoding the target name would work fine,
be clearer, and avoid relying on unportable make behavior.

> > > Sorry, I cannot follow - which reset? which semicolon?
> > 
> > Sorry, :%s/reset/rest/. The semicolon is the same one in your previous
> > comments. I meant the part:
> > 	: $(TIMESTAMP_FILE) $(VERSION_FILE) depend
> > 		$(MAKE) -C tpl/board/$(BOARDDIR) all
> 
> Ah, I see what you mean. But that's a colon (':') - a semicolon if a
> ';'.

You called it a semicolon first. :-)

-Scott