Re: [PATCH v5 13/13] arm: u8500: Package Stemmy device trees in FIT
Stephan Gerhold <[email protected]>
| Newsgroups | org.u-boot-project.lists.u-boot |
|---|---|
| Message-ID | <[email protected]> |
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 <[email protected]> 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