[PATCH v2] ARM: dts: stm32: add U-Boot specific dtsi for stm32mp157f-dk2

[Dario Binacchi]

Hi Patrice

On Mon, Jun 29, 2026 at 9:29 AM Patrice CHOTARD
<patrice.chotard at foss.st.com> wrote:
>
>
>
> On 5/29/26 17:31, Dario Binacchi wrote:
> > The STM32MP157F-DK2 board is a direct derivative of the STM32MP157C-DK2.
> > As explained by ST [1], the "STM32MP157F-DK2 is assembled with latest
> > STM32MP15 RevZ cut which is the only one available for new product now.
> > The major differences in this 'F' variant SoC are the increased max
> > frequency of the Cortex-A7 cores and the correction of some HW bugs
> > (Device Rev.Z, see errata ES0438)".
> >
> > On the software/architecture side, this variant utilizes SCMI (System
> > Control and Management Interface) for secure world resource management.
> > As a consequence, the underlying stm32mp15-scmi.dtsi explicitly deletes
> > several fixed clock nodes (such as clk_hse, clk_hsi, clk_lse, clk_lsi,
> > and clk_csi), making it impossible to include stm32mp157c-dk2-u-boot.dtsi
> > without triggering compilation errors due to these missing clock labels.
> >
> > So let's introduce a dedicated stm32mp157f-dk2-u-boot.dtsi that directly
> > defines the fwu-mdata node. This provides the necessary FWU metadata
> > configuration required to allow automatic A/B bank selection, supported
> > for STM32MP15 boards by commit 560d8f32703f ("board: st: stm32mp15:
> > support dynamic A/B bank bootup").
> >
> > [1] https://community.st.com/t5/stm32-mpus-products-and-hardware/i-m-wondering-what-s-the-difference-point-between-stm32mp157c/td-p/213460
> > Signed-off-by: Dario Binacchi <dario.binacchi at amarulasolutions.com>
> >
> > ---
> >
> > Changes in v2:
> > - Drop the inclusion of stm32mp157c-dk2-u-boot.dtsi to avoid compilation
> >   errors caused by deleted clock nodes in stm32mp15-scmi.dtsi.
> > - Directly define the fwu-mdata node in stm32mp157f-dk2-u-boot.dtsi to
> >   properly support automatic A/B bank selection.
> >
> >  arch/arm/dts/stm32mp157f-dk2-u-boot.dtsi | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >  create mode 100644 arch/arm/dts/stm32mp157f-dk2-u-boot.dtsi
> >
> > diff --git a/arch/arm/dts/stm32mp157f-dk2-u-boot.dtsi b/arch/arm/dts/stm32mp157f-dk2-u-boot.dtsi
> > new file mode 100644
> > index 000000000000..77805417bfa9
> > --- /dev/null
> > +++ b/arch/arm/dts/stm32mp157f-dk2-u-boot.dtsi
> > @@ -0,0 +1,11 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * Copyright (C) 2026 Amarula Solutions, Dario Binacchi <dario.binacchi at amarulasolutions.com>
> > + */
> > +
> > +/ {
> > +     fwu-mdata {
> > +             compatible = "u-boot,fwu-mdata-gpt";
> > +             fwu-mdata-store = <&sdmmc1>;
> > +     };
> > +};
>
> Hi Dario
>
> What about including stm32mp157c-dk2-u-boot.dtsi instead of defining fwu-mdata node ?

This was actually the approach used in v1. However, as you pointed out in the
v1 thread [1], it does not build because stm32mp15-scmi.dtsi removes the
clk_hse, clk_hsi, clk_lse, clk_lsi and clk_csi labels referenced by
stm32mp157c-dk2-u-boot.dtsi.

That's why I dropped the include in v2 and defined only the fwu-mdata node
locally.

Or am I perhaps missing something?

[1] https://patchwork.ozlabs.org/project/uboot/patch/20260512071157.564081-1-dario.binacchi@amarulasolutions.com/

Thanks and regards,
Dario

>
> Thanks
> Patrice



-- 
Dario Binacchi
Senior Embedded Software Engineer
M. +39 328 0625246
dario.binacchi at amarulasolutions.com
―――――――――――――――
Amarula Solutions SRL
 Via Felice Cavallotti 25D, 41012 Carpi, MO, IT
info at amarulasolutions.com
www.amarulasolutions.com