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

[Patrice CHOTARD]

On 7/2/26 15:00, Dario Binacchi wrote:
> 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?

Hi Dario

Sorry, you are right, it's just following an internal discussion with Patrick and i totally forgot your 
v1 implementation already included  stm32mp157c-dk2-u-boot.dtsi and has compilatioj issue.

Patrice


> 
> [1] https://patchwork.ozlabs.org/project/uboot/patch/20260512071157.564081-1-dario.binacchi@amarulasolutions.com/
> 
> Thanks and regards,
> Dario
> 
>>
>> Thanks
>> Patrice
> 
> 
>