Re: [PATCH 8/8] configs: am64x_evm_r5/a53_defconfig: Enable configs required for Ethboot

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1299 bytes --]

On Fri, Jan 21, 2022 at 08:48:05PM +0530, Vignesh Raghavendra wrote:
> 
> 
> On 21/01/22 8:27 pm, Tom Rini wrote:
> > On Fri, Jan 21, 2022 at 09:59:21AM +0530, Vignesh Raghavendra wrote:
> >>
> >>
> >> On 17/01/22 8:47 am, Tom Rini wrote:
> >>> On Fri, Dec 24, 2021 at 12:55:36PM +0530, Vignesh Raghavendra wrote:
> >>>
> >>>> Enable config options needed to support Ethernet boot on AM64x SK.
> >>>>
> >>>> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> >>>
> >>> This leads to size overflow on r5.
> >>>
> >>
> >> Hmm, I don't see size limitations locally.
> >>
> >> $ size spl/u-boot-spl
> >>    text	   data	    bss	    dec	    hex	filename
> >>  230470	  10401	  12992	 253863	  3dfa7	spl/u-boot-spl
> >>
> >> vs limit declared in configs/am64x_evm_r5_defconfig:
> >>
> >> CONFIG_SPL_SIZE_LIMIT=0x190000
> >>
> >> I don't expect size overflow?
> > 
> > What toolchain are you using?  I'm seeing this with gcc-11.1.
> > 
> 
> I am using 10.3.1. Will try 11.1. But what's is overflow error being
> reported?
> 
> Also, could you try if [1] compiles fine on top of latest master?
> 
> [1]
> https://lore.kernel.org/u-boot/20220121071754.197577-1-vigneshr@ti.com/T/#t

Yes, this builds, so I guess you can set aside the report.  Thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]