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

Tom Rini
2022-01-21T15:22:30Z

[-- 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 --]