[PATCH] x86: Fix VIA C3 Longhaul v2 identification

From: Krzysztof Helt
Date: Sun May 17 2009 - 11:58:32 EST


From: Krzysztof Helt <krzysztof.h1@xxxxx>

An identification of the Longhaul versions (v1, v2 and v3) selects only
v1 or v3 version. There is a special case for late Samuel2 and early Ezra cores
which support the Longhaul v2 but the Longhaul v1 is selected. These cores
are described in the comment as supporting the Longhaul v2.

Select the Longhaul v2 for cores which support it.

Signed-off-by: Krzysztof Helt <krzysztof.h1@xxxxx>
---
I don't know if selecting the Longhaul v1 for these cores was a typo or done or
purpose. I cannot find any reference to this part of the code.

I have tested the change and it enables the Longhaul v2 on my Samuel2 core
but voltage change is not supported by my motherboard (PCChips something).
The voltage range is detected as a single value.

diff --git a/arch/x86/kernel/cpu/cpufreq/longhaul.c b/arch/x86/kernel/cpu/cpufreq/longhaul.c
index ce2ed3e..cabd2fa 100644
--- a/arch/x86/kernel/cpu/cpufreq/longhaul.c
+++ b/arch/x86/kernel/cpu/cpufreq/longhaul.c
@@ -813,7 +813,7 @@ static int __init longhaul_cpu_init(struct cpufreq_policy *policy)
memcpy(eblcr, samuel2_eblcr, sizeof(samuel2_eblcr));
break;
case 1 ... 15:
- longhaul_version = TYPE_LONGHAUL_V1;
+ longhaul_version = TYPE_LONGHAUL_V2;
if (c->x86_mask < 8) {
cpu_model = CPU_SAMUEL2;
cpuname = "C3 'Samuel 2' [C5B]";
--
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/