* [PATCH] cpufreq, ondemand: Remove leftover debug line
@ 2013-05-08 19:34 Borislav Petkov
2013-05-12 15:16 ` Viresh Kumar
0 siblings, 1 reply; 3+ messages in thread
From: Borislav Petkov @ 2013-05-08 19:34 UTC (permalink / raw)
To: cpufreq; +Cc: LKML, Borislav Petkov
From: Borislav Petkov <bp@suse.de>
I don't see how the virtual address of the tuners pointer would be of
any help to anyone so remove it.
Signed-off-by: Borislav Petkov <bp@suse.de>
---
drivers/cpufreq/cpufreq_ondemand.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c
index b0ffef96bf77..4b9bb5def6f1 100644
--- a/drivers/cpufreq/cpufreq_ondemand.c
+++ b/drivers/cpufreq/cpufreq_ondemand.c
@@ -547,7 +547,6 @@ static int od_init(struct dbs_data *dbs_data)
tuners->io_is_busy = should_io_be_busy();
dbs_data->tuners = tuners;
- pr_info("%s: tuners %p\n", __func__, tuners);
mutex_init(&dbs_data->mutex);
return 0;
}
--
1.8.3.rc0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] cpufreq, ondemand: Remove leftover debug line
2013-05-08 19:34 [PATCH] cpufreq, ondemand: Remove leftover debug line Borislav Petkov
@ 2013-05-12 15:16 ` Viresh Kumar
2013-05-13 12:14 ` Rafael J. Wysocki
0 siblings, 1 reply; 3+ messages in thread
From: Viresh Kumar @ 2013-05-12 15:16 UTC (permalink / raw)
To: Borislav Petkov; +Cc: cpufreq, LKML, Borislav Petkov
On Thu, May 9, 2013 at 1:04 AM, Borislav Petkov <bp@alien8.de> wrote:
> From: Borislav Petkov <bp@suse.de>
>
> I don't see how the virtual address of the tuners pointer would be of
> any help to anyone so remove it.
>
> Signed-off-by: Borislav Petkov <bp@suse.de>
> ---
> drivers/cpufreq/cpufreq_ondemand.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c
> index b0ffef96bf77..4b9bb5def6f1 100644
> --- a/drivers/cpufreq/cpufreq_ondemand.c
> +++ b/drivers/cpufreq/cpufreq_ondemand.c
> @@ -547,7 +547,6 @@ static int od_init(struct dbs_data *dbs_data)
> tuners->io_is_busy = should_io_be_busy();
>
> dbs_data->tuners = tuners;
> - pr_info("%s: tuners %p\n", __func__, tuners);
> mutex_init(&dbs_data->mutex);
> return 0;
> }
Sorry for this message :(
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] cpufreq, ondemand: Remove leftover debug line
2013-05-12 15:16 ` Viresh Kumar
@ 2013-05-13 12:14 ` Rafael J. Wysocki
0 siblings, 0 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2013-05-13 12:14 UTC (permalink / raw)
To: Viresh Kumar; +Cc: Borislav Petkov, cpufreq, LKML, Borislav Petkov
On Sunday, May 12, 2013 08:46:34 PM Viresh Kumar wrote:
> On Thu, May 9, 2013 at 1:04 AM, Borislav Petkov <bp@alien8.de> wrote:
> > From: Borislav Petkov <bp@suse.de>
> >
> > I don't see how the virtual address of the tuners pointer would be of
> > any help to anyone so remove it.
> >
> > Signed-off-by: Borislav Petkov <bp@suse.de>
> > ---
> > drivers/cpufreq/cpufreq_ondemand.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c
> > index b0ffef96bf77..4b9bb5def6f1 100644
> > --- a/drivers/cpufreq/cpufreq_ondemand.c
> > +++ b/drivers/cpufreq/cpufreq_ondemand.c
> > @@ -547,7 +547,6 @@ static int od_init(struct dbs_data *dbs_data)
> > tuners->io_is_busy = should_io_be_busy();
> >
> > dbs_data->tuners = tuners;
> > - pr_info("%s: tuners %p\n", __func__, tuners);
> > mutex_init(&dbs_data->mutex);
> > return 0;
> > }
>
> Sorry for this message :(
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Applied to the linux-next branch.
Thanks,
Rafael
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-05-13 12:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-08 19:34 [PATCH] cpufreq, ondemand: Remove leftover debug line Borislav Petkov
2013-05-12 15:16 ` Viresh Kumar
2013-05-13 12:14 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox