[PATCH v3 05/12] pmdomain: Make genpd_get_from_provider() public
From: Geert Uytterhoeven <geert+renesas@glider.be>
Date: 2026-09-02 11:30:11
Also in:
arm-scmi, linux-clk, linux-devicetree, linux-pm, linux-renesas-soc, lkml
Subsystem:
generic pm domains, power management core, the rest · Maintainers:
Ulf Hansson, "Rafael J. Wysocki", Linus Torvalds
Unlike the corresponding function in the clock subsystem (of_clk_get_from_provider()), genpd_get_from_provider() is private, and thus cannot be used by PM Domain drivers. Make it public, so it be used by the R-Car X5H Module Controller driver. Add an "of_"-prefix for consistency with public genpd function naming. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- v3: - Add "of_" prefix, v2: - No changes. --- drivers/pmdomain/core.c | 22 +++++++++++----------- include/linux/pm_domain.h | 7 +++++++ 2 files changed, 18 insertions(+), 11 deletions(-)
diff --git a/drivers/pmdomain/core.c b/drivers/pmdomain/core.c
index 842c4169e290677c..ac3a07066e7fc0e7 100644
--- a/drivers/pmdomain/core.c
+++ b/drivers/pmdomain/core.c@@ -2890,7 +2890,7 @@ void of_genpd_del_provider(struct device_node *np) EXPORT_SYMBOL_GPL(of_genpd_del_provider); /** - * genpd_get_from_provider() - Look-up PM domain + * of_genpd_get_from_provider() - Look-up PM domain * @genpdspec: OF phandle args to use for look-up * * Looks for a PM domain provider under the node specified by @genpdspec and if
@@ -2900,8 +2900,7 @@ EXPORT_SYMBOL_GPL(of_genpd_del_provider); * Returns a valid pointer to struct generic_pm_domain on success or ERR_PTR() * on failure. */ -static struct generic_pm_domain *genpd_get_from_provider( - const struct of_phandle_args *genpdspec) +struct generic_pm_domain *of_genpd_get_from_provider(const struct of_phandle_args *genpdspec) { struct generic_pm_domain *genpd = ERR_PTR(-ENOENT); struct of_genpd_provider *provider;
@@ -2923,6 +2922,7 @@ static struct generic_pm_domain *genpd_get_from_provider( return genpd; } +EXPORT_SYMBOL_GPL(of_genpd_get_from_provider); /** * of_genpd_add_child_ids() - Parse power-domains-child-ids property
@@ -2995,7 +2995,7 @@ int of_genpd_add_child_ids(struct device_node *np, goto err_unwind; /* Get the parent domain */ - parent_genpd = genpd_get_from_provider(&parent_args); + parent_genpd = of_genpd_get_from_provider(&parent_args); of_node_put(parent_args.np); if (IS_ERR(parent_genpd)) { pr_err("Failed to get parent domain for %pOF: %ld\n",
@@ -3078,7 +3078,7 @@ int of_genpd_remove_child_ids(struct device_node *np, if (ret) continue; - parent_genpd = genpd_get_from_provider(&parent_args); + parent_genpd = of_genpd_get_from_provider(&parent_args); of_node_put(parent_args.np); if (IS_ERR(parent_genpd)) continue;
@@ -3109,7 +3109,7 @@ int of_genpd_add_device(const struct of_phandle_args *genpdspec, struct device * mutex_lock(&gpd_list_lock); - genpd = genpd_get_from_provider(genpdspec); + genpd = of_genpd_get_from_provider(genpdspec); if (IS_ERR(genpd)) { ret = PTR_ERR(genpd); goto out;
@@ -3141,13 +3141,13 @@ int of_genpd_add_subdomain(const struct of_phandle_args *parent_spec, mutex_lock(&gpd_list_lock); - parent = genpd_get_from_provider(parent_spec); + parent = of_genpd_get_from_provider(parent_spec); if (IS_ERR(parent)) { ret = PTR_ERR(parent); goto out; } - subdomain = genpd_get_from_provider(subdomain_spec); + subdomain = of_genpd_get_from_provider(subdomain_spec); if (IS_ERR(subdomain)) { ret = PTR_ERR(subdomain); goto out;
@@ -3179,13 +3179,13 @@ int of_genpd_remove_subdomain(const struct of_phandle_args *parent_spec, mutex_lock(&gpd_list_lock); - parent = genpd_get_from_provider(parent_spec); + parent = of_genpd_get_from_provider(parent_spec); if (IS_ERR(parent)) { ret = PTR_ERR(parent); goto out; } - subdomain = genpd_get_from_provider(subdomain_spec); + subdomain = of_genpd_get_from_provider(subdomain_spec); if (IS_ERR(subdomain)) { ret = PTR_ERR(subdomain); goto out;
@@ -3373,7 +3373,7 @@ static int __genpd_dev_pm_attach(struct device *dev, struct device *base_dev, return ret; mutex_lock(&gpd_list_lock); - pd = genpd_get_from_provider(&pd_args); + pd = of_genpd_get_from_provider(&pd_args); of_node_put(pd_args.np); if (IS_ERR(pd)) { mutex_unlock(&gpd_list_lock);
diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
index f925614aebdbc914..0ef717428f2935e7 100644
--- a/include/linux/pm_domain.h
+++ b/include/linux/pm_domain.h@@ -458,6 +458,7 @@ int of_genpd_add_provider_simple(struct device_node *np, int of_genpd_add_provider_onecell(struct device_node *np, struct genpd_onecell_data *data); void of_genpd_del_provider(struct device_node *np); +struct generic_pm_domain *of_genpd_get_from_provider(const struct of_phandle_args *genpdspec); int of_genpd_add_device(const struct of_phandle_args *args, struct device *dev); int of_genpd_add_subdomain(const struct of_phandle_args *parent_spec, const struct of_phandle_args *subdomain_spec);
@@ -492,6 +493,12 @@ static inline int of_genpd_add_provider_onecell(struct device_node *np, static inline void of_genpd_del_provider(struct device_node *np) {} +static inline struct generic_pm_domain *of_genpd_get_from_provider( + const struct of_phandle_args *genpdspec) +{ + return ERR_PTR(-ENODEV); +} + static inline int of_genpd_add_device(const struct of_phandle_args *args, struct device *dev) {
--
2.43.0