On Monday 17 August 2009 15:31:30 Wolfgang Denk wrote:
> Mike wrote:
> > > Mike, I am aware that Blackfin is already using the C preprocessor to
> > > generate it's linker scripts. I guess this step can now be folded into
> > > this general mechanism. Do you want to provide such a patch, or shall
> > > I do it?
> >
> > toolchains that are older than binutils-2.17 wont work with the Blackfin
> > port, so compatibility with pre-2.16 isnt something this port needs to
> > worry about
>
> Agreed, but the cpp step is now _always_ performed, and in your case
> twice - first in your board directory, and then again by the top level
> Makefile. You could save the local run now.

ok, i missed that aspect.  i think the changes needed are:

lib_blackfin/Makefile:
        -$(obj)u-boot.lds: u-boot.lds.S
        -   $(CPP) $(CPPFLAGS) -D__ASSEMBLY__ -P $^ > $@
lib_blackfin/config.mk:
        -LDSCRIPT = $(obj)lib_$(ARCH)/u-boot.lds
        +LDSCRIPT = $(obj)lib_$(ARCH)/u-boot.lds.S

i can post a patch once yours gets merged
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to