[U-Boot] [PATCH for 2018.03] sunxi: README.sunxi64: Add hint about non-debug ATF

[U-Boot] [PATCH for 2018.03] sunxi: README.sunxi64: Add hint about non-debug ATF

[Andre Przywara]

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. This is about 12KB smaller than the standard debug build, and
so again allows 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@arm.com>
---
Hi,

as the level of desperation about Allwinner 64-bit FIT image sizes rises,
we are getting more into the slightly cheaty section with our mitigations.
However as my previous attempts in fixing this issue were not that warmly
welcomed, I now suggest this rather pragmatic solution, which avoids hacking
U-Boot badly. This fixes the various H5 builds for me.

Cheers,
Andre.

 board/sunxi/README.sunxi64 | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/board/sunxi/README.sunxi64 b/board/sunxi/README.sunxi64
index c492f749b8..eefa5001a2 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.
+As sometimes the ATF build process is a bit picky about the toolchain used,
+or if you can't be bothered with building ATF, there are known working
+binaries in the firmware repository[3], purely for convenience reasons.
+
  SPL/U-Boot
 ------------
 Both U-Boot proper and the SPL are using the 64-bit mode. As the boot ROM
-- 
2.14.1

[U-Boot] [PATCH for 2018.03] sunxi: README.sunxi64: Add hint about non-debug ATF

[Jagan Teki]

On Thu, Feb 22, 2018 at 7:20 AM, Andre Przywara <andre.przywara@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. This is about 12KB smaller than the standard debug build, and
> so again allows 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.

So for H5 we can't able to use debug (:

>
> Signed-off-by: Andre Przywara <andre.przywara@arm.com>
> ---
> Hi,
>
> as the level of desperation about Allwinner 64-bit FIT image sizes rises,
> we are getting more into the slightly cheaty section with our mitigations.
> However as my previous attempts in fixing this issue were not that warmly
> welcomed, I now suggest this rather pragmatic solution, which avoids hacking
> U-Boot badly. This fixes the various H5 builds for me.
>
> Cheers,
> Andre.
>
>  board/sunxi/README.sunxi64 | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/board/sunxi/README.sunxi64 b/board/sunxi/README.sunxi64
> index c492f749b8..eefa5001a2 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.

This is something like release build even w/o DEBUG=0 it will build
same is it? In that case we can say not to use DEBUG=1 just make
PLAT=sun50iw1p1 bl31

[U-Boot] [PATCH for 2018.03] sunxi: README.sunxi64: Add hint about non-debug ATF

[Andre Przywara]

Hi,

On 22/02/18 08:05, Jagan Teki wrote:
> On Thu, Feb 22, 2018 at 7:20 AM, Andre Przywara <andre.przywara@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. This is about 12KB smaller than the standard debug build, and
>> so again allows 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.
> 
> So for H5 we can't able to use debug (:

Only for the ATF and only as long as we have that size limit, which will
ideally be lifted with v2018.05-rc1 (or so).
If you have a better idea, I am all ears.

Cheers,
Andre.

>>
>> Signed-off-by: Andre Przywara <andre.przywara@arm.com>
>> ---
>> Hi,
>>
>> as the level of desperation about Allwinner 64-bit FIT image sizes rises,
>> we are getting more into the slightly cheaty section with our mitigations.
>> However as my previous attempts in fixing this issue were not that warmly
>> welcomed, I now suggest this rather pragmatic solution, which avoids hacking
>> U-Boot badly. This fixes the various H5 builds for me.
>>
>> Cheers,
>> Andre.
>>
>>  board/sunxi/README.sunxi64 | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/board/sunxi/README.sunxi64 b/board/sunxi/README.sunxi64
>> index c492f749b8..eefa5001a2 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.
> 
> This is something like release build even w/o DEBUG=0 it will build
> same is it? In that case we can say not to use DEBUG=1 just make
> PLAT=sun50iw1p1 bl31
> 

[U-Boot] [PATCH for 2018.03] sunxi: README.sunxi64: Add hint about non-debug ATF

[Maxime Ripard]

On Thu, Feb 22, 2018 at 09:03:05AM +0000, Andre Przywara wrote:
> Hi,
> 
> On 22/02/18 08:05, Jagan Teki wrote:
> > On Thu, Feb 22, 2018 at 7:20 AM, Andre Przywara <andre.przywara@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. This is about 12KB smaller than the standard debug build, and
> >> so again allows 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.
> > 
> > So for H5 we can't able to use debug (:
> 
> Only for the ATF and only as long as we have that size limit, which will
> ideally be lifted with v2018.05-rc1 (or so).
> If you have a better idea, I am all ears.

And I'd expect people that want to enable the debug option to also be
able to figure out what they can remove in the configuration.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180222/44791319/attachment.sig>