From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Andr=c3=a9_Przywara?= Date: Wed, 21 Mar 2018 19:03:57 +0000 Subject: [U-Boot] [PATCH v4 01/19] sunxi: README.sunxi64: Add hint about non-debug of ARM Trusted Firmware In-Reply-To: References: <20180314015715.15615-1-andre.przywara@arm.com> <20180314015715.15615-2-andre.przywara@arm.com> Message-ID: <2d6d713c-3f82-c64f-b87b-c80bae1d2445@arm.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 21/03/18 18:40, Jagan Teki wrote: > On Wed, Mar 14, 2018 at 7:26 AM, Andre Przywara 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 >> Acked-by: Maxime Ripard >> --- >> 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.