On 2019/09/27 11:18, Kurt Miller wrote:
> On Fri, 2019-09-27 at 16:11 +0100, Stuart Henderson wrote:
> > On 2019/09/27 10:35, Kurt Miller wrote:
> > > 
> > > On Fri, 2019-09-27 at 23:50 +1000, Jonathan Gray wrote:
> > > > 
> > > > On Fri, Sep 27, 2019 at 06:44:24PM +1000, Jonathan Gray wrote:
> > > > > 
> > > > > 
> > > > > With U-Boot 2019.10-rc4 the am335x_evm SPL no longer fits when 
> > > > > building
> > > > > with gcc 6.3 but does fit with 7.4.
> > > > and with gcc 7.4 arm-trusted-firmware doesn't fit...
> > > > 
> > > > aarch64-none-elf-ld.bfd: build/rk3399/debug/bl31/bl31.elf section 
> > > > `.pmusram' will not fit in region `PMUSRAM'
> > > > aarch64-none-elf-ld.bfd: region `PMUSRAM' overflowed by 3928 bytes
> > > > 
> > > This because CFLAGS passed by ports is overriding -Os
> > > in the build.
> > > 
> > > Does the following look ok?
> > > 
> > > * Disable passing CFLAGS on to build as it overrides -Os and causes
> > > the build to fail.
> > > * Add rk3328
> > > 
> > > Index: Makefile
> > > ===================================================================
> > > RCS file: /cvs/ports/sysutils/arm-trusted-firmware/Makefile,v
> > > retrieving revision 1.11
> > > diff -u -p -u -r1.11 Makefile
> > > --- Makefile      12 Jul 2019 20:49:39 -0000      1.11
> > > +++ Makefile      27 Sep 2019 14:31:37 -0000
> > > @@ -9,6 +9,7 @@ GH_PROJECT=       arm-trusted-firmware
> > >  GH_TAGNAME=      v2.1
> > >  
> > >  EPOCH=           0
> > > +REVISION=        0
> > >  
> > >  CATEGORIES=      sysutils
> > >  
> > > @@ -23,8 +24,10 @@ MAKE_ENV+= BUILD_STRING="${GH_TAGNAME:S/
> > >  
> > >  NO_TEST= Yes
> > >  USE_GMAKE=       Yes
> > > +CFLAGS=
> > This is non-obvious enough that I'd like to have a comment please.
> > Or would it be better to make it explicit with CFLAGS=-Os instead?
> > 
> 
> Checking the build log, I see various optimization levels
> in use. I think we should allow it to use those levels
> since this is building a firmware component.

> We could try rearrange the flags so that CFLAGS from the
> env comes first if that is desired. I will a comment in
> if we decide to keep the CFLAGS= in the port make file.

Oh in that case I'm totally happy to avoid us overriding it,
CFLAGS= and a comment sounds like the best option them.

Reply via email to