Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230630185602.2175559-1-robh@kernel.org> <0aa7227d-264f-a87d-bef1-9f5d1c43a184@arm.com>
In-Reply-To: <0aa7227d-264f-a87d-bef1-9f5d1c43a184@arm.com>
From:   Rob Herring <robh@kernel.org>
Date:   Wed, 5 Jul 2023 15:40:38 -0600
Message-ID: <CAL_Jsq+T_8JNAQ1Ca8kNH0LCkiytJuSZPZeC_pvyOOG6xvQh+g@mail.gmail.com>
Subject: Re: [PATCH 1/2] dt-bindings: arm: pmu: Add Cortex-A520, Cortex-A715,
 and Cortex-A720
To:     Robin Murphy <robin.murphy@arm.com>
Cc:     Will Deacon <will@kernel.org>, Mark Rutland <mark.rutland@arm.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, Jul 3, 2023 at 1:40=E2=80=AFPM Robin Murphy <robin.murphy@arm.com> =
wrote:
>
> On 2023-06-30 19:56, Rob Herring wrote:
> > Add compatible strings for the Arm Cortex-A520, Cortex-A715, and
> > Cortex-A720 CPU PMUs.
> >
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> >   Documentation/devicetree/bindings/arm/pmu.yaml | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/pmu.yaml b/Documenta=
tion/devicetree/bindings/arm/pmu.yaml
> > index e14358bf0b9c..0cc468d6c372 100644
> > --- a/Documentation/devicetree/bindings/arm/pmu.yaml
> > +++ b/Documentation/devicetree/bindings/arm/pmu.yaml
> > @@ -49,7 +49,10 @@ properties:
> >             - arm,cortex-a77-pmu
> >             - arm,cortex-a78-pmu
> >             - arm,cortex-a510-pmu
> > +          - arm,cortex-a520-pmu
> >             - arm,cortex-a710-pmu
> > +          - arm,cortex-a715-pmu
> > +          - arm,cortex-a720-pmu
> >             - arm,cortex-x1-pmu
> >             - arm,cortex-x2-pmu
>
> Should we catch up on Cortex-X (3 and 4 now) while we're at it?

Yes. I'll add those.

Looking a bit further at this, it seems a bunch of platforms
(including a Cortex-X3 based one) are just using "arm,armv8-pmuv3".
That's supposed to be just for s/w models, but it's not enforceable.

Rob