@@ -2298,6 +2298,16 @@ static bool genpd_present(const struct generic_pm_domain *genpd)
return ret;
}
+static void of_genpd_get_power_supply_irq(struct generic_pm_domain *pd)
+{
+ if (of_property_read_bool(pd->dev.of_node, "power-supply-needs-irq")) {
+ pd->domain.ops.suspend = genpd_suspend_noirq;
+ pd->domain.ops.resume = genpd_resume_noirq;
+ pd->domain.ops.suspend_noirq = NULL;
+ pd->domain.ops.resume_noirq = NULL;
+ }
+}
+
/**
* of_genpd_add_provider_simple() - Register a simple PM domain provider
* @np: Device node pointer associated with the PM domain provider.
@@ -2343,6 +2353,8 @@ int of_genpd_add_provider_simple(struct device_node *np,
genpd->provider = &np->fwnode;
genpd->has_provider = true;
+ of_genpd_get_power_supply_irq(genpd);
+
return 0;
}
EXPORT_SYMBOL_GPL(of_genpd_add_provider_simple);
@@ -2394,6 +2406,8 @@ int of_genpd_add_provider_onecell(struct device_node *np,
genpd->provider = &np->fwnode;
genpd->has_provider = true;
+
+ of_genpd_get_power_supply_irq(genpd);
}
ret = genpd_add_provider(np, data->xlate, data);
Handle a new DT property power-supply-needs-irq that boards can use if the power-domains' power-supply node (regulator) needs interrupts to function. An example is a pmic regulator on i2c. Since suspend/resume doesn't work in these cases currently, switch to the normal, outer suspend/resume callbacks. Usually such a regulator already times out during suspend_noirq: [ 41.024193] buck4: failed to disable: -ETIMEDOUT Initially systemd suspend problems had been discussed at https://lore.kernel.org/linux-arm-kernel/20211002005954.1367653-8-l.stach@pengutronix.de/ which led to discussing the pmic that contains the regulators which serve as power-domain power-supplies: https://lore.kernel.org/linux-pm/573166b75e524517782471c2b7f96e03fd93d175.camel@puri.sm/T/ So this enables boards to use power-supply-needs-irq and fix systemd suspend and resume. Possibly one can find more changes this property should configure. They can be added later when testing them. Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm> --- drivers/base/power/domain.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+)