]> www.infradead.org Git - users/jedix/linux-maple.git/commit
pmdomain: arm: scmi_pm_domain: Remove redundant state verification
authorSudeep Holla <sudeep.holla@arm.com>
Fri, 14 Mar 2025 09:58:51 +0000 (09:58 +0000)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 17 Mar 2025 10:12:01 +0000 (11:12 +0100)
commit6cf5db71dc6d51ede4ff77cab126876ecf065048
tree0be6a7adaa0ed9544104e996705160863767477a
parent6ec7c4a297baea6be43d0b09d19bde8bbc1fec15
pmdomain: arm: scmi_pm_domain: Remove redundant state verification

Currently, scmi_pd_power() explicitly verifies whether the requested
power state was applied by calling state_get(). While this check could
detect failures where the state was not properly updated, ensuring
correctness is the responsibility of the SCMI firmware.

Removing this redundant state_get() call eliminates an unnecessary
round-trip to the firmware, improving efficiency. Any mismatches
between the requested and actual states should be handled by the SCMI
firmware, which must return a failure if state_set() is unsuccessful.

Additionally, in some cases, checking the state after powering off a
domain may be unreliable or unsafe, depending on the firmware
implementation.

This patch removes the redundant verification, simplifying the function
without compromising correctness.

Reported-and-tested-by: Ranjani Vaidyanathan <ranjani.vaidyanathan@nxp.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>
Link: https://lore.kernel.org/r/20250314095851.443979-1-sudeep.holla@arm.com
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/pmdomain/arm/scmi_pm_domain.c