[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <1310114243.2940.3.camel@phoenix>
Date: Fri, 08 Jul 2011 16:37:23 +0800
From: Axel Lin <axel.lin@...il.com>
To: linux-kernel@...r.kernel.org
Cc: Rafal Bilski <rafalbilski@...eria.pl>,
Dave Jones <davej@...hat.com>, cpufreq@...r.kernel.org
Subject: [RFC][PATCH] cpufreq: e_powersaver: Fix checking brand for
EPS_BRAND_C3
Do not return -ENODEV for EPS_BRAND_C3.
Signed-off-by: Axel Lin <axel.lin@...il.com>
---
I just found the code is suspect.
I don't have this hardware handy.
Anyone can help to confirm and test if this patch works?
Axel
drivers/cpufreq/e_powersaver.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/cpufreq/e_powersaver.c b/drivers/cpufreq/e_powersaver.c
index 35a257d..637f055 100644
--- a/drivers/cpufreq/e_powersaver.c
+++ b/drivers/cpufreq/e_powersaver.c
@@ -199,8 +199,9 @@ static int eps_cpu_init(struct cpufreq_policy *policy)
break;
case EPS_BRAND_C3:
printk(KERN_CONT "C3\n");
- return -ENODEV;
break;
+ default:
+ return -ENODEV;
}
/* Enable Enhanced PowerSaver */
rdmsrl(MSR_IA32_MISC_ENABLE, val);
--
1.7.4.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists