From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Alexander Koskovich <AKoskovich@pm.me>,
Krzysztof Kozlowski <krzk@kernel.org>
Cc: 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>,
Dmitry Baryshkov <lumag@kernel.org>,
Thara Gopinath <thara.gopinath@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: sm8250: Add missing CPU7 3.09GHz OPP
Date: Mon, 9 Mar 2026 12:03:00 +0100 [thread overview]
Message-ID: <55250bff-c46c-4bf7-9a94-27db4f86b037@oss.qualcomm.com> (raw)
In-Reply-To: <N51k53pJOUTOpQrfJsuk9rZdo_VxhPtCdG1Aj-8uqmoS1Bl7q7FUarMhq5b5sZwAzJrH6SvH5n1KeK7T2mSn3ssMThLiEdmKNfccXata1-U=@pm.me>
On 3/8/26 7:03 PM, Alexander Koskovich wrote:
> On Sunday, March 8th, 2026 at 4:48 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
>> git grep "ASUS ROG Phone" gives me zero results so this is insufficient
>> argument considering you are changing multiple other devices which might
>> not support that.
>
> Also I'm not 100% sure that this applies to all SM8250 devices, should I just fold this into the ROG 3 DTS submission later and put it in the board specific DTS? As I can't validate it on any other SM8250.
The qcom-cpufreq-hw driver reads out the LUT that has been programmed
into the device and only considers OPPs that match those entries, so no
worries here.
This is the case for all Qualcomm SoCs starting with SDM845.
For reference, SDM630/MSM8998 use the same hardware, but need
programming from the HLOS which is notoriously long and annoying, there
you can actually make mistakes that result in instability.
Konrad
next prev parent reply other threads:[~2026-03-09 11:03 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-08 4:26 [PATCH] arm64: dts: qcom: sm8250: Add missing CPU7 3.09GHz OPP Alexander Koskovich
2026-03-08 8:48 ` Krzysztof Kozlowski
2026-03-08 18:03 ` Alexander Koskovich
2026-03-09 11:03 ` Konrad Dybcio [this message]
2026-03-11 13:13 ` Konrad Dybcio
2026-03-30 16:01 ` 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=55250bff-c46c-4bf7-9a94-27db4f86b037@oss.qualcomm.com \
--to=konrad.dybcio@oss.qualcomm.com \
--cc=AKoskovich@pm.me \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lumag@kernel.org \
--cc=robh@kernel.org \
--cc=thara.gopinath@linaro.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