]> git.baikalelectronics.ru Git - uboot.git/commit
lib: optee: Remove CONFIG_OPTEE_LOAD_ADDR
authorAlexandru Gagniuc <mr.nuke.me@gmail.com>
Tue, 7 Sep 2021 17:07:08 +0000 (12:07 -0500)
committerTom Rini <trini@konsulko.com>
Tue, 5 Oct 2021 12:47:13 +0000 (08:47 -0400)
commit140fbccfe786736630f71553b159207e86a04407
treea93d4555c5d65f5c3c4ba30b3dda2c15d048a140
parent96bb4e739541d257c5571506ce16ec9d66a03f37
lib: optee: Remove CONFIG_OPTEE_LOAD_ADDR

This value is not used by u-boot, and it should not. The load address
of an OPTEE image is defined by said image. Either a uImage or a FIT
will have a defined load address and entry point. Those values are the
correct ones, not CONFIG_OPTEE_LOAD_ADDR.

Commit d734c45d333a ("optee: Add CONFIG_OPTEE_LOAD_ADDR") justifies
this config by requiring its presence in u-boot's .config for other
images as part of a larger build, claiming it is "the best way".

This argument is not persuasive. U-boot's configuration is driven by
platform requirements, not the other way around. It seems more likely
that the argument is conflating tooling issues with Kconfig. Yocto and
buildroot have excellent mechanisms for defining values across the
board (pun intended). u-boot's Kconfig is the wrong place to do it.

Furthermore, it is not "best" for u-boot because it hardcodes a value
which is then not used. In fact the load address that u-boot uses is
the one derived from the OPTEE image.

Confused yet? I sure was. To prevent future confusion, remove
CONFIG_OPTEE_LOAD_ADDR.

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
configs/warp7_defconfig
include/configs/warp7.h
lib/optee/Kconfig