From: Stephan Gerhold <stephan@gerhold.net>
To: Bjorn Andersson <andersson@kernel.org>
Cc: Andy Gross <agross@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org,
~postmarketos/upstreaming@lists.sr.ht,
Stephan Gerhold <stephan@gerhold.net>
Subject: [PATCH 0/8] arm64: dts: qcom: msm8916: Rework regulator constraints
Date: Wed, 17 May 2023 20:48:39 +0200 [thread overview]
Message-ID: <20230510-msm8916-regulators-v1-0-54d4960a05fc@gerhold.net> (raw)
Rework the regulator constraints for the MSM8916 device trees to be
closer to reality. There are several mistakes in there, some of them
taken over directly from Qualcomm's vendor kernel. Fortunately, none of
the mistakes is absolutely critical because it turns out that the RPM
firmware also validates the voltages and silently clamps the requests
to a proper range. Still, this behavior should be clearly represented
in the device tree rather than pretending to apply the wrong voltages.
To make the regulator constraints more easily maintainable with a large
number of similar MSM8916 boards I propose moving the voltages for the
standard components in the SoC to the shared msm8916-pm8916.dtsi
include. With this only the actual board-specific regulators are
described in the board DT.
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
---
Stephan Gerhold (8):
arm64: dts: qcom: apq8016-sbc: Fix regulator constraints
arm64: dts: qcom: apq8016-sbc: Fix 1.8V power rail on LS expansion
arm64: dts: qcom: msm8916: Fix regulator constraints
arm64: dts: qcom: msm8916: Disable audio codecs by default
arm64: dts: qcom: pm8916: Move default regulator "-supply"s
arm64: dts: qcom: msm8916-pm8916: Clarify purpose
arm64: dts: qcom: msm8916: Define regulator constraints next to usage
arm64: dts: qcom: msm8916-pm8916: Mark always-on regulators
arch/arm64/boot/dts/qcom/apq8016-sbc.dts | 145 +++++----------------
arch/arm64/boot/dts/qcom/msm8916-acer-a1-724.dts | 115 ++--------------
.../boot/dts/qcom/msm8916-alcatel-idol347.dts | 110 +---------------
arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts | 110 +---------------
arch/arm64/boot/dts/qcom/msm8916-gplus-fl8005a.dts | 110 +---------------
arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts | 125 ++++--------------
.../boot/dts/qcom/msm8916-longcheer-l8150.dts | 110 +---------------
.../boot/dts/qcom/msm8916-longcheer-l8910.dts | 110 +---------------
arch/arm64/boot/dts/qcom/msm8916-pm8916.dtsi | 121 ++++++++++++++---
.../dts/qcom/msm8916-samsung-a2015-common.dtsi | 110 +---------------
.../boot/dts/qcom/msm8916-samsung-gt5-common.dtsi | 110 +---------------
.../boot/dts/qcom/msm8916-samsung-j5-common.dtsi | 103 ---------------
.../boot/dts/qcom/msm8916-samsung-serranove.dts | 103 ---------------
arch/arm64/boot/dts/qcom/msm8916-ufi.dtsi | 103 ---------------
.../boot/dts/qcom/msm8916-wingtech-wt88047.dts | 119 +++--------------
arch/arm64/boot/dts/qcom/msm8916.dtsi | 1 +
arch/arm64/boot/dts/qcom/pm8916.dtsi | 4 +-
17 files changed, 236 insertions(+), 1473 deletions(-)
---
base-commit: 4272e06e19f388ccfe1f04f19060ea84d2a19a8b
change-id: 20230510-msm8916-regulators-97fa33735efe
Best regards,
--
Stephan Gerhold <stephan@gerhold.net>
next reply other threads:[~2023-05-17 18:49 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-17 18:48 Stephan Gerhold [this message]
2023-05-17 18:48 ` [PATCH 1/8] arm64: dts: qcom: apq8016-sbc: Fix regulator constraints Stephan Gerhold
2023-05-17 18:48 ` [PATCH 2/8] arm64: dts: qcom: apq8016-sbc: Fix 1.8V power rail on LS expansion Stephan Gerhold
2023-05-17 18:48 ` [PATCH 3/8] arm64: dts: qcom: msm8916: Fix regulator constraints Stephan Gerhold
2023-05-26 13:38 ` Bryan O'Donoghue
2023-05-26 14:03 ` Stephan Gerhold
2023-05-26 15:42 ` Bryan O'Donoghue
2023-05-26 15:43 ` Bryan O'Donoghue
2023-05-17 18:48 ` [PATCH 4/8] arm64: dts: qcom: msm8916: Disable audio codecs by default Stephan Gerhold
2023-05-17 18:48 ` [PATCH 5/8] arm64: dts: qcom: pm8916: Move default regulator "-supply"s Stephan Gerhold
2023-05-17 18:48 ` [PATCH 6/8] arm64: dts: qcom: msm8916-pm8916: Clarify purpose Stephan Gerhold
2023-05-17 18:48 ` [PATCH 7/8] arm64: dts: qcom: msm8916: Define regulator constraints next to usage Stephan Gerhold
2023-05-25 23:35 ` Konrad Dybcio
2023-05-26 6:47 ` Stephan Gerhold
2023-05-26 21:11 ` Konrad Dybcio
2023-05-27 9:22 ` Stephan Gerhold
2023-05-17 18:48 ` [PATCH 8/8] arm64: dts: qcom: msm8916-pm8916: Mark always-on regulators Stephan Gerhold
2023-05-25 23:39 ` Konrad Dybcio
2023-05-26 0:28 ` Konrad Dybcio
2023-05-26 6:36 ` Stephan Gerhold
2023-05-26 8:50 ` Konrad Dybcio
2023-05-26 12:55 ` Stephan Gerhold
2023-05-25 4:54 ` [PATCH 0/8] arm64: dts: qcom: msm8916: Rework regulator constraints Bjorn Andersson
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=20230510-msm8916-regulators-v1-0-54d4960a05fc@gerhold.net \
--to=stephan@gerhold.net \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=phone-devel@vger.kernel.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=~postmarketos/upstreaming@lists.sr.ht \
/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).