From: Val Packett <val@packett.cool>
To: Ulf Hansson <ulf.hansson@oss.qualcomm.com>,
Reza Kurniawan <imkyufie@gmail.com>,
Sudeep Holla <sudeep.holla@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Lorenzo Pieralisi <lpieralisi@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>, Ulf Hansson <ulfh@kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Daniel Lezcano <daniel.lezcano@kernel.org>,
Christian Loehle <christian.loehle@arm.com>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
Abel Vesa <abelvesa@kernel.org>,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
linux-arm-msm@vger.kernel.org,
Alejandro Tafalla <atafalla@dnyon.com>
Subject: Re: [PATCH v3 1/4] dt-bindings: arm: psci: Add skip-suspend-mode-switch property
Date: Fri, 13 Nov 2026 10:29:36 -0300 [thread overview]
Message-ID: <f354e6b0-c4ce-44f0-b315-60895e90ec3d@packett.cool> (raw)
In-Reply-To: <CAPx+jO-rMeg8BvemcMUcC10enjt0dkqWvsK+uRwc75rWpLXhMg@mail.gmail.com>
On 11/13/26 10:09 AM, Ulf Hansson wrote:
> + Sudeep
>
> On Thu, Oct 22, 2026 at 11:41 AM Reza Kurniawan <imkyufie@gmail.com> wrote:
>> From: Alejandro Tafalla <atafalla@dnyon.com>
>>
>> On some SoCs (e.g. Qualcomm MSM8917 and MSM8937), PSCI_FEATURES reports
>> that OS-initiated mode is supported, but every PSCI_SET_SUSPEND_MODE
>> call is rejected with NOT_SUPPORTED.
>>
>> PSCI provides no call to read back the current suspend mode, so the OS
>> cannot tell which mode the firmware is operating in or why the switch
>> is rejected.
>>
>> Add an optional flag that tells the OS not to issue
>> PSCI_SET_SUSPEND_MODE on such firmware.
> I think it would be nice to further clarify what's going on in the
> PSCI FW. If I understand correctly, the only supported mode is the
> PSCI OS-initiated mode, hence it's always running in this mode, right?
>
> This is clearly a violation of the PSCI spec - and I guess updating
> the FW isn't possible at this late state, or is it?
>
> [..]
Updating low-level FW in retail devices just doesn't happen, sadly.
Can't be done by the user, IIUC not even if Qualcomm wanted to help out,
because firmware is signed by the OEM and won't load w/o that signature.
And OEMs, well.. we can't even get the vendors for modern high-end
laptops to pull important bug fixes from Qcom.. Thinkpad X13s still
requires that stupid arm64.nopauth workaround, retail Hamoa laptops all
struggle with the DC ZVA crash/reset
(https://github.com/icecream95/x1e-crash) that no one from the Qcom
IoT/Linux world has ever encountered because of different firmware
apparently, and so on..
MSM8917/MSM8937 are found in decade old low-end mobile e-waste, only
could be updated by force if someone finds a bug in firmware secure boot :)
Thanks,
~val
next prev parent reply other threads:[~2026-11-13 13:29 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-22 9:41 [PATCH v3 0/4] Fix PSCI on MSM8917 and MSM8937 Reza Kurniawan
2026-10-22 9:41 ` [PATCH v3 1/4] dt-bindings: arm: psci: Add skip-suspend-mode-switch property Reza Kurniawan
2026-11-06 21:29 ` Rob Herring
2026-11-09 2:44 ` Reza Kurniawan
2026-11-13 13:09 ` Ulf Hansson
2026-11-13 13:29 ` Val Packett [this message]
2026-10-22 9:41 ` [PATCH v3 2/4] cpuidle: psci: Allow skipping the suspend mode switch Reza Kurniawan
2026-11-13 13:14 ` Ulf Hansson
2026-10-22 9:41 ` [PATCH v3 3/4] arm64: dts: qcom: msm8917: Use a shallower cluster state to fix unresponsive idle Reza Kurniawan
2026-10-22 10:14 ` Konrad Dybcio
2026-10-24 0:59 ` Reza Kurniawan
2026-10-22 9:41 ` [PATCH v3 4/4] arm64: dts: qcom: Skip suspend mode switch on MSM8917 and MSM8937 Reza Kurniawan
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=f354e6b0-c4ce-44f0-b315-60895e90ec3d@packett.cool \
--to=val@packett.cool \
--cc=abelvesa@kernel.org \
--cc=andersson@kernel.org \
--cc=atafalla@dnyon.com \
--cc=christian.loehle@arm.com \
--cc=conor+dt@kernel.org \
--cc=daniel.lezcano@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=imkyufie@gmail.com \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=mark.rutland@arm.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=sudeep.holla@kernel.org \
--cc=ulf.hansson@oss.qualcomm.com \
--cc=ulfh@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
all inboxes | Powered by JetHome®