On Fri, 4 Oct 2024 at 16:59, Marek Vasut <[email protected]> wrote: > > Describe the u-boot.its generation in stm32mp15xx-dhsom-u-boot.dtsi > binman {} DT node as a replacement for current CONFIG_SPL_FIT_SOURCE > use, dispose of both u-boot-dhcom.its and u-boot-dhcor.its. > > Use fdt-SEQ/config-SEQ to generate a list of fdt-N fitImage images {} and > matching configuration {} node entries. The configuration node entry names > no longer encode _somrevN_boardrevN suffix, which was never really used, so > drop this functionality by default. Rework board_fit_config_name_match() to > match on the new configuration node entry names. > > Users who do need the match on _somrevN_boardrevN can either replace the > fdt-SEQ/config-SEQ with fixed fdt-N/config-N nodes which each encode the > matching 'description = "NAME_somrevN_boardrevN"' to restore the old > behavior verbatim, or better use SPL DT overlays for U-Boot control DT > the same way e.g. i.MX8MP DHCOM does to support multiple SoM and board > variants. > > Signed-off-by: Marek Vasut <[email protected]> > --- > Cc: Patrice Chotard <[email protected]> > Cc: Patrick Delaunay <[email protected]> > Cc: Sean Anderson <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Tom Rini <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > --- > arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi | 1 + > arch/arm/dts/stm32mp15xx-dhcor-u-boot.dtsi | 1 + > arch/arm/dts/stm32mp15xx-dhsom-u-boot.dtsi | 53 +++++++++++ > board/dhelectronics/dh_stm32mp1/board.c | 19 +++- > .../dh_stm32mp1/u-boot-dhcom.its | 91 ------------------- > .../dh_stm32mp1/u-boot-dhcor.its | 70 -------------- > configs/stm32mp15_dhcom_basic_defconfig | 2 - > configs/stm32mp15_dhcor_basic_defconfig | 2 - > 8 files changed, 70 insertions(+), 169 deletions(-) > create mode 100644 arch/arm/dts/stm32mp15xx-dhsom-u-boot.dtsi > delete mode 100644 board/dhelectronics/dh_stm32mp1/u-boot-dhcom.its > delete mode 100644 board/dhelectronics/dh_stm32mp1/u-boot-dhcor.its
For the use of Binman: Reviewed-by: Simon Glass <[email protected]>
