Re: [PATCH v5 13/13] arm: u8500: Package Stemmy device trees in FIT

[Stephan Gerhold <stephan.gerhold@linaro.org>]

On Sun, Aug 23, 2026 at 10:39:32PM +0200, Linus Walleij wrote:
> Build all seven supported Samsung Ux500 device trees into the FIT
> appended to U-Boot. Select the appropriate configuration at runtime
> from the machine ID passed by the Samsung firmware. Codina and Codina
> TMO share a machine ID, so distinguish Codina TMO by its board ID.
> 
> Keep a Kconfig default for the generic multi-DTB build dependency, but
> remove it from the defconfig since the board selection is now dynamic.
> Print the firmware machine and board IDs to aid diagnostics and future
> machine ID mappings.
> 
> Signed-off-by: Linus Walleij <linusw@kernel.org>

Yes! This is great, thanks :)

> ---
>  arch/arm/Kconfig          |  1 +
>  board/ste/stemmy/Kconfig  |  6 ++++
>  board/ste/stemmy/stemmy.c | 77 ++++++++++++++++++++++++++++++++++++++++++++---
>  configs/stemmy_defconfig  |  1 -
>  4 files changed, 79 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 2a01ac9b7675..8e697b87d50f 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1278,6 +1278,7 @@ config ARCH_U8500
>  	imply DM_REGULATOR
>  	imply DM_REGULATOR_FIXED
>  	imply DM_RTC
> +	imply MULTI_DTB_FIT
>  	imply NOMADIK_GPIO
>  	imply NOMADIK_MTU_TIMER
>  	imply PINCTRL

It's a bit weird to have this in ARCH_U8500 since there may be other
(non-Samsung) U8500 boards who couldn't use MULTI_DTB_FIT.

Can you put this into stemmy_defconfig?

> diff --git a/board/ste/stemmy/Kconfig b/board/ste/stemmy/Kconfig
> index b890ba51cb03..e891ddf9f8b2 100644
> --- a/board/ste/stemmy/Kconfig
> +++ b/board/ste/stemmy/Kconfig
> @@ -9,4 +9,10 @@ config SYS_VENDOR
>  config SYS_CONFIG_NAME
>  	default "stemmy"
>  
> +config DEFAULT_DEVICE_TREE
> +	default "st/ste-ux500-samsung-janice"
> +
> +config OF_LIST
> +	default "st/ste-ux500-samsung-codina st/ste-ux500-samsung-codina-tmo st/ste-ux500-samsung-gavini st/ste-ux500-samsung-golden st/ste-ux500-samsung-janice st/ste-ux500-samsung-kyle st/ste-ux500-samsung-skomer"
> +
>  endif

I don't mind adding this here, but I'm also not sure if there is any
advantage compared to adding this directly in stemmy_defconfig?

Thanks,
Stephan