Re: [PATCH v1 1/2] ARM: dts: stm32: add support of stm32mp157f-dk2 board

[Alexandre TORGUE <alexandre.torgue@foss.st.com>]

Hi Himanshu

On 5/24/25 12:03, Himanshu Bhavani wrote:
> STM32MP157F is similar to the STM32MP157C, so reuse the existing
> stm32mp157c-dk2.dts as a base for the STM32MP157F-DK2 board.
> 
> Datasheet: https://www.st.com/resource/en/data_brief/stm32mp157f-dk2.pdf
> Signed-off-by: Himanshu Bhavani <himanshu.bhavani@siliconsignals.io>
> ---
>   arch/arm/boot/dts/st/Makefile            |  3 ++-
>   arch/arm/boot/dts/st/stm32mp157f-dk2.dts | 15 +++++++++++++++
>   2 files changed, 17 insertions(+), 1 deletion(-)
>   create mode 100644 arch/arm/boot/dts/st/stm32mp157f-dk2.dts
> 
> diff --git a/arch/arm/boot/dts/st/Makefile b/arch/arm/boot/dts/st/Makefile
> index 60d55516f723..38179638e5eb 100644
> --- a/arch/arm/boot/dts/st/Makefile
> +++ b/arch/arm/boot/dts/st/Makefile
> @@ -70,7 +70,8 @@ dtb-$(CONFIG_ARCH_STM32) += \
>   	stm32mp157c-lxa-tac-gen2.dtb \
>   	stm32mp157c-odyssey.dtb \
>   	stm32mp157c-osd32mp1-red.dtb \
> -	stm32mp157c-phycore-stm32mp1-3.dtb
> +	stm32mp157c-phycore-stm32mp1-3.dtb \
> +	stm32mp157f-dk2.dtb
>   dtb-$(CONFIG_ARCH_U8500) += \
>   	ste-snowball.dtb \
>   	ste-hrefprev60-stuib.dtb \
> diff --git a/arch/arm/boot/dts/st/stm32mp157f-dk2.dts b/arch/arm/boot/dts/st/stm32mp157f-dk2.dts
> new file mode 100644
> index 000000000000..ab406b2c44c1
> --- /dev/null
> +++ b/arch/arm/boot/dts/st/stm32mp157f-dk2.dts
> @@ -0,0 +1,15 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) 2025 Silicon Signals Pvt. Ltd.
> + *
> + * Author: Himanshu Bhavani <himanshu.bhavani@siliconsignals.io>
> + */
> +
> +/dts-v1/;
> +
> +#include "stm32mp157c-dk2.dts"

In this case what is the aim of this series if you just use stm32mp157C 
devicetree ?

ST people (mainly Amélie) are preparing a full support of 
STM32MP157F-DK2 leveraging the F variant compare to the C variant. 
Series will be sent this week.

Regards
Alex

> +
> +/ {
> +	model = "STMicroelectronics STM32MP157F-DK2 Discovery Board";
> +	compatible = "st,stm32mp157f-dk2", "st,stm32mp157";
> +};