Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1402920 > unrolled thread

[PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov

Started byViresh Kumar <viresh.kumar@linaro.org>
First post2016-05-18 14:30 +0200
Last post2016-05-19 04:20 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov Viresh Kumar <viresh.kumar@linaro.org> - 2016-05-18 14:30 +0200
    Re: [PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov "Rafael J. Wysocki" <rafael@kernel.org> - 2016-05-18 23:10 +0200
      Re: [PATCH 4/6] cpufreq: conservative: Remove declaration of  cs_dbs_gov Viresh Kumar <viresh.kumar@linaro.org> - 2016-05-19 04:20 +0200

#1402920 — [PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov

FromViresh Kumar <viresh.kumar@linaro.org>
Date2016-05-18 14:30 +0200
Subject[PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov
Message-ID<rA8LT-25R-11@gated-at.bofh.it>
The code doesn't use 'cs_dbs_gov' anymore before it is defined and the
extra line for declaring it can be removed.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
 drivers/cpufreq/cpufreq_conservative.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c
index 5a15f649602c..904839171d94 100644
--- a/drivers/cpufreq/cpufreq_conservative.c
+++ b/drivers/cpufreq/cpufreq_conservative.c
@@ -127,8 +127,6 @@ static struct notifier_block cs_cpufreq_notifier_block = {
 };
 
 /************************** sysfs interface ************************/
-static struct dbs_governor cs_dbs_gov;
-
 static ssize_t store_sampling_down_factor(struct gov_attr_set *attr_set,
 					  const char *buf, size_t count)
 {
-- 
2.7.1.410.g6faf27b

[toc] | [next] | [standalone]


#1403288

From"Rafael J. Wysocki" <rafael@kernel.org>
Date2016-05-18 23:10 +0200
Message-ID<rAgT8-7if-11@gated-at.bofh.it>
In reply to#1402920
On Wed, May 18, 2016 at 2:25 PM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> The code doesn't use 'cs_dbs_gov' anymore before it is defined and the
> extra line for declaring it can be removed.

This sort of conflicts with a patch I've just posted
(https://patchwork.kernel.org/patch/9122121/) and I'd prefer to go
with that one if that's not a big deal.

[toc] | [prev] | [next] | [standalone]


#1403401 — Re: [PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov

FromViresh Kumar <viresh.kumar@linaro.org>
Date2016-05-19 04:20 +0200
SubjectRe: [PATCH 4/6] cpufreq: conservative: Remove declaration of cs_dbs_gov
Message-ID<rAlJ8-21U-17@gated-at.bofh.it>
In reply to#1403288
On 18-05-16, 23:03, Rafael J. Wysocki wrote:
> On Wed, May 18, 2016 at 2:25 PM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> > The code doesn't use 'cs_dbs_gov' anymore before it is defined and the
> > extra line for declaring it can be removed.
> 
> This sort of conflicts with a patch I've just posted
> (https://patchwork.kernel.org/patch/9122121/) and I'd prefer to go
> with that one if that's not a big deal.

Yeah, drop it.

-- 
viresh

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web