Re: [PATCH v3 5/5] ARM: dts: stm32: add ddrperfm on stm32mp157c

[Alexandre Torgue]

Hi Gerald

On 8/27/19 5:08 PM, Gerald BAEZA wrote:
> The DDRPERFM is the DDR Performance Monitor embedded
> in STM32MP1 SOC.
>
> Signed-off-by: Gerald Baeza <gerald.baeza@xxxxxx>
> ---
>  Âarch/arm/boot/dts/stm32mp157c.dtsi | 8 ++++++++
>  Â1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi 
> b/arch/arm/boot/dts/stm32mp157c.dtsi
> index 0c4e6eb..6ea6933 100644
> --- a/arch/arm/boot/dts/stm32mp157c.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157c.dtsi
> @@ -1378,6 +1378,14 @@
>  ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ };
>  ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ };
>
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ ddrperfm: perf@5a007000 {
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ compatible = "st,stm32-ddr-pmu";
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ reg = <0x5a007000 0x400>;
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ clocks = <&rcc DDRPERFM>;
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ resets = <&rcc DDRPERFM_R>;
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ status = "okay";

No need to add "status = "okay"" here.

regards
Alex

> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ };
> +
>  ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ usart1: serial@5c000000 {
>  ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ compatible = "st,stm32h7-uart";
>  ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ reg = <0x5c000000 0x400>;
> --
> 2.7.4