Re: [PATCH v3] dt-bindings: mfd: convert to yaml Qualcomm SPMI PMIC

From: Lee Jones <lee@kernel.org>
To: Rob Herring <robh@kernel.org>
Cc: Lee Jones <lee.jones@linaro.org>,
	David Heidelberg <david@ixit.cz>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Andy Gross <agross@kernel.org>,
	Caleb Connolly <caleb@connolly.tech>,
	linux-arm-msm <linux-arm-msm@vger.kernel.org>,
	Stephen Boyd <sboyd@kernel.org>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	~okias/devicetree@lists.sr.ht,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	devicetree@vger.kernel.org
Subject: Re: [PATCH v3] dt-bindings: mfd: convert to yaml Qualcomm SPMI PMIC
Date: Wed, 10 Aug 2022 16:23:43 +0100	[thread overview]
Message-ID: <YvPNf9qL+zdXEyK+@google.com> (raw)
In-Reply-To: <CAL_JsqL54yLFApJHZodEPUc4MucFg=0qMXr=koiprJB8z4dY-w@mail.gmail.com>

On Wed, 10 Aug 2022, Rob Herring wrote:

> On Wed, Aug 10, 2022 at 4:57 AM Lee Jones <lee.jones@linaro.org> wrote:
> >
> > On Tue, 09 Aug 2022, Rob Herring wrote:
> >
> > > On Sun, 26 Jun 2022 21:16:30 +0200, David Heidelberg wrote:
> > > > Convert Qualcomm SPMI PMIC binding to yaml format.
> > > >
> > > > Additional changes:
> > > >  - filled many missing compatibles
> > > >
> > > > Co-developed-by: Caleb Connolly <caleb@connolly.tech>
> > > > Signed-off-by: David Heidelberg <david@ixit.cz>
> > > > ---
> > > > v3:
> > > >  - added subnodes, there are two not converted to YAML yet, but it works
> > > >  - now it prints milion directly unrelated warning to this binding
> > > >    (it's related to the included subnodes bindings, can be merged,
> > > >     but it'll generate more warnings and preferably anyone can takeover
> > > >     from here)
> > > >  - add qcom,pmx65
> > > >
> > > > v2:
> > > >  - changed author to myself, kept Caleb as co-author
> > > >  - moved nodename to properties
> > > >  - add nodenames for pm* with deprecated property
> > > >  - add ^$ to pattern properties
> > > >  - dropped interrupt-names property
> > > >  - added reg prop. to the nodes which have register in nodename
> > > >  - added compatible pmx55
> > > > Signed-off-by: David Heidelberg <david@ixit.cz>
> > > > ---
> > > >  .../bindings/mfd/qcom,spmi-pmic.txt           |  94 ---------
> > > >  .../bindings/mfd/qcom,spmi-pmic.yaml          | 191 ++++++++++++++++++
> > > >  2 files changed, 191 insertions(+), 94 deletions(-)
> > > >  delete mode 100644 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.txt
> > > >  create mode 100644 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml
> > > >
> > >
> > > As this hasn't been picked up, I applied it. Thanks!
> >
> > I'd prefer to take it via MFD if it's all the same to you.
> >
> > In case there are additional fix-ups required during the next cycle.
> 
> I'm going to send it to Linus this week for rc1 so that won't be an issue.

Works for me then, thanks.

Acked-by: Lee Jones <lee@kernel.org>

-- 
Lee Jones [李琼斯]