Heinrich,

On 02.04.19 19:19, Heinrich Schuchardt wrote:
The SPL image for the Tinker Board has to fit into 32 KiB. This includes
up to 2 KiB for the file header.

A new configuration variable CONFIG_SPL_SIZE_LIMIT is introduced to define
the board specific limit.

A common Makefile function is used for this test and the test against
CONFIG_BOARD_SIZE_LIMIT.

Move the board size check from arch/arm/mach-imx/Makefile to Makefile.

Has anything from this series been applied? I now have a working patch that applies on top of this and adds a full SPL SRAM size check (including HEAP, GD and stack; via a host tool) which works for socfpga (as an example of a platform where SPL binary is loaded to limited SRAM).

Actually, my patch would replace your patch 3/4 but build on 1/4 (2/4 and 4/4 are arch-specific).

How should we proceed here? I could send a series including your 1/4, or I could send a series completely building on this series, at the downside of more or less reverting your 2/4.

Regards,
Simon


v4:
        use a common function for all size checks in the Makefiles

Heinrich Schuchardt (4):
   Makefile: reusable function for BOARD_SIZE_CHECK
   imx: move BOARD_SIZE_CHECK to main Makefile
   configs: define CONFIG_SPL_SIZE_LIMIT
   configs: rk3288: Tinker Board SPL file must fit into 32 KiB

  Kconfig                         |  8 ++++++++
  Makefile                        | 33 +++++++++++++++++++++++----------
  arch/arm/mach-imx/Makefile      | 16 ----------------
  configs/tinker-rk3288_defconfig |  1 +
  4 files changed, 32 insertions(+), 26 deletions(-)

--
2.20.1

_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to