[PATCH 1/1] api: move API related config options into submenu
Heinrich Schuchardt
heinrich.schuchardt at canonical.com
Mon Mar 6 11:18:17 CET 2023
On 3/4/23 16:32, Tom Rini wrote:
> On Fri, Mar 03, 2023 at 11:31:22PM +0100, Heinrich Schuchardt wrote:
>
>> Kconfig settings that are related to the API for standalone applications
>> should be in the API sub-menu and not on the top level.
>>
>> CONFIG_STANDALONE_LOAD_ADDR is only relevant if standalone example
>> applications are built.
>>
>> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>> ---
>> Kconfig | 8 --------
>> api/Kconfig | 11 ++++++++++-
>> 2 files changed, 10 insertions(+), 9 deletions(-)
>
> Did you put this through CI? It's possible that some envs don't do
> "loadaddr=CONFIG_STANDALONE_LOAD_ADDR" and not enable API stuff anymore,
> but I think that's why I did what I did when migrating.
>
Hello Tom,
we should keep the main Kconfig menu clean of detail settings. I don't
thin that there is an issue with the current patch.
STANDALONE_LOAD_ADDR is not used for loadaddr:
$ git grep -n STANDALONE_LOAD_ADDR
(based on origin/master)
api/Kconfig:15
config STANDALONE_LOAD_ADDR
config.mk:79
export CONFIG_STANDALONE_LOAD_ADDR
configs/display5_defconfig:33
CONFIG_STANDALONE_LOAD_ADDR=0x10001000
configs/display5_factory_defconfig:30
CONFIG_STANDALONE_LOAD_ADDR=0x10001000
configs/microchip_mpfs_icicle_defconfig:15
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv32_defconfig:12
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv32_smode_defconfig:13
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv32_spl_defconfig:15
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv64_defconfig:12
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv64_smode_defconfig:13
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/qemu-riscv64_spl_defconfig:14
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/sifive_unleashed_defconfig:21
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/sifive_unmatched_defconfig:24
CONFIG_STANDALONE_LOAD_ADDR=0x80200000
configs/xtfpga_defconfig:12
CONFIG_STANDALONE_LOAD_ADDR=0x00800000
examples/standalone/Makefile:45
LDFLAGS_STANDALONE += -Ttext $(CONFIG_STANDALONE_LOAD_ADDR)
tools/patman/test_checkpatch.py:208
CONFIG_STANDALONE_LOAD_ADDR
With the patch applied
https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/15474
showed no issues.
Best regards
Heinrich
More information about the U-Boot
mailing list