From: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
To: agross@kernel.org, andersson@kernel.org,
konrad.dybcio@somainline.org, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, lee@kernel.org,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org
Cc: bryan.odonoghue@linaro.org
Subject: [PATCH v6 0/1] arm64: dts: qcom: pwm: Drop PWM reg dependency + update MAINTAINERS
Date: Wed, 28 Sep 2022 01:05:16 +0100 [thread overview]
Message-ID: <20220928000517.228382-1-bryan.odonoghue@linaro.org> (raw)
V6:
- Drops the change to MAINTAINERS
Reading Documentation/process/submitting-patches.rst the declaration of
who needs to be mailed and what "Maintained" and "Supported" mean don't
match up at all.
In any case that's out of scope for this simple yaml change so I'll send
along a proposed patch separately to this.
V5:
- Resend the yaml fix
- Update MAINTAINERS to mark Lee as Maintainer not Supporter of the YAML
files for MFD.
I rely on scripts/get_mainter.pl to tell me who needs to be mailed and
who doesn't, given the YAML change has to go through Lee, the MAINTAINERS
file should make that show up in get_maintainers.pl
V4:
- Churn patch#1 prefix to "dt-bindings: mfd: qcom,spmi-pmic:" - Krzysztof
- Add's Krzysztof's RB as indicated
V3:
- Splits dtsi and yaml
- Uses Krzysztof's suggested commit log in the yaml
V2:
The accompanying patch removes reg = <> and pwm@reg from the yaml and dtsi.
This follows on from discussions between Bupesh, Dmitry, Bjorn, Krzysztof and myself.
https://lore.kernel.org/all/20220719205058.1004942-1-bhupesh.sharma@linaro.org/
https://lore.kernel.org/all/20220721195502.1525214-1-bhupesh.sharma@linaro.org/
https://lore.kernel.org/all/20220822120300.2633790-1-bryan.odonoghue@linaro.org/
The previous discussion tended towards either removing pwm@reg and reg = <> or
extending out the yaml to support multiple reg declarations for PWM compatible.
This patch does the former. I've left node: label in place, dropped both pwm@reg
and reg = <> I kept "label: nodename" though because it looked more like what we
already have for rpm regulators.
Per our previous discussion I've modified the yaml and dtsi in one go.
Bryan O'Donoghue (1):
dt-bindings: mfd: qcom,spmi-pmic: Drop PWM reg dependency
Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
--
2.37.3
next reply other threads:[~2022-09-28 0:05 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-28 0:05 Bryan O'Donoghue [this message]
2022-09-28 0:05 ` [PATCH v6 1/1] dt-bindings: mfd: qcom,spmi-pmic: Drop PWM reg dependency Bryan O'Donoghue
2022-10-24 12:31 ` Lee Jones
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=20220928000517.228382-1-bryan.odonoghue@linaro.org \
--to=bryan.odonoghue@linaro.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@somainline.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lee@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=robh+dt@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).