Skip to content

Commit 7cfa380

Browse files
author
Ulf Hansson
committed
cpuidle: psci: Opt-out from genpd's common ->sync_state() support
The cpuidle-psci-domain implements its own specific ->sync_state() callback. Let's set the GENPD_FLAG_NO_SYNC_STATE to inform genpd about it. Moreover, let's call of_genpd_sync_state() to make sure genpd tries to power off unused PM domains. Tested-by: Hiago De Franco <hiago.franco@toradex.com> # Colibri iMX8X Tested-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> # TI AM62A,Xilinx ZynqMP ZCU106 Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Link: https://lore.kernel.org/r/20250701114733.636510-11-ulf.hansson@linaro.org
1 parent 8efc9b1 commit 7cfa380

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

drivers/cpuidle/cpuidle-psci-domain.c

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,8 @@ static int psci_pd_init(struct device_node *np, bool use_osi)
6363
if (!pd_provider)
6464
goto free_pd;
6565

66-
pd->flags |= GENPD_FLAG_IRQ_SAFE | GENPD_FLAG_CPU_DOMAIN;
66+
pd->flags |= GENPD_FLAG_IRQ_SAFE | GENPD_FLAG_CPU_DOMAIN |
67+
GENPD_FLAG_NO_SYNC_STATE;
6768

6869
/*
6970
* Allow power off when OSI has been successfully enabled.
@@ -128,11 +129,16 @@ static void psci_pd_remove(void)
128129

129130
static void psci_cpuidle_domain_sync_state(struct device *dev)
130131
{
132+
struct psci_pd_provider *pd_provider;
133+
131134
/*
132135
* All devices have now been attached/probed to the PM domain topology,
133136
* hence it's fine to allow domain states to be picked.
134137
*/
135138
psci_pd_allow_domain_state = true;
139+
140+
list_for_each_entry(pd_provider, &psci_pd_providers, link)
141+
of_genpd_sync_state(pd_provider->node);
136142
}
137143

138144
static const struct of_device_id psci_of_match[] = {

0 commit comments

Comments
 (0)