All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Robert Marko <robimarko@gmail.com>
Cc: agross@kernel.org, bjorn.andersson@linaro.org,
	lgirdwood@gmail.com, broonie@kernel.org,
	krzysztof.kozlowski+dt@linaro.org, linux-arm-msm@vger.kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	konrad.dybcio@somainline.org,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Subject: Re: [PATCH v5 1/7] dt-bindings: regulator: qcom,spmi-regulator: Convert to dtschema
Date: Mon, 23 May 2022 21:22:43 -0500	[thread overview]
Message-ID: <20220524022243.GA2625732-robh@kernel.org> (raw)
In-Reply-To: <20220519133015.484639-1-robimarko@gmail.com>

On Thu, May 19, 2022 at 03:30:09PM +0200, Robert Marko wrote:
> Convert the bindings of Qualcomm SPMI regulators to DT schema.
> 
> Signed-off-by: Robert Marko <robimarko@gmail.com>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> Changes in v4:
> * Remove blank interrupts and interrupt-names as generic properties
> to only allow them per compatible
> 
> Changes in v3:
> * Remove quotes around refs
> * Use much stricter regex for regulator node matching
> * Add supply matching per compatible
> * Add blank interrupts and interrupt-names as generic properties
> 
> Changes in v2:
> * Remove the forgotten text bindings
> * Move allOf after patternProperties
> * Use my private email as the maintainer email
> 
> I am aware that syscon alone is not really acceptable, its converted
> directly from the old text bindings.
> 
> Any advice on how to solve this is appreciated.

Add a compatible for the block in question, but that means you have to 
have a schema for it. Or you can just drop the node. This binding isn't 
really about that node.

Rob

  parent reply	other threads:[~2022-05-24  2:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-19 13:30 [PATCH v5 1/7] dt-bindings: regulator: qcom,spmi-regulator: Convert to dtschema Robert Marko
2022-05-19 13:30 ` [PATCH v5 2/7] regulator: qcom_spmi: add support for HT_P150 Robert Marko
2022-05-19 13:30 ` [PATCH v5 3/7] regulator: qcom_spmi: add support for HT_P600 Robert Marko
2022-05-19 13:30 ` [PATCH v5 4/7] dt-bindings: regulator: qcom,spmi-regulator: add PMP8074 PMIC Robert Marko
2022-05-19 13:30 ` [PATCH v5 5/7] regulator: qcom_spmi: add support for PMP8074 regulators Robert Marko
2022-05-19 13:30 ` [PATCH v5 6/7] arm64: dts: qcom: add PMP8074 DTSI Robert Marko
2022-05-19 13:30 ` [PATCH v5 7/7] arm64: dts: qcom: ipq8074-hk01: add VQMMC supply Robert Marko
2022-05-24  2:22 ` Rob Herring [this message]
2022-05-30 17:09   ` [PATCH v5 1/7] dt-bindings: regulator: qcom,spmi-regulator: Convert to dtschema Robert Marko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220524022243.GA2625732-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=agross@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konrad.dybcio@somainline.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robimarko@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.