diff mbox series

cpufreq: Remove unused for_each_policy macro

Message ID 1617692459-35670-1-git-send-email-zhangshaokun@hisilicon.com (mailing list archive)
State Awaiting Upstream
Delegated to: Rafael Wysocki
Headers show
Series cpufreq: Remove unused for_each_policy macro | expand

Commit Message

Shaokun Zhang April 6, 2021, 7 a.m. UTC
macro 'for_each_policy' has become unused since commit
f963735a3ca3 ("cpufreq: Create for_each_{in}active_policy()"),
Remove it.

Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Shaokun Zhang <zhangshaokun@hisilicon.com>
---
 drivers/cpufreq/cpufreq.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Viresh Kumar April 6, 2021, 7:06 a.m. UTC | #1
On 06-04-21, 15:00, Shaokun Zhang wrote:
> macro 'for_each_policy' has become unused since commit
> f963735a3ca3 ("cpufreq: Create for_each_{in}active_policy()"),
> Remove it.
> 
> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> Cc: Viresh Kumar <viresh.kumar@linaro.org>
> Signed-off-by: Shaokun Zhang <zhangshaokun@hisilicon.com>
> ---
>  drivers/cpufreq/cpufreq.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> index 1d1b563cea4b..802abc925b2a 100644
> --- a/drivers/cpufreq/cpufreq.c
> +++ b/drivers/cpufreq/cpufreq.c
> @@ -42,9 +42,6 @@ static LIST_HEAD(cpufreq_policy_list);
>  #define for_each_inactive_policy(__policy)		\
>  	for_each_suitable_policy(__policy, false)
>  
> -#define for_each_policy(__policy)			\
> -	list_for_each_entry(__policy, &cpufreq_policy_list, policy_list)
> -
>  /* Iterate over governors */
>  static LIST_HEAD(cpufreq_governor_list);
>  #define for_each_governor(__governor)				\

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Rafael J. Wysocki April 8, 2021, 5:28 p.m. UTC | #2
On Tue, Apr 6, 2021 at 9:06 AM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
> On 06-04-21, 15:00, Shaokun Zhang wrote:
> > macro 'for_each_policy' has become unused since commit
> > f963735a3ca3 ("cpufreq: Create for_each_{in}active_policy()"),
> > Remove it.
> >
> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> > Cc: Viresh Kumar <viresh.kumar@linaro.org>
> > Signed-off-by: Shaokun Zhang <zhangshaokun@hisilicon.com>
> > ---
> >  drivers/cpufreq/cpufreq.c | 3 ---
> >  1 file changed, 3 deletions(-)
> >
> > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> > index 1d1b563cea4b..802abc925b2a 100644
> > --- a/drivers/cpufreq/cpufreq.c
> > +++ b/drivers/cpufreq/cpufreq.c
> > @@ -42,9 +42,6 @@ static LIST_HEAD(cpufreq_policy_list);
> >  #define for_each_inactive_policy(__policy)           \
> >       for_each_suitable_policy(__policy, false)
> >
> > -#define for_each_policy(__policy)                    \
> > -     list_for_each_entry(__policy, &cpufreq_policy_list, policy_list)
> > -
> >  /* Iterate over governors */
> >  static LIST_HEAD(cpufreq_governor_list);
> >  #define for_each_governor(__governor)                                \
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Applied as 5.13 material, thanks!
diff mbox series

Patch

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index 1d1b563cea4b..802abc925b2a 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -42,9 +42,6 @@  static LIST_HEAD(cpufreq_policy_list);
 #define for_each_inactive_policy(__policy)		\
 	for_each_suitable_policy(__policy, false)
 
-#define for_each_policy(__policy)			\
-	list_for_each_entry(__policy, &cpufreq_policy_list, policy_list)
-
 /* Iterate over governors */
 static LIST_HEAD(cpufreq_governor_list);
 #define for_each_governor(__governor)				\