diff mbox series

[2/8] cpufreq: dt: Use auto-registration for energy model

Message ID f66047fca65a2d368e881d62e1623dbeef2eacd2.1628579170.git.viresh.kumar@linaro.org
State New
Headers show
Series cpufreq: Auto-register with energy model | expand

Commit Message

Viresh Kumar Aug. 10, 2021, 7:36 a.m. UTC
Use the CPUFREQ_REGISTER_WITH_EM flag to allow cpufreq core to
automatically register with the energy model.

This allows removal of boiler plate code from the driver and fixes the
unregistration part as well.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

---
 drivers/cpufreq/cpufreq-dt.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

-- 
2.31.1.272.g89b43f80a514

Comments

Lukasz Luba Aug. 10, 2021, 10:19 a.m. UTC | #1
On 8/10/21 8:36 AM, Viresh Kumar wrote:
> Use the CPUFREQ_REGISTER_WITH_EM flag to allow cpufreq core to

> automatically register with the energy model.

> 

> This allows removal of boiler plate code from the driver and fixes the

> unregistration part as well.

> 

> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

> ---

>   drivers/cpufreq/cpufreq-dt.c | 5 ++---

>   1 file changed, 2 insertions(+), 3 deletions(-)

> 

> diff --git a/drivers/cpufreq/cpufreq-dt.c b/drivers/cpufreq/cpufreq-dt.c

> index ece52863ba62..b727006e85af 100644

> --- a/drivers/cpufreq/cpufreq-dt.c

> +++ b/drivers/cpufreq/cpufreq-dt.c

> @@ -143,8 +143,6 @@ static int cpufreq_init(struct cpufreq_policy *policy)

>   		cpufreq_dt_attr[1] = &cpufreq_freq_attr_scaling_boost_freqs;

>   	}

>   

> -	dev_pm_opp_of_register_em(cpu_dev, policy->cpus);

> -

>   	return 0;

>   

>   out_clk_put:

> @@ -176,7 +174,8 @@ static int cpufreq_exit(struct cpufreq_policy *policy)

>   

>   static struct cpufreq_driver dt_cpufreq_driver = {

>   	.flags = CPUFREQ_NEED_INITIAL_FREQ_CHECK |

> -		 CPUFREQ_IS_COOLING_DEV,

> +		 CPUFREQ_IS_COOLING_DEV |

> +		 CPUFREQ_REGISTER_WITH_EM,

>   	.verify = cpufreq_generic_frequency_table_verify,

>   	.target_index = set_target,

>   	.get = cpufreq_generic_get,

> 


Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
diff mbox series

Patch

diff --git a/drivers/cpufreq/cpufreq-dt.c b/drivers/cpufreq/cpufreq-dt.c
index ece52863ba62..b727006e85af 100644
--- a/drivers/cpufreq/cpufreq-dt.c
+++ b/drivers/cpufreq/cpufreq-dt.c
@@ -143,8 +143,6 @@  static int cpufreq_init(struct cpufreq_policy *policy)
 		cpufreq_dt_attr[1] = &cpufreq_freq_attr_scaling_boost_freqs;
 	}
 
-	dev_pm_opp_of_register_em(cpu_dev, policy->cpus);
-
 	return 0;
 
 out_clk_put:
@@ -176,7 +174,8 @@  static int cpufreq_exit(struct cpufreq_policy *policy)
 
 static struct cpufreq_driver dt_cpufreq_driver = {
 	.flags = CPUFREQ_NEED_INITIAL_FREQ_CHECK |
-		 CPUFREQ_IS_COOLING_DEV,
+		 CPUFREQ_IS_COOLING_DEV |
+		 CPUFREQ_REGISTER_WITH_EM,
 	.verify = cpufreq_generic_frequency_table_verify,
 	.target_index = set_target,
 	.get = cpufreq_generic_get,