admin/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2025-12-07 20:06:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

pmdomain: core: Add the genpd->dev to the genpd provider bus

Browse Source 
To take the next step for a more common handling of the genpd providers,
let's add the genpd->dev to the genpd provider bus when registering a genpd
OF provider.

Also note, to allow us to add devices to the genpd provider bus we need to
make sure the bus has been registered first, which is done via a
core_initcall. Hence, calls to of_genpd_add_provider_simple|onecell() must
be done after the bus has been registered, else they will fail.

Suggested-by: Saravana Kannan <saravanak@google.com>
Reviewed-by: Abel Vesa <abel.vesa@linaro.org>
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-7-ulf.hansson@linaro.org
This commit is contained in:
Ulf Hansson
2025-07-01 13:47:08 +02:00
parent 31cb750770
commit 18a3a510ec
1 changed files with 34 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
drivers/pmdomain/core.c
View File

@@ -2611,16 +2611,25 @@ int of_genpd_add_provider_simple(struct device_node *np,
if (!np || !genpd)
return -EINVAL;
if (!genpd_bus_registered)
return -ENODEV;
if (!genpd_present(genpd))
return -EINVAL;
genpd->dev.of_node = np;
ret = device_add(&genpd->dev);
if (ret)
return ret;
/* Parse genpd OPP table */
if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
ret = dev_pm_opp_of_add_table(&genpd->dev);
if (ret)
return dev_err_probe(&genpd->dev, ret, "Failed to add OPP table\n");
if (ret) {
dev_err_probe(&genpd->dev, ret, "Failed to add OPP table\n");
goto err_del;
}
/*
* Save table for faster processing while setting performance
@@ -2631,19 +2640,22 @@ int of_genpd_add_provider_simple(struct device_node *np,
}
ret = genpd_add_provider(np, genpd_xlate_simple, genpd);
if (ret) {
if (genpd->opp_table) {
dev_pm_opp_put_opp_table(genpd->opp_table);
dev_pm_opp_of_remove_table(&genpd->dev);
}
return ret;
}
if (ret)
goto err_opp;
genpd->provider = &np->fwnode;
genpd->has_provider = true;
return 0;
err_opp:
if (genpd->opp_table) {
dev_pm_opp_put_opp_table(genpd->opp_table);
dev_pm_opp_of_remove_table(&genpd->dev);
}
err_del:
device_del(&genpd->dev);
return ret;
}
EXPORT_SYMBOL_GPL(of_genpd_add_provider_simple);
@@ -2662,6 +2674,9 @@ int of_genpd_add_provider_onecell(struct device_node *np,
if (!np || !data)
return -EINVAL;
if (!genpd_bus_registered)
return -ENODEV;
if (!data->xlate)
data->xlate = genpd_xlate_onecell;
@@ -2675,12 +2690,17 @@ int of_genpd_add_provider_onecell(struct device_node *np,
genpd->dev.of_node = np;
ret = device_add(&genpd->dev);
if (ret)
goto error;
/* Parse genpd OPP table */
if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
ret = dev_pm_opp_of_add_table_indexed(&genpd->dev, i);
if (ret) {
dev_err_probe(&genpd->dev, ret,
"Failed to add OPP table for index %d\n", i);
device_del(&genpd->dev);
goto error;
}
@@ -2716,6 +2736,8 @@ error:
dev_pm_opp_put_opp_table(genpd->opp_table);
dev_pm_opp_of_remove_table(&genpd->dev);
}
device_del(&genpd->dev);
}
return ret;
@@ -2748,6 +2770,8 @@ void of_genpd_del_provider(struct device_node *np)
dev_pm_opp_put_opp_table(gpd->opp_table);
dev_pm_opp_of_remove_table(&gpd->dev);
}
device_del(&gpd->dev);
}
}