[U-Boot] [RFC PATCH v2 11/11] spl: Kconfig: Add CONFIG_SPL_TEXT_BASE as Kconfig option

Masahiro Yamada
2016-08-23T05:45:22Z

2016-08-23 0:21 GMT+09:00 Andrew F. Davis <afd@ti.com>:
> Add a new Kconfig CONFIG_SPL_TEXT_BASE. Set its default value to 0x0.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
> Kconfig | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Kconfig b/Kconfig
> index cb5af5b..2935c41 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -319,6 +319,14 @@ config SYS_TEXT_BASE
> help
> TODO: Move CONFIG_SYS_TEXT_BASE for all the architecture
>
> +config SPL_TEXT_BASE
> + hex "SPL Text Base"
> + depends on SPL
> + default 0x0
> + help
> + When SPL is enabled the image is linked with this as the
> + starting address in memeory for which the SPL will be loaded.
> +
> config SPL_LOAD_FIT
> bool "Enable SPL loading U-Boot as a FIT"
> depends on FIT

For example, my SoC, configs/uniphier_ld20_defconfig,
needs to set CONFIG_SPL_TEXT_BASE to 0x100000
because its Boot ROM loads SPL onto 0x100000.

It is not user-configurable at all, so I do not see much sense for the prompt.
I am not quite sure for other SoCs, but I guess similar situation.

(If we support relocation for SPL, we can make SPL_TEXT_BASE
a configurable parameter, but we do not have it at this point of time.)

Is it better to hide the prompt unless EXPERT is defined?

config SPL_TEXT_BASE
hex "SPL Text Base" if EXPERT

--
Best Regards
Masahiro Yamada