Re: [PATCH v2 2/7] dt-bindings: stm32: add kernel clocks for ETR and STM

[Krzysztof Kozlowski <krzk@kernel.org>]

On Fri, Oct 23, 2026 at 06:43:18PM +0200, Gabriel Fernandez wrote:
> Add the kernel clocks of the ETR (CK_KER_ETR) and of the STM
> (CK_KER_STM) for STM32MP25, with the matching SCMI clocks
> (CK_SCMI_KER_ETR and CK_SCMI_KER_STM).
> These are the trace (ATB) clocks of the CoreSight Embedded
> Trace Router and System Trace Macrocell, while the bus clocks
> only provide register access (APB).
> 
> Also rename the bus clocks according to the reference manual,
> without changing their values: CK_BUS_STM500 into CK_BUS_STM,
> CK_SCMI_STM500 into CK_SCMI_BUS_STM and CK_SCMI_ETR into
> CK_SCMI_BUS_ETR.
> 
> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@foss.st.com>
> ---
>  include/dt-bindings/clock/st,stm32mp25-rcc.h | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/include/dt-bindings/clock/st,stm32mp25-rcc.h b/include/dt-bindings/clock/st,stm32mp25-rcc.h
> index 242f61205a9f..cef997508b56 100644
> --- a/include/dt-bindings/clock/st,stm32mp25-rcc.h
> +++ b/include/dt-bindings/clock/st,stm32mp25-rcc.h
> @@ -109,7 +109,7 @@
>  /* LOW SPEED MCU CLOCK */
>  #define CK_ICN_LS_MCU		88
>  
> -#define CK_BUS_STM500		89
> +#define CK_BUS_STM		89

This has almost three years, so you are affecting users for no real
gain.

Best regards,
Krzysztof