[PATCH 0/5] spl: binman: Fixes for BINMAN_SYMBOLS

Alper Nebi Yasak alpernebiyasak at gmail.com
Fri Jun 10 12:58:00 CEST 2022


There's some trouble with an i.MX8M series [1] trying to use binman
symbols. The crux of it is the 'u_boot_any' symbols BINMAN_SYMBOLS
configs declare, and the boards creating partial binman images including
an SPL without a U-Boot the symbol is referring to.

Normally this should be easy to resolve by disabling BINMAN_SYMBOLS
configs, but that causes a build error. Apparently some parts of the SPL
code (RAW_IMAGE_SUPPORT, RAM_DEVICE) use the symbols directly without
guarding them by BINMAN_SYMBOLS, implicitly requiring it.

The first patch fixes the issue above, the rest are minor things I
tinkered with while trying to understand the issue. These apply onto
u-boot/next. I have also triggered an Azure CI run [2] via a Github pull
request.

[1] arm64: binman: use binman symbols for imx
https://lore.kernel.org/u-boot/20220603071715.15212-1-peng.fan@oss.nxp.com/

[2] #20220610.3 spl: binman: Fixes for BINMAN_SYMBOLS
https://dev.azure.com/u-boot/u-boot/_build/results?buildId=4431&view=results


Alper Nebi Yasak (5):
  spl: binman: Fix use of undeclared u_boot_any symbols
  spl: binman: Make TPL_BINMAN_SYMBOLS depend on TPL_FRAMEWORK
  spl: binman: Declare extern symbols for VPL as well
  spl: binman: Let u-boot-spl/vpl symbol declarations be disabled
  spl: binman: Add a config option for binman symbols in VPL

 common/spl/Kconfig     | 12 ++++++------
 common/spl/Kconfig.tpl | 14 +++++++-------
 common/spl/Kconfig.vpl | 12 ++++++++++++
 common/spl/spl.c       | 13 +++++++++----
 common/spl/spl_ram.c   |  2 +-
 include/spl.h          |  2 ++
 6 files changed, 37 insertions(+), 18 deletions(-)

-- 
2.36.1



More information about the U-Boot mailing list