From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Srivatsa S. Bhat" Subject: Re: [PATCH 1/5] cpufreq: Remove extra blank line Date: Thu, 12 Sep 2013 13:46:09 +0530 Message-ID: <52317849.90101@linux.vnet.ibm.com> References: <22f0d94203b9e3d29f123c0e80df4b012e01dbb5.1378963070.git.viresh.kumar@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <22f0d94203b9e3d29f123c0e80df4b012e01dbb5.1378963070.git.viresh.kumar@linaro.org> Sender: cpufreq-owner@vger.kernel.org To: Viresh Kumar Cc: rjw@sisk.pl, swarren@wwwdotorg.org, linaro-kernel@lists.linaro.org, patches@linaro.org, cpufreq@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-pm@vger.kernel.org On 09/12/2013 10:55 AM, Viresh Kumar wrote: > We don't need a blank line just at start of a block, lets remove it. > Well, I felt having that line avoids clutter, especially since the code around it was already a bit hard to read.. Anyway, I don't have any strong opinions either way. So no objections from my side. (But you might have to rebase this patch on top of Rafael's tree, due to the recent changes he pushed in to this code). Regards, Srivatsa S. Bhat > Signed-off-by: Viresh Kumar > --- > drivers/cpufreq/cpufreq.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index 5a64f66..28477eb 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -1185,7 +1185,6 @@ static int __cpufreq_remove_dev_prepare(struct device *dev, > if (cpu != policy->cpu && !frozen) { > sysfs_remove_link(&dev->kobj, "cpufreq"); > } else if (cpus > 1) { > - > new_cpu = cpufreq_nominate_new_policy_cpu(policy, cpu, frozen); > if (new_cpu >= 0) { > WARN_ON(lock_policy_rwsem_write(cpu)); >