[PATCH 2.6] cpufreq: two minor cleanups

From: Jean Delvare
Date: Tue Jun 28 2005 - 16:31:29 EST


Hi Dave,

Here are two possible cleanups in cpufreq.c:
* ret has no need to be unsigned in cpufreq_driver_target()
* ret has no need to be initialized in __cpufreq_governor()

Thanks.

Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx>

drivers/cpufreq/cpufreq.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

--- linux-2.6.12.orig/drivers/cpufreq/cpufreq.c 2005-06-21 21:25:31.000000000 +0200
+++ linux-2.6.12/drivers/cpufreq/cpufreq.c 2005-06-21 21:29:01.000000000 +0200
@@ -1130,7 +1130,7 @@
unsigned int target_freq,
unsigned int relation)
{
- unsigned int ret;
+ int ret;

policy = cpufreq_cpu_get(policy->cpu);
if (!policy)
@@ -1151,7 +1151,7 @@

static int __cpufreq_governor(struct cpufreq_policy *policy, unsigned int event)
{
- int ret = -EINVAL;
+ int ret;

if (!try_module_get(policy->governor->owner))
return -EINVAL;


--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/