[U-Boot] [PATCH 1/2 V2] Move libgcc inclusion from common Makefile to platform configs files

Mike Frysinger vapier at gentoo.org
Sat Jun 20 14:57:41 CEST 2009


On Saturday 20 June 2009 07:30:41 Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 07:08 Sat 20 Jun     , Mike Frysinger wrote:
> > On Saturday 20 June 2009 06:40:07 Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > On 06:18 Sat 20 Jun     , Mike Frysinger wrote:
> > > > On Saturday 20 June 2009 05:33:26 Jean-Christophe wrote:
> > > > > This patch moves the libgcc Makefile inclusion from the toplevel
> > > > > Makefile to the arch_config.mk files. This is in preparation for
> > > > > the ARM architecture to move away from including libgcc function
> > > > > and only using self-contained U-Boot functions as done in Linux.
> > > >
> > > > why not change the top level Makefile to read:
> > > > PLATFORM_LIBGCC ?= ...
> > > >
> > > > then any board/arch that doesnt want it can simply do:
> > > > PLATFORM_LIBGCC = # dont want it
> > >
> > > because you need to provide the equivalent functions for standalone
> > > application and api and U-Boot ofcourse
> >
> > so move it to config.mk.  this doesnt change the important point: leave
> > PLATFORM_LIBGCC default in the toplevel common files.  what i proposed
> > doesnt limit what you want to do with arm in any way.
>
> I think it's better to let it at arch level and force any new arch adding
> to manage it instead provide a default one

considering most arches want this code, i dont think so.  you're proposing we 
duplicate the same common code in all arches to support one deviating arch -- 
arm.  the defaults should reflect the common state while the deviating ones 
change the behavior as they like.
-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/20090620/89cd12ea/attachment.pgp 


More information about the U-Boot mailing list