From: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
To: Sudeep Holla <sudeep.holla@arm.com>,
Cristian Marussi <cristian.marussi@arm.com>
Cc: arm-scmi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Ranjani Vaidyanathan <ranjani.vaidyanathan@nxp.com>,
Chuck Cannon <chuck.cannon@nxp.com>, Peng Fan <peng.fan@nxp.com>
Subject: [PATCH 0/2] firmware: arm_scmi: add pm ops for scmi_power_control
Date: Fri, 20 Jun 2025 11:37:12 +0800 [thread overview]
Message-ID: <20250620-scmi-pm-v1-0-c2f02cae5122@nxp.com> (raw)
When testing on i.MX95, two consecutive suspend message send to the Linux
agent, Linux will suspend(by the 1st suspend message) and wake up(by the
2nd suspend message).
The ARM SCMI spec does not allow for filtering of which messages an agent
wants to get on the system power protocol. To i.MX95, as we use mailbox
to receive message, and the mailbox supports wake up, so linux will also
get a repeated suspend message. This will cause Linux to wake (and should
then go back into suspend).
This patchset is to make the 2nd suspend message could suspend linux
again.
So why SCMI fireware couldn't block the 2nd suspend message from being
sent to Linux agent? Per checking with our SCMI firmware owner:
The SM(System Manager) does not know exactly when Linux is in suspend.
There are no handshakes that clearly tell the SM this. The flow should
be, if in suspend and you send a suspend (or graceful reset/power off)
it will wake and then do the request action
Move information could be found in patch 2 commit log
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
Peng Fan (2):
firmware: arm_scmi: bus: Add pm ops
firmware: arm_scmi: power_control: Set SCMI_SYSPOWER_IDLE in pm resume
drivers/firmware/arm_scmi/bus.c | 45 ++++++++++++++++++++++++++
drivers/firmware/arm_scmi/scmi_power_control.c | 24 +++++++++++---
2 files changed, 64 insertions(+), 5 deletions(-)
---
base-commit: 4325743c7e209ae7845293679a4de94b969f2bef
change-id: 20250620-scmi-pm-8f6170769230
Best regards,
--
Peng Fan <peng.fan@nxp.com>
next reply other threads:[~2025-06-20 3:39 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-20 3:37 Peng Fan (OSS) [this message]
2025-06-20 3:37 ` [PATCH 1/2] firmware: arm_scmi: bus: Add pm ops Peng Fan (OSS)
2025-06-20 3:55 ` Dan Carpenter
2025-06-20 5:21 ` Peng Fan
2025-06-20 3:37 ` [PATCH 2/2] firmware: arm_scmi: power_control: Set SCMI_SYSPOWER_IDLE in pm resume Peng Fan (OSS)
2025-06-20 17:40 ` kernel test robot
2025-06-23 12:57 ` Dhruva Gole
2025-06-23 14:29 ` Peng Fan
2025-06-23 15:04 ` Cristian Marussi
2025-06-23 16:27 ` Sudeep Holla
2025-06-24 1:23 ` Peng Fan
2025-06-24 10:21 ` Sudeep Holla
2025-06-24 14:58 ` Peng Fan
2025-07-01 15:07 ` Peng Fan
2025-07-02 15:26 ` Sudeep Holla
2025-06-23 14:48 ` [PATCH 0/2] firmware: arm_scmi: add pm ops for scmi_power_control Cristian Marussi
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=20250620-scmi-pm-v1-0-c2f02cae5122@nxp.com \
--to=peng.fan@oss.nxp.com \
--cc=arm-scmi@vger.kernel.org \
--cc=chuck.cannon@nxp.com \
--cc=cristian.marussi@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=peng.fan@nxp.com \
--cc=ranjani.vaidyanathan@nxp.com \
--cc=sudeep.holla@arm.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®