Messages in this thread | | | Date | Fri, 23 May 2014 20:40:43 +0530 | Subject | Re: [PATCH v2] cpufreq: s5pv210: drop check for CONFIG_PM_VERBOSE | From | Viresh Kumar <> |
| |
On 23 May 2014 20:35, Paul Bolle <pebolle@tiscali.nl> wrote: > A pr_err() was added in v3.1. It was guarded by a check for > CONFIG_PM_VERBOSE. The Kconfig symbol PM_VERBOSE was removed in v3.0. So > this pr_err() has never been used. Drop that check and clean up the > message a bit. > > Signed-off-by: Paul Bolle <pebolle@tiscali.nl> > --- > v1 was called "cpufreq: s5pv210: remove unused call of pr_err()". v2 > keeps the pr_err() and trims down its arguments, as Sachin requested. > > Still untested. > > drivers/cpufreq/s5pv210-cpufreq.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/cpufreq/s5pv210-cpufreq.c b/drivers/cpufreq/s5pv210-cpufreq.c > index ab2c1a40d437..19a10b89fef7 100644 > --- a/drivers/cpufreq/s5pv210-cpufreq.c > +++ b/drivers/cpufreq/s5pv210-cpufreq.c > @@ -175,10 +175,8 @@ static int s5pv210_target(struct cpufreq_policy *policy, unsigned int index) > mutex_lock(&set_freq_lock); > > if (no_cpufreq_access) { > -#ifdef CONFIG_PM_VERBOSE > - pr_err("%s:%d denied access to %s as it is disabled" > - "temporarily\n", __FILE__, __LINE__, __func__); > -#endif > + pr_err("Denied access to %s as it is disabled temporarily\n", > + __func__); > ret = -EINVAL; > goto exit; > }
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
| |