Re: [PATCH v6 1/2] dt-bindings: perf: marvell: Extend CN10K DDR PMU binding for CN20K

[Krzysztof Kozlowski]

On Tue, May 26, 2026 at 10:13:29PM +0530, Geetha sowjanya wrote:
> Marvell CN20K SoCs integrate a DDR Performance Monitoring Unit (PMU)
> associated with the DDR controller. The block provides hardware counters
> to monitor DDR traffic and performance events and is accessed via a
> dedicated MMIO region.
> 
> The CN20K DDR PMU is functionally equivalent to the CN10K DDR PMU, with
> minor register offset differences.
> 
> Changes in v6:
> - dt-bindings: Document CN20K in the existing marvell-cn10k-ddr.yaml;
>   add maintainer, description, compatible enum entry, and a CN20K example
>   with unit-address aligned to reg.

Changelog is not part of commit msg, but changelog area, so after ---.

> 
> Signed-off-by: Geetha sowjanya <gakula@xxxxxxxxxxx>
> ---
>  .../bindings/perf/marvell-cn10k-ddr.yaml       | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/perf/marvell-cn10k-ddr.yaml b/Documentation/devicetree/bindings/perf/marvell-cn10k-ddr.yaml
> index a18dd0a8c43a..79fae9fdb6f1 100644
> --- a/Documentation/devicetree/bindings/perf/marvell-cn10k-ddr.yaml
> +++ b/Documentation/devicetree/bindings/perf/marvell-cn10k-ddr.yaml
> @@ -4,16 +4,22 @@
>  $id: http://devicetree.org/schemas/perf/marvell-cn10k-ddr.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: Marvell CN10K DDR performance monitor
> +title: Marvell CN10K / CN20K DDR performance monitor
> +
> +description:
> +  Performance Monitoring Unit (PMU) for the DDR controller on Marvell
> +  CN10K and CN20K SoCs. The block is accessed via a dedicated MMIO region.
>  
>  maintainers:
>    - Bharat Bhushan <bbhushan2@xxxxxxxxxxx>
> +  - Geetha sowjanya <gakula@xxxxxxxxxxx>
>  
>  properties:
>    compatible:
>      items:
>        - enum:
>            - marvell,cn10k-ddr-pmu
> +          - marvell,cn20k-ddr-pmu
>  
>    reg:
>      maxItems: 1
> @@ -35,3 +41,13 @@ examples:
>              reg = <0x87e1 0xc0000000 0x0 0x10000>;
>          };
>      };
> +  - |
> +    bus {
> +        #address-cells = <2>;
> +        #size-cells = <2>;
> +
> +        pmu@c20000000000 {
> +            compatible = "marvell,cn20k-ddr-pmu";
> +            reg = <0xc200 0x00000000 0x0 0x100000>;

No need for a new example.

Best regards,
Krzysztof