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

[Masahiro Yamada]

2016-08-23 0:21 GMT+09:00 Andrew F. Davis <afd at ti.com>:
> Add a new Kconfig CONFIG_SPL_TEXT_BASE. Set its default value to 0x0.
>
> Signed-off-by: Andrew F. Davis <afd at 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