[U-Boot] [PATCH v4 01/19] sunxi: README.sunxi64: Add hint about non-debug of ARM Trusted Firmware

[André Przywara]

Hi,

On 21/03/18 18:40, Jagan Teki wrote:
> On Wed, Mar 14, 2018 at 7:26 AM, Andre Przywara <andre.przywara at 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.przywara at arm.com>
>> Acked-by: Maxime Ripard <maxime.ripard at 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.