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
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date:   Mon, 12 Sep 2022 08:50:21 +0200
Message-Id: <CMU81K51YF5E.4KH2T53ZEIKW@otso>
Cc:     <linux-arm-msm@vger.kernel.org>,
        <~postmarketos/upstreaming@lists.sr.ht>,
        <phone-devel@vger.kernel.org>, "Andy Gross" <agross@kernel.org>,
        "Bjorn Andersson" <bjorn.andersson@linaro.org>,
        "Thomas Gleixner" <tglx@linutronix.de>,
        "Marc Zyngier" <maz@kernel.org>, <linux-kernel@vger.kernel.org>,
        <devicetree@vger.kernel.org>
Subject: Re: [PATCH v2] dt-bindings: qcom,pdc: convert to YAML
From:   "Luca Weiss" <luca.weiss@fairphone.com>
To:     "Rob Herring" <robh@kernel.org>
References: <20220103074348.6039-1-luca.weiss@fairphone.com>
 <20220909143950.GA992904-robh@kernel.org>
In-Reply-To: <20220909143950.GA992904-robh@kernel.org>
Precedence: bulk

Hi Rob,

On Fri Sep 9, 2022 at 4:39 PM CEST, Rob Herring wrote:
> On Mon, Jan 03, 2022 at 08:43:48AM +0100, Luca Weiss wrote:
> > Convert the PDC interrupt controller bindings to YAML.
> >=20
> > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > ---
> > Changes since v1:
> > * Adjust description of second reg-name as suggested by Maulik Shah
> >=20
> > @Rob Herring: Hope it's ok to keep your R-b given the above changes
> >=20
> > This patch depends on the following patch, which fixed sm8250 & sm8350
> > compatibles and adds sm6350.
> > https://lore.kernel.org/linux-arm-msm/20211213082614.22651-4-luca.weiss=
@fairphone.com/
> >=20
> >  .../interrupt-controller/qcom,pdc.txt         | 77 -----------------
> >  .../interrupt-controller/qcom,pdc.yaml        | 86 +++++++++++++++++++
> >  2 files changed, 86 insertions(+), 77 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/interrupt-control=
ler/qcom,pdc.txt
> >  create mode 100644 Documentation/devicetree/bindings/interrupt-control=
ler/qcom,pdc.yaml
>
> In checking top compatibles without schemas[1][2], I found this. Now=20
> applied with sm8150 compatible which was the only change since this.

Thanks for digging this out, I have totally forgotten about it!

Regards
Luca

>
> Rob
>
> [1] https://gitlab.com/robherring/linux-dt/-/jobs/3005191129
> [2] https://gitlab.com/robherring/linux-dt/-/jobs/3005191129/artifacts/fi=
le/all-compatible-warnings.log