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. Cheers, Andre. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot