shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-05 10:31:46 +09:00
Code Issues Packages Projects Releases Wiki Activity

UPSTREAM: PM: domains: Allow genpd providers to manage OPP tables directly by its FW

Browse Source 
In some cases the OPP tables aren't specified in device tree, but rather
encoded in the FW. To allow a genpd provider to specify them dynamically
instead, let's add a new genpd flag, GENPD_FLAG_OPP_TABLE_FW.

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Link: https://lore.kernel.org/r/20230825112633.236607-13-ulf.hansson@linaro.org
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

Bug: 323966425
Change-Id: I14f40b8cd4acce92ad5fd7a1769c513e7cbfa7ae
(cherry picked from commit 3dd91515ef43dd43e32e2a84e4bd881b64fb33ae)
Signed-off-by: Nikunj Kela <quic_nkela@quicinc.com>
Signed-off-by: Anant Goel <quic_anantg@quicinc.com>
This commit is contained in:
Ulf Hansson
2023-08-25 13:26:32 +02:00
committed by Anant Goel
parent c638aef4e9
commit 5c0092ff97
2 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
drivers/base/power/domain.c
View File

@@ -130,6 +130,7 @@ static const struct genpd_lock_ops genpd_spin_ops = {
#define genpd_is_active_wakeup(genpd) (genpd->flags & GENPD_FLAG_ACTIVE_WAKEUP) #define genpd_is_active_wakeup(genpd) (genpd->flags & GENPD_FLAG_ACTIVE_WAKEUP)
#define genpd_is_cpu_domain(genpd) (genpd->flags & GENPD_FLAG_CPU_DOMAIN) #define genpd_is_cpu_domain(genpd) (genpd->flags & GENPD_FLAG_CPU_DOMAIN)
#define genpd_is_rpm_always_on(genpd) (genpd->flags & GENPD_FLAG_RPM_ALWAYS_ON) #define genpd_is_rpm_always_on(genpd) (genpd->flags & GENPD_FLAG_RPM_ALWAYS_ON)
#define genpd_is_opp_table_fw(genpd) (genpd->flags & GENPD_FLAG_OPP_TABLE_FW)
static inline bool irq_safe_dev_in_sleep_domain(struct device *dev, static inline bool irq_safe_dev_in_sleep_domain(struct device *dev,
const struct generic_pm_domain *genpd) const struct generic_pm_domain *genpd)
@@ -2319,7 +2320,7 @@ int of_genpd_add_provider_simple(struct device_node *np,
genpd->dev.of_node = np; genpd->dev.of_node = np;
/* Parse genpd OPP table */ /* Parse genpd OPP table */
if (genpd->set_performance_state) { if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
ret = dev_pm_opp_of_add_table(&genpd->dev); ret = dev_pm_opp_of_add_table(&genpd->dev);
if (ret) if (ret)
return dev_err_probe(&genpd->dev, ret, "Failed to add OPP table\n"); return dev_err_probe(&genpd->dev, ret, "Failed to add OPP table\n");
@@ -2334,7 +2335,7 @@ int of_genpd_add_provider_simple(struct device_node *np,
ret = genpd_add_provider(np, genpd_xlate_simple, genpd); ret = genpd_add_provider(np, genpd_xlate_simple, genpd);
if (ret) { if (ret) {
if (genpd->set_performance_state) { if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
dev_pm_opp_put_opp_table(genpd->opp_table); dev_pm_opp_put_opp_table(genpd->opp_table);
dev_pm_opp_of_remove_table(&genpd->dev); dev_pm_opp_of_remove_table(&genpd->dev);
} }
@@ -2378,7 +2379,7 @@ int of_genpd_add_provider_onecell(struct device_node *np,
genpd->dev.of_node = np; genpd->dev.of_node = np;
/* Parse genpd OPP table */ /* Parse genpd OPP table */
if (genpd->set_performance_state) { if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
ret = dev_pm_opp_of_add_table_indexed(&genpd->dev, i); ret = dev_pm_opp_of_add_table_indexed(&genpd->dev, i);
if (ret) { if (ret) {
dev_err_probe(&genpd->dev, ret, dev_err_probe(&genpd->dev, ret,
@@ -2414,7 +2415,7 @@ error:
genpd->provider = NULL; genpd->provider = NULL;
genpd->has_provider = false; genpd->has_provider = false;
if (genpd->set_performance_state) { if (!genpd_is_opp_table_fw(genpd) && genpd->set_performance_state) {
dev_pm_opp_put_opp_table(genpd->opp_table); dev_pm_opp_put_opp_table(genpd->opp_table);
dev_pm_opp_of_remove_table(&genpd->dev); dev_pm_opp_of_remove_table(&genpd->dev);
} }
@@ -2446,7 +2447,7 @@ void of_genpd_del_provider(struct device_node *np)
if (gpd->provider == &np->fwnode) { if (gpd->provider == &np->fwnode) {
gpd->has_provider = false; gpd->has_provider = false;
if (!gpd->set_performance_state) if (genpd_is_opp_table_fw(gpd) || !gpd->set_performance_state)
continue; continue;
dev_pm_opp_put_opp_table(gpd->opp_table); dev_pm_opp_put_opp_table(gpd->opp_table);

5
include/linux/pm_domain.h
View File

@@ -60,6 +60,10 @@
* GENPD_FLAG_MIN_RESIDENCY: Enable the genpd governor to consider its * GENPD_FLAG_MIN_RESIDENCY: Enable the genpd governor to consider its
* components' next wakeup when determining the * components' next wakeup when determining the
* optimal idle state. * optimal idle state.
*
* GENPD_FLAG_OPP_TABLE_FW: The genpd provider supports performance states,
* but its corresponding OPP tables are not
* described in DT, but are given directly by FW.
*/ */
#define GENPD_FLAG_PM_CLK (1U << 0) #define GENPD_FLAG_PM_CLK (1U << 0)
#define GENPD_FLAG_IRQ_SAFE (1U << 1) #define GENPD_FLAG_IRQ_SAFE (1U << 1)
@@ -68,6 +72,7 @@
#define GENPD_FLAG_CPU_DOMAIN (1U << 4) #define GENPD_FLAG_CPU_DOMAIN (1U << 4)
#define GENPD_FLAG_RPM_ALWAYS_ON (1U << 5) #define GENPD_FLAG_RPM_ALWAYS_ON (1U << 5)
#define GENPD_FLAG_MIN_RESIDENCY (1U << 6) #define GENPD_FLAG_MIN_RESIDENCY (1U << 6)
#define GENPD_FLAG_OPP_TABLE_FW (1U << 7)
enum gpd_status { enum gpd_status {
GENPD_STATE_ON = 0, /* PM domain is on */ GENPD_STATE_ON = 0, /* PM domain is on */