On 21/03/18 19:08, Jagan Teki wrote: > On Thu, Mar 22, 2018 at 12:33 AM, André Przywara <andre.przyw...@arm.com> > wrote: >> Hi, >> >> On 21/03/18 18:40, Jagan Teki wrote: >>> On Wed, Mar 14, 2018 at 7:26 AM, Andre Przywara <andre.przyw...@arm.com> >>> wrote: >>>> As we are running into issues where the final U-Boot FIT image file is >>>> exceeding our size limit, add a hint to the README.sunxi64 file >>>> to point out the possibility of building non-debug versions of the ATF >>>> binary. These are about 12KB smaller than the standard debug build, and >>>> so allow successful U-Boot builds for many boards with the Allwinner H5 >>>> SoC. >>>> Please note that under normal circumstances the debug build is still >>>> recommended, as it gives valuable clues in case something goes wrong in >>>> the ATF. >>>> >>>> Signed-off-by: Andre Przywara <andre.przyw...@arm.com> >>>> Acked-by: Maxime Ripard <maxime.rip...@bootlin.com> >>>> --- >>>> board/sunxi/README.sunxi64 | 6 ++++++ >>>> 1 file changed, 6 insertions(+) >>>> >>>> diff --git a/board/sunxi/README.sunxi64 b/board/sunxi/README.sunxi64 >>>> index 5a363d27b8..df1dbc818f 100644 >>>> --- a/board/sunxi/README.sunxi64 >>>> +++ b/board/sunxi/README.sunxi64 >>>> @@ -38,6 +38,12 @@ the root of your U-Boot build directory (or create a >>>> symbolic link). >>>> $ export BL31=/src/arm-trusted-firmware/build/sun50iw1p1/debug/bl31.bin >>>> (adjust the actual path accordingly) >>>> >>>> +If you run into size issues with the resulting U-Boot image file, it might >>>> +help to use a release build, by using "DEBUG=0" when building bl31.bin. >>> >>> I thought I've commented this in previous version patch, w/o DEBUG=0 >>> can also build release right? like >>> >>> make PLAT=sun50iw1p1 bl31 >> >> Yes, DEBUG=0 is the (current) default in the Makefile. But what's the >> question? The original recipe said DEBUG=1, so stating DEBUG=0 makes it >> obvious what to change. Given that with DEBUG not defined to 1, ATF is >> *completely* silent, I would very much like to encourage people to use >> DEBUG builds, and just suggesting DEBUG=0 as a workaround. > > My question is without mentioning any DEBUG just build like this > > make PLAT=sun50iw1p1 bl31 > > will be similar to > > make PLAT=sun50iw1p1 DEBUG=0 bl31
Yes, that's what I said above. The Makefile in the ATF root directory sets DEBUG to 0, if not overridden on the make command line. Cheers, Andre. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot