From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mike Frysinger <vapier@gentoo.org>
Date: Sun, 14 Jun 2009 11:32:05 -0400
Subject: [U-Boot] [PATCH/next v3] allow boards to customize compiler
	options on a per-file/dir basis
In-Reply-To: <20090614151008.GK22102@game.jcrosoft.org>
References: <1242630364-2510-1-git-send-email-vapier@gentoo.org>
	<20090614144551.43271832E416@gemini.denx.de>
	<20090614151008.GK22102@game.jcrosoft.org>
Message-ID: <200906141132.06546.vapier@gentoo.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 Sunday 14 June 2009 11:10:08 Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 16:45 Sun 14 Jun     , Wolfgang Denk wrote:
> > Dear Jean-Christophe PLAGNIOL-VILLARD,
> >
> > In message <20090614142939.GG22102@game.jcrosoft.org> you wrote:
> > > > +# Allow boards to use custom optimize flags on a per dir/file basis
> > > > +BCURDIR := $(notdir $(CURDIR))
> > > >  $(obj)%.s:	%.S
> > > > -	$(CPP) $(AFLAGS) -o $@ $<
> > > > +	$(CPP) $(AFLAGS) $(AFLAGS_$(@F)) $(AFLAGS_$(BCURDIR)) -o $@ $<
> > >
> > > we can drop this one as there is no obj %.s
> >
> > But since it's already present it also makes sense to keep it, so we
> > don;t have to figure out what it needs to be in case we have to add an
> > assembler file later.
>
> the .s is supposed to be gcc temp file

not really.  the .s file means "do not run preprocessor before sending to 
assembler".  that is all.

> It make sense to avoid it as assembly file suffix is .S for source

usually, yes, but no need to force people to conform if they want to write 
pure assembly with no CPP stuff.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20090614/23083ec4/attachment.pgp