From: Neil Armstrong <neil.armstrong@linaro.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>
Cc: "Leo Yan" <leo.yan@linux.dev>,
"Georgi Djakov" <djakov@kernel.org>,
"Shawn Guo" <shawn.guo@linaro.org>,
"Stephan Gerhold" <stephan@gerhold.net>,
"Zac Crosby" <zac@squareup.com>, "Bastian Köcher" <git@kchr.de>,
"Jeremy McNicoll" <jeremymc@redhat.com>,
"Rohit Agarwal" <quic_rohiagar@quicinc.com>,
"Melody Olvera" <quic_molvera@quicinc.com>,
cros-qcom-dts-watchers@chromium.org,
"Stephen Boyd" <swboyd@chromium.org>,
"Rajendra Nayak" <quic_rjendra@quicinc.com>,
"Martin Botka" <martin.botka@somainline.org>,
"Jonathan Marek" <jonathan@marek.ca>,
"Vinod Koul" <vkoul@kernel.org>,
"Tengfei Fan" <quic_tengfan@quicinc.com>,
"Fenglin Wu" <quic_fenglinw@quicinc.com>,
"Abel Vesa" <abel.vesa@linaro.org>,
"Alexandru Marc Serdeliuc" <serdeliuk@yahoo.com>,
"Vladimir Zapolskiy" <vladimir.zapolskiy@linaro.org>,
"Sibi Sankar" <quic_sibis@quicinc.com>,
"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
"Jun Nie" <jun.nie@linaro.org>, "Max Chen" <mchen@squareup.com>,
"Vincent Knecht" <vincent.knecht@mailoo.org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 16/21] arm64: dts: qcom: sm8550: correct sleep clock frequency
Date: Mon, 30 Dec 2024 10:42:10 +0100 [thread overview]
Message-ID: <a6106504-8b0b-4e0e-b287-7bc08a04b932@linaro.org> (raw)
In-Reply-To: <20241224-fix-board-clocks-v3-16-e9b08fbeadd3@linaro.org>
On 24/12/2024 11:17, Dmitry Baryshkov wrote:
> The SM8550 platform uses PMK8550 to provide sleep clock. According to the
> documentation, that clock has 32.7645 kHz frequency. Correct the sleep
> clock definition.
>
> Fixes: 0b12da4e28d8 ("arm64: dts: qcom: add base AIM300 dtsi")
> Fixes: b5e25ded2721 ("arm64: dts: qcom: sm8550: add support for the SM8550-HDK board")
> Fixes: 71342fb91eae ("arm64: dts: qcom: Add base SM8550 MTP dts")
> Fixes: d228efe88469 ("arm64: dts: qcom: sm8550-qrd: add QRD8550")
> Fixes: ba2c082a401f ("arm64: dts: qcom: sm8550: Add support for Samsung Galaxy Z Fold5")
> Fixes: 39c596304e44 ("arm64: dts: qcom: Add SM8550 Xperia 1 V")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/sm8550-hdk.dts | 2 +-
> arch/arm64/boot/dts/qcom/sm8550-mtp.dts | 2 +-
> arch/arm64/boot/dts/qcom/sm8550-qrd.dts | 2 +-
> arch/arm64/boot/dts/qcom/sm8550-samsung-q5q.dts | 2 +-
> arch/arm64/boot/dts/qcom/sm8550-sony-xperia-yodo-pdx234.dts | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi b/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
> index f6960e2d466a26bec4e5c3d31ac0bf54168ca1e5..e6ac529e6b7216ac4b9e10900c5ddc9a06c9011c 100644
> --- a/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
> +++ b/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
> @@ -367,7 +367,7 @@ &pm8550b_eusb2_repeater {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &ufs_mem_hc {
> diff --git a/arch/arm64/boot/dts/qcom/sm8550-hdk.dts b/arch/arm64/boot/dts/qcom/sm8550-hdk.dts
> index 01c921602605726928bcba71abd937869880ea0d..29bc1ddfc7b25f203c9f3b530610e45c44ae4fb2 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8550-hdk.dts
> @@ -1172,7 +1172,7 @@ &sdhc_2 {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &swr0 {
> diff --git a/arch/arm64/boot/dts/qcom/sm8550-mtp.dts b/arch/arm64/boot/dts/qcom/sm8550-mtp.dts
> index ab447fc252f7dd705fbe29725805bc1c146c10f9..5648ab60ba4c4bfaf5baa289969898277ee57cef 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8550-mtp.dts
> @@ -825,7 +825,7 @@ &sdhc_2 {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &swr0 {
> diff --git a/arch/arm64/boot/dts/qcom/sm8550-qrd.dts b/arch/arm64/boot/dts/qcom/sm8550-qrd.dts
> index 6052dd922ec55c65089e6b08e36c355b01bf24fa..3a6cb279130489168f8d20a6e27808647debdb41 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550-qrd.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8550-qrd.dts
> @@ -1005,7 +1005,7 @@ &remoteproc_mpss {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &swr0 {
> diff --git a/arch/arm64/boot/dts/qcom/sm8550-samsung-q5q.dts b/arch/arm64/boot/dts/qcom/sm8550-samsung-q5q.dts
> index 3c5d8d26704fd925857b5ddcc6067c2b14e1403d..e8383faac576a2f401fff74231a764712c832291 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550-samsung-q5q.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8550-samsung-q5q.dts
> @@ -565,7 +565,7 @@ &remoteproc_mpss {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &tlmm {
> diff --git a/arch/arm64/boot/dts/qcom/sm8550-sony-xperia-yodo-pdx234.dts b/arch/arm64/boot/dts/qcom/sm8550-sony-xperia-yodo-pdx234.dts
> index 85d487ef80a0be5c3586d9de69327abd2b64209b..d90dc7b37c4a74cbfb03c929646fda3381413084 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550-sony-xperia-yodo-pdx234.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8550-sony-xperia-yodo-pdx234.dts
> @@ -722,7 +722,7 @@ &sdhc_2 {
> };
>
> &sleep_clk {
> - clock-frequency = <32000>;
> + clock-frequency = <32764>;
> };
>
> &tlmm {
>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
next prev parent reply other threads:[~2024-12-30 9:42 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-24 10:16 [PATCH v3 00/21] arm64: dts: qcom: correct sleep clocks Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 01/21] arm64: dts: qcom: msm8916: correct sleep clock frequency Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 02/21] arm64: dts: qcom: msm8939: " Dmitry Baryshkov
2024-12-30 9:49 ` Bryan O'Donoghue
2024-12-30 14:10 ` Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 03/21] arm64: dts: qcom: msm8994: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 04/21] arm64: dts: qcom: qcs404: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 05/21] arm64: dts: qcom: q[dr]u1000: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 06/21] arm64: dts: qcom: qrb4210-rb2: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 07/21] arm64: dts: qcom: sar2130p: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 08/21] arm64: dts: qcom: sc7280: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 09/21] arm64: dts: qcom: sdx75: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 10/21] arm64: dts: qcom: sm4450: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 11/21] arm64: dts: qcom: sm6125: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 12/21] arm64: dts: qcom: sm6375: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 13/21] arm64: dts: qcom: sm8250: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 14/21] arm64: dts: qcom: sm8350: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 15/21] arm64: dts: qcom: sm8450: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 16/21] arm64: dts: qcom: sm8550: " Dmitry Baryshkov
2024-12-30 9:42 ` Neil Armstrong [this message]
2024-12-24 10:17 ` [PATCH v3 17/21] arm64: dts: qcom: sm8650: " Dmitry Baryshkov
2024-12-30 9:42 ` Neil Armstrong
2024-12-24 10:17 ` [PATCH v3 18/21] arm64: dts: qcom: x1e80100: " Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 19/21] arm64: dts: qcom: sc8180x: drop extra XO clock frequencies Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 20/21] arm64: dts: qcom: sdm670: move board clocks to sdm670.dtsi file Dmitry Baryshkov
2024-12-24 10:17 ` [PATCH v3 21/21] arm64: dts: qcom: q[dr]u1000: move board clocks to qdu1000.dtsi file Dmitry Baryshkov
2024-12-27 4:15 ` [PATCH v3 00/21] arm64: dts: qcom: correct sleep clocks 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=a6106504-8b0b-4e0e-b287-7bc08a04b932@linaro.org \
--to=neil.armstrong@linaro.org \
--cc=abel.vesa@linaro.org \
--cc=andersson@kernel.org \
--cc=bryan.odonoghue@linaro.org \
--cc=conor+dt@kernel.org \
--cc=cros-qcom-dts-watchers@chromium.org \
--cc=devicetree@vger.kernel.org \
--cc=djakov@kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=git@kchr.de \
--cc=jeremymc@redhat.com \
--cc=jonathan@marek.ca \
--cc=jun.nie@linaro.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=leo.yan@linux.dev \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.botka@somainline.org \
--cc=mchen@squareup.com \
--cc=quic_fenglinw@quicinc.com \
--cc=quic_molvera@quicinc.com \
--cc=quic_rjendra@quicinc.com \
--cc=quic_rohiagar@quicinc.com \
--cc=quic_sibis@quicinc.com \
--cc=quic_tengfan@quicinc.com \
--cc=robh@kernel.org \
--cc=serdeliuk@yahoo.com \
--cc=shawn.guo@linaro.org \
--cc=stephan@gerhold.net \
--cc=swboyd@chromium.org \
--cc=vincent.knecht@mailoo.org \
--cc=vkoul@kernel.org \
--cc=vladimir.zapolskiy@linaro.org \
--cc=zac@squareup.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®