[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

Udo van den Heuvel wrote:
> Hello Rudolf,
> 
> I saw this line in my logs:
> 
> hwmon-vid: Unknown VRM version of your x86 CPU
> 
> 
> I run a VIA Epia CL-6000E board for my firewall.
> Could you fix the patch?
> Can I help?

Yes give me please the cat /proc/cpuinfo

and result of "cpuid" utillity

Thanks
Regards
Rudolf

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

Rudolf Marek wrote:
>>I saw this line in my logs:
>>
>>hwmon-vid: Unknown VRM version of your x86 CPU
>>
>>
>>I run a VIA Epia CL-6000E board for my firewall.
>>Could you fix the patch?
>>Can I help?
> 
> Yes give me please the cat /proc/cpuinfo
> 
> and result of "cpuid" utillity

# cat /proc/cpuinfo
processor       : 0
vendor_id       : CentaurHauls
cpu family      : 6
model           : 7
model name      : VIA Samuel 2
stepping        : 3
cpu MHz         : 601.477
cache size      : 64 KB
fdiv_bug        : no
hlt_bug         : no
f00f_bug        : no
coma_bug        : no
fpu             : yes
fpu_exception   : yes
cpuid level     : 1
wp              : yes
flags           : fpu de tsc msr cx8 mtrr pge mmx 3dnow
bogomips        : 1204.51

cpuid utility? Which/where?
It is a VIA C3 Samuel 2.

Kind regards,
Udo

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

> 
> # cat /proc/cpuinfo
> processor       : 0
> vendor_id       : CentaurHauls
> cpu family      : 6
> model           : 7
> model name      : VIA Samuel 2
> stepping        : 3
> cpu MHz         : 601.477
> cache size      : 64 KB
> fdiv_bug        : no
> hlt_bug         : no
> f00f_bug        : no
> coma_bug        : no
> fpu             : yes
> fpu_exception   : yes
> cpuid level     : 1
> wp              : yes
> flags           : fpu de tsc msr cx8 mtrr pge mmx 3dnow
> bogomips        : 1204.51

This is OK for me now.

Please can you test attached patch? Because I cant see how it can fail.

Thanks

Regards
Rudolf
-------------- next part --------------
diff -Naur a/drivers/hwmon/hwmon-vid.c b/drivers/hwmon/hwmon-vid.c
--- a/drivers/hwmon/hwmon-vid.c	2005-10-28 02:02:08.000000000 +0200
+++ b/drivers/hwmon/hwmon-vid.c	2005-11-22 21:41:51.327622000 +0100
@@ -49,20 +49,22 @@
         . . . .
        11110  =  0.800 V
        11111  =  0.000 V (off)
+
+    The 17 specification is in fact Intel Mobile Voltage Positioning -
+    (IMVP-II). You can find more information in the datasheet of Max1718
+    http://www.maxim-ic.com/quick_view2.cfm/qv_pk/2452
+
 */
 
 /* vrm is the VRM/VRD document version multiplied by 10.
    val is the 4-, 5- or 6-bit VID code.
    Returned value is in mV to avoid floating point in the kernel. */
-int vid_from_reg(int val, int vrm)
+int vid_from_reg(int val, u8 vrm)
 {
 	int vid;
 
 	switch(vrm) {
 
-	case  0:
-		return 0;
-
 	case 100:               /* VRD 10.0 */
 		if((val & 0x1f) = 0x1f)
 			return 0;
@@ -91,10 +93,16 @@
 	case 84:		/* VRM 8.4 */
 		val &= 0x0f;
 				/* fall through */
-	default:		/* VRM 8.2 */
+	case 82:		/* VRM 8.2 */
 		return(val = 0x1f ? 0 :
 		       val & 0x10  ? 5100 - (val) * 100 :
 		                     2050 - (val) * 50);
+	case 17:		/* Intel IMVP-II */
+		return(val & 0x10 ? 975 - (val & 0xF) * 25 :
+				    1750 - val * 50);
+	default:		/* report 0 for unknown */
+		printk(KERN_INFO "hwmon-vid: requested unknown VRM version\n");
+		return 0;
 	}
 }
 
@@ -108,30 +116,36 @@
 	u8 vendor;
 	u8 eff_family;
 	u8 eff_model;
-	int vrm_type;
+	u8 eff_stepping;
+	u8 vrm_type;
 };
 
 #define ANY 0xFF
 
 #ifdef CONFIG_X86
 
+/* the stepping parameter is highest acceptable stepping for current line */
+
 static struct vrm_model vrm_models[] = {
-	{X86_VENDOR_AMD, 0x6, ANY, 90},		/* Athlon Duron etc */
-	{X86_VENDOR_AMD, 0xF, ANY, 24},		/* Athlon 64, Opteron */
-	{X86_VENDOR_INTEL, 0x6, 0x9, 85},	/* 0.13um too */
-	{X86_VENDOR_INTEL, 0x6, 0xB, 85},	/* Tualatin */
-	{X86_VENDOR_INTEL, 0x6, ANY, 82},	/* any P6 */
-	{X86_VENDOR_INTEL, 0x7, ANY, 0},	/* Itanium */
-	{X86_VENDOR_INTEL, 0xF, 0x0, 90},	/* P4 */
-	{X86_VENDOR_INTEL, 0xF, 0x1, 90},	/* P4 Willamette */
-	{X86_VENDOR_INTEL, 0xF, 0x2, 90},	/* P4 Northwood */
-	{X86_VENDOR_INTEL, 0xF, 0x3, 100},	/* P4 Prescott */
-	{X86_VENDOR_INTEL, 0xF, 0x4, 100},	/* P4 Prescott */
-	{X86_VENDOR_INTEL, 0x10,ANY, 0},	/* Itanium 2 */
-	{X86_VENDOR_UNKNOWN, ANY, ANY, 0}	/* stop here */
+	{X86_VENDOR_AMD, 0x6, ANY, ANY, 90},		/* Athlon Duron etc */
+	{X86_VENDOR_AMD, 0xF, ANY, ANY, 24},		/* Athlon 64, Opteron and above VRM 24 */
+	{X86_VENDOR_INTEL, 0x6, 0x9, ANY, 85},		/* 0.13um too */
+	{X86_VENDOR_INTEL, 0x6, 0xB, ANY, 85},		/* Tualatin */
+	{X86_VENDOR_INTEL, 0x6, ANY, ANY, 82},		/* any P6 */
+	{X86_VENDOR_INTEL, 0x7, ANY, ANY, 0},		/* Itanium */
+	{X86_VENDOR_INTEL, 0xF, 0x0, ANY, 90},		/* P4 */
+	{X86_VENDOR_INTEL, 0xF, 0x1, ANY, 90},		/* P4 Willamette */
+	{X86_VENDOR_INTEL, 0xF, 0x2, ANY, 90},		/* P4 Northwood */
+	{X86_VENDOR_INTEL, 0xF, ANY, ANY, 100},		/* Prescott and above assume VRD 10 */
+	{X86_VENDOR_INTEL, 0x10, ANY, ANY, 0},		/* Itanium 2 */
+	{X86_VENDOR_CENTAUR, 0x6, 0x7, ANY, 85},	/* Eden ESP/Ezra */
+	{X86_VENDOR_CENTAUR, 0x6, 0x8, 0x7, 85},	/* Ezra T */
+	{X86_VENDOR_CENTAUR, 0x6, 0x9, 0x7, 85},	/* Nemiah */
+	{X86_VENDOR_CENTAUR, 0x6, 0x9, ANY, 17},	/* C3-M */
+	{X86_VENDOR_UNKNOWN, ANY, ANY, ANY, 0}		/* stop here */
 };
 
-static int find_vrm(u8 eff_family, u8 eff_model, u8 vendor)
+static int find_vrm(u8 eff_family, u8 eff_model, u8 eff_stepping, u8 vendor)
 {
 	int i = 0;
 
@@ -139,7 +153,8 @@
 		if (vrm_models[i].vendor=vendor)
 			if ((vrm_models[i].eff_family=eff_family)
 			 && ((vrm_models[i].eff_model=eff_model) ||
-			     (vrm_models[i].eff_model=ANY)))
+			     (vrm_models[i].eff_model=ANY)) &&
+			     (eff_stepping <= vrm_models[i].eff_stepping))
 				return vrm_models[i].vrm_type;
 		i++;
 	}
@@ -147,12 +162,11 @@
 	return 0;
 }
 
-int vid_which_vrm(void)
+u8 vid_which_vrm(void)
 {
 	struct cpuinfo_x86 *c = cpu_data;
 	u32 eax;
-	u8 eff_family, eff_model;
-	int vrm_ret;
+	u8 eff_family, eff_model, eff_stepping, vrm_ret;
 
 	if (c->x86 < 6)		/* Any CPU with family lower than 6 */
 		return 0;	/* doesn't have VID and/or CPUID */
@@ -160,20 +174,23 @@
 	eax = cpuid_eax(1);
 	eff_family = ((eax & 0x00000F00)>>8);
 	eff_model  = ((eax & 0x000000F0)>>4);
+	eff_stepping = eax & 0xF;
 	if (eff_family = 0xF) {	/* use extended model & family */
 		eff_family += ((eax & 0x00F00000)>>20);
 		eff_model += ((eax & 0x000F0000)>>16)<<4;
 	}
-	vrm_ret = find_vrm(eff_family,eff_model,c->x86_vendor);
+	printk("CALLING WITH: %x %x %x %x\n", eff_family, eff_model, eff_stepping, c->x86_vendor);
+	vrm_ret = find_vrm(eff_family, eff_model, eff_stepping, c->x86_vendor);
+	printk("%d is back\n",vrm_ret);
 	if (vrm_ret = 0)
 		printk(KERN_INFO "hwmon-vid: Unknown VRM version of your "
 		       "x86 CPU\n");
 	return vrm_ret;
 }
 
-/* and now something completely different for the non-x86 world */
+/* and now for something completely different for the non-x86 world */
 #else
-int vid_which_vrm(void)
+u8 vid_which_vrm(void)
 {
 	printk(KERN_INFO "hwmon-vid: Unknown VRM version of your CPU\n");
 	return 0;
diff -Naur a/include/linux/hwmon-vid.h b/include/linux/hwmon-vid.h
--- a/include/linux/hwmon-vid.h	2005-10-28 02:02:08.000000000 +0200
+++ b/include/linux/hwmon-vid.h	2005-11-22 21:40:45.127484750 +0100
@@ -23,14 +23,14 @@
 #ifndef _LINUX_HWMON_VID_H
 #define _LINUX_HWMON_VID_H
 
-int vid_from_reg(int val, int vrm);
+int vid_from_reg(int val, u8 vrm);
 int vid_which_vrm(void);
 
 /* vrm is the VRM/VRD document version multiplied by 10.
    val is in mV to avoid floating point in the kernel.
    Returned value is the 4-, 5- or 6-bit VID code.
    Note that only VRM 9.x is supported for now. */
-static inline int vid_to_reg(int val, int vrm)
+static inline int vid_to_reg(int val, u8 vrm)
 {
 	switch (vrm) {
 	case 91:		/* VRM 9.1 */

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

Rudolf Marek wrote:
>>bogomips        : 1204.51
> 
> 
> This is OK for me now.
> 
> Please can you test attached patch? Because I cant see how it can fail.

I can try the patch this afternoon or later, gotta go for work now.
Thanks, I'll let you know!

Kind regards,
Udo

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

Rudolf Marek wrote:
> Please can you test attached patch? Because I cant see how it can fail.
When compiling I get:

# patch -p1 < /mnt/samba/download/linux/patches/via_vrm_patch_test7
(Stripping trailing CRs from patch.)
patching file drivers/hwmon/hwmon-vid.c
(Stripping trailing CRs from patch.)
patching file include/linux/hwmon-vid.h
[root@epia linux]# make bzImage modules modules_install install
  CHK     include/linux/version.h
  CHK     include/linux/compile.h
  CHK     usr/initramfs_list
  CC [M]  drivers/hwmon/hwmon-vid.o
drivers/hwmon/hwmon-vid.c:166: error: conflicting types for ?vid_which_vrm?
include/linux/hwmon-vid.h:27: error: previous declaration of
?vid_which_vrm? was here
make[2]: *** [drivers/hwmon/hwmon-vid.o] Error 1
make[1]: *** [drivers/hwmon] Error 2
make: *** [drivers] Error 2
[root@epia linux]#

This is on a 2.6.14.2 kernel.

Any ideas?
Maybe just put in the right code file (i.e.: replace)?

Kind regards,
Udo

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

Hello

Sorry for delay, I'm extra busy.
In attachment is patch that should compile.

Please apply it and see the system log for result. (and of course mail it back)

Thanks
Regards
Rudolf
-------------- next part --------------
diff -Naur a/drivers/hwmon/hwmon-vid.c b/drivers/hwmon/hwmon-vid.c
--- a/drivers/hwmon/hwmon-vid.c	2005-10-28 02:02:08.000000000 +0200
+++ b/drivers/hwmon/hwmon-vid.c	2005-11-22 21:41:51.327622000 +0100
@@ -49,20 +49,22 @@
         . . . .
        11110  =  0.800 V
        11111  =  0.000 V (off)
+
+    The 17 specification is in fact Intel Mobile Voltage Positioning -
+    (IMVP-II). You can find more information in the datasheet of Max1718
+    http://www.maxim-ic.com/quick_view2.cfm/qv_pk/2452
+
 */
 
 /* vrm is the VRM/VRD document version multiplied by 10.
    val is the 4-, 5- or 6-bit VID code.
    Returned value is in mV to avoid floating point in the kernel. */
-int vid_from_reg(int val, int vrm)
+int vid_from_reg(int val, u8 vrm)
 {
 	int vid;
 
 	switch(vrm) {
 
-	case  0:
-		return 0;
-
 	case 100:               /* VRD 10.0 */
 		if((val & 0x1f) = 0x1f)
 			return 0;
@@ -91,10 +93,16 @@
 	case 84:		/* VRM 8.4 */
 		val &= 0x0f;
 				/* fall through */
-	default:		/* VRM 8.2 */
+	case 82:		/* VRM 8.2 */
 		return(val = 0x1f ? 0 :
 		       val & 0x10  ? 5100 - (val) * 100 :
 		                     2050 - (val) * 50);
+	case 17:		/* Intel IMVP-II */
+		return(val & 0x10 ? 975 - (val & 0xF) * 25 :
+				    1750 - val * 50);
+	default:		/* report 0 for unknown */
+		printk(KERN_INFO "hwmon-vid: requested unknown VRM version\n");
+		return 0;
 	}
 }
 
@@ -108,30 +116,36 @@
 	u8 vendor;
 	u8 eff_family;
 	u8 eff_model;
-	int vrm_type;
+	u8 eff_stepping;
+	u8 vrm_type;
 };
 
 #define ANY 0xFF
 
 #ifdef CONFIG_X86
 
+/* the stepping parameter is highest acceptable stepping for current line */
+
 static struct vrm_model vrm_models[] = {
-	{X86_VENDOR_AMD, 0x6, ANY, 90},		/* Athlon Duron etc */
-	{X86_VENDOR_AMD, 0xF, ANY, 24},		/* Athlon 64, Opteron */
-	{X86_VENDOR_INTEL, 0x6, 0x9, 85},	/* 0.13um too */
-	{X86_VENDOR_INTEL, 0x6, 0xB, 85},	/* Tualatin */
-	{X86_VENDOR_INTEL, 0x6, ANY, 82},	/* any P6 */
-	{X86_VENDOR_INTEL, 0x7, ANY, 0},	/* Itanium */
-	{X86_VENDOR_INTEL, 0xF, 0x0, 90},	/* P4 */
-	{X86_VENDOR_INTEL, 0xF, 0x1, 90},	/* P4 Willamette */
-	{X86_VENDOR_INTEL, 0xF, 0x2, 90},	/* P4 Northwood */
-	{X86_VENDOR_INTEL, 0xF, 0x3, 100},	/* P4 Prescott */
-	{X86_VENDOR_INTEL, 0xF, 0x4, 100},	/* P4 Prescott */
-	{X86_VENDOR_INTEL, 0x10,ANY, 0},	/* Itanium 2 */
-	{X86_VENDOR_UNKNOWN, ANY, ANY, 0}	/* stop here */
+	{X86_VENDOR_AMD, 0x6, ANY, ANY, 90},		/* Athlon Duron etc */
+	{X86_VENDOR_AMD, 0xF, ANY, ANY, 24},		/* Athlon 64, Opteron and above VRM 24 */
+	{X86_VENDOR_INTEL, 0x6, 0x9, ANY, 85},		/* 0.13um too */
+	{X86_VENDOR_INTEL, 0x6, 0xB, ANY, 85},		/* Tualatin */
+	{X86_VENDOR_INTEL, 0x6, ANY, ANY, 82},		/* any P6 */
+	{X86_VENDOR_INTEL, 0x7, ANY, ANY, 0},		/* Itanium */
+	{X86_VENDOR_INTEL, 0xF, 0x0, ANY, 90},		/* P4 */
+	{X86_VENDOR_INTEL, 0xF, 0x1, ANY, 90},		/* P4 Willamette */
+	{X86_VENDOR_INTEL, 0xF, 0x2, ANY, 90},		/* P4 Northwood */
+	{X86_VENDOR_INTEL, 0xF, ANY, ANY, 100},		/* Prescott and above assume VRD 10 */
+	{X86_VENDOR_INTEL, 0x10, ANY, ANY, 0},		/* Itanium 2 */
+	{X86_VENDOR_CENTAUR, 0x6, 0x7, ANY, 85},	/* Eden ESP/Ezra */
+	{X86_VENDOR_CENTAUR, 0x6, 0x8, 0x7, 85},	/* Ezra T */
+	{X86_VENDOR_CENTAUR, 0x6, 0x9, 0x7, 85},	/* Nemiah */
+	{X86_VENDOR_CENTAUR, 0x6, 0x9, ANY, 17},	/* C3-M */
+	{X86_VENDOR_UNKNOWN, ANY, ANY, ANY, 0}		/* stop here */
 };
 
-static int find_vrm(u8 eff_family, u8 eff_model, u8 vendor)
+static int find_vrm(u8 eff_family, u8 eff_model, u8 eff_stepping, u8 vendor)
 {
 	int i = 0;
 
@@ -139,7 +153,8 @@
 		if (vrm_models[i].vendor=vendor)
 			if ((vrm_models[i].eff_family=eff_family)
 			 && ((vrm_models[i].eff_model=eff_model) ||
-			     (vrm_models[i].eff_model=ANY)))
+			     (vrm_models[i].eff_model=ANY)) &&
+			     (eff_stepping <= vrm_models[i].eff_stepping))
 				return vrm_models[i].vrm_type;
 		i++;
 	}
@@ -147,12 +162,11 @@
 	return 0;
 }
 
-int vid_which_vrm(void)
+u8 vid_which_vrm(void)
 {
 	struct cpuinfo_x86 *c = cpu_data;
 	u32 eax;
-	u8 eff_family, eff_model;
-	int vrm_ret;
+	u8 eff_family, eff_model, eff_stepping, vrm_ret;
 
 	if (c->x86 < 6)		/* Any CPU with family lower than 6 */
 		return 0;	/* doesn't have VID and/or CPUID */
@@ -160,20 +174,23 @@
 	eax = cpuid_eax(1);
 	eff_family = ((eax & 0x00000F00)>>8);
 	eff_model  = ((eax & 0x000000F0)>>4);
+	eff_stepping = eax & 0xF;
 	if (eff_family = 0xF) {	/* use extended model & family */
 		eff_family += ((eax & 0x00F00000)>>20);
 		eff_model += ((eax & 0x000F0000)>>16)<<4;
 	}
-	vrm_ret = find_vrm(eff_family,eff_model,c->x86_vendor);
+	printk("CALLING WITH: %x %x %x %x\n", eff_family, eff_model, eff_stepping, c->x86_vendor);
+	vrm_ret = find_vrm(eff_family, eff_model, eff_stepping, c->x86_vendor);
+	printk("%d is back\n",vrm_ret);
 	if (vrm_ret = 0)
 		printk(KERN_INFO "hwmon-vid: Unknown VRM version of your "
 		       "x86 CPU\n");
 	return vrm_ret;
 }
 
-/* and now something completely different for the non-x86 world */
+/* and now for something completely different for the non-x86 world */
 #else
-int vid_which_vrm(void)
+u8 vid_which_vrm(void)
 {
 	printk(KERN_INFO "hwmon-vid: Unknown VRM version of your CPU\n");
 	return 0;
diff -Naur a/include/linux/hwmon-vid.h b/include/linux/hwmon-vid.h
--- a/include/linux/hwmon-vid.h	2005-10-28 02:02:08.000000000 +0200
+++ b/include/linux/hwmon-vid.h	2005-12-03 23:16:27.186109750 +0100
@@ -23,14 +23,14 @@
 #ifndef _LINUX_HWMON_VID_H
 #define _LINUX_HWMON_VID_H
 
-int vid_from_reg(int val, int vrm);
-int vid_which_vrm(void);
+int vid_from_reg(int val, u8 vrm);
+u8 vid_which_vrm(void);
 
 /* vrm is the VRM/VRD document version multiplied by 10.
    val is in mV to avoid floating point in the kernel.
    Returned value is the 4-, 5- or 6-bit VID code.
    Note that only VRM 9.x is supported for now. */
-static inline int vid_to_reg(int val, int vrm)
+static inline int vid_to_reg(int val, u8 vrm)
 {
 	switch (vrm) {
 	case 91:		/* VRM 9.1 */

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

Rudolf Marek wrote:
> Sorry for delay, I'm extra busy.
> In attachment is patch that should compile.

Compiles fine, thanks!
Reboot scheduled for later today.

Kind regards,
Udo

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

No messages anymore!
How to test functionality?

Kind regards,
Udo

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

Hello,

You should see CALLING WITH in the log....

(it should appear when you load some chip driver)

Regards

Rudolf

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

> Now I got:
> 
> Dec  4 12:56:59 epia kernel: vt1211: Ignoring new-style parameters in
> presence of obsolete ones
> Dec  4 12:56:59 epia kernel: CALLING WITH: 6 7 3 5
> Dec  4 12:56:59 epia kernel: 85 is back
> 
> Is that enough info?
> At least it's no error?
> 
> Kind regards,
> Udo

Hmm so previous version must have worked too. I did not change anything in the code...
(except int->u8)

Anyway this is good. I will post new patch to Jean so it finaly will be in 2.6.16...

Regards
Rudolf

[lm-sensors] Re: hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Rudolf,

Rudolf Marek wrote:
>>Dec  4 12:56:59 epia kernel: vt1211: Ignoring new-style parameters in
>>presence of obsolete ones
>>Dec  4 12:56:59 epia kernel: CALLING WITH: 6 7 3 5
>>Dec  4 12:56:59 epia kernel: 85 is back
>>
>>Is that enough info?
>>At least it's no error?
> 
> Hmm so previous version must have worked too. I did not change anything in the code...
> (except int->u8)

I use this at the moment:
Linux epia 2.6.14.3-dbg #3 PREEMPT Sun Dec 4 07:45:24 CET 2005 i686 i686
i386 GNU/Linux

-dbg because of via rhine ethernet driver fix

> Anyway this is good. I will post new patch to Jean so it finaly will be in 2.6.16...

OK, would be interested to see this in the kernel.
Thank you!

Kind regards,
Udo

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello,

Same problem as some time ago when
Rudolf Marek wrote:
> Please apply it and see the system log for result. (and of course mail it back)

I now have a VIA Epia EN12000 which shows in dmesg:

hwmon-vid: Unknown VRM version of your x86 CPU

# cat /proc/cpuinfo
processor       : 0
vendor_id       : CentaurHauls
cpu family      : 6
model           : 10
model name      : VIA Esther processor 1200MHz
stepping        : 9
cpu MHz         : 1197.098
cache size      : 128 KB
fdiv_bug        : no
hlt_bug         : no
f00f_bug        : no
coma_bug        : no
fpu             : yes
fpu_exception   : yes
cpuid level     : 1
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge
cmov pat clflush acpi mmx fxsr sse sse2 tm pni est tm2 rng rng_en ace
ace_en ace2 ace2_en phe phe_en pmm pmm_en
bogomips        : 2396.86
clflush size    : 64


Need more info?

Thanks in advance!

Kind regards,
Udo

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Jean Delvare]

Hi Udo, 

On Sun, 25 Feb 2007 18:02:50 +0100, Udo van den Heuvel wrote:
> Hello,
> 
> Same problem as some time ago when
> Rudolf Marek wrote:
> > Please apply it and see the system log for result. (and of course mail it back)
> 
> I now have a VIA Epia EN12000 which shows in dmesg:
> 
> hwmon-vid: Unknown VRM version of your x86 CPU
> 
> # cat /proc/cpuinfo
> processor       : 0
> vendor_id       : CentaurHauls
> cpu family      : 6
> model           : 10
> model name      : VIA Esther processor 1200MHz
> stepping        : 9
> cpu MHz         : 1197.098
> cache size      : 128 KB
> fdiv_bug        : no
> hlt_bug         : no
> f00f_bug        : no
> coma_bug        : no
> fpu             : yes
> fpu_exception   : yes
> cpuid level     : 1
> wp              : yes
> flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge
> cmov pat clflush acpi mmx fxsr sse sse2 tm pni est tm2 rng rng_en ace
> ace_en ace2 ace2_en phe phe_en pmm pmm_en
> bogomips        : 2396.86
> clflush size    : 64

Indeed we do not support this CPU family at the moment.

> Need more info?

We'd need to know which VRM standard this CPU wants. Which hardware
monitoring chip and driver are you using? Can you try different VRM
standards and see if one appears to match the expected CPU core voltage?

-- 
Jean Delvare

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Udo van den Heuvel]

Hello Jean,

Jean Delvare wrote:
>> Same problem as some time ago when
>> Rudolf Marek wrote:
>>> Please apply it and see the system log for result. (and of course mail it back)
>> I now have a VIA Epia EN12000 which shows in dmesg:
>>
>> hwmon-vid: Unknown VRM version of your x86 CPU
>>
>> # cat /proc/cpuinfo
>> processor       : 0
>> vendor_id       : CentaurHauls
>> cpu family      : 6
>> model           : 10
>> model name      : VIA Esther processor 1200MHz
(....)
> Indeed we do not support this CPU family at the moment.

OK. It can be fixed?

>> Need more info?
> 
> We'd need to know which VRM standard this CPU wants. Which hardware
> monitoring chip and driver are you using? 

driver w83627hf
chip w83697hf

> Can you try different VRM
> standards and see if one appears to match the expected CPU core voltage?

EPIA-EN Operating Guide v.1.10.pdf says that this CPU uses 1.004v at 1.2
GHz. I measure  +0.83V, though.
How can I try a different VRM?


Kind regards,
Udo

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Jean Delvare]

Udo,

On Tue, 27 Feb 2007 17:08:11 +0100, Udo van den Heuvel wrote:
> Jean Delvare wrote:
> >> Same problem as some time ago when
> >> Rudolf Marek wrote:
> >>> Please apply it and see the system log for result. (and of course mail it back)
> >> I now have a VIA Epia EN12000 which shows in dmesg:
> >>
> >> hwmon-vid: Unknown VRM version of your x86 CPU
> >>
> >> # cat /proc/cpuinfo
> >> processor       : 0
> >> vendor_id       : CentaurHauls
> >> cpu family      : 6
> >> model           : 10
> >> model name      : VIA Esther processor 1200MHz
> (....)
> > Indeed we do not support this CPU family at the moment.
> 
> OK. It can be fixed?
>
> >> Need more info?
> > 
> > We'd need to know which VRM standard this CPU wants. Which hardware
> > monitoring chip and driver are you using? 
> 
> driver w83627hf
> chip w83697hf

Unfortunately this chip doesn't have VID pins, so we won't be able to
investigate.

OTOH, the good news is that it means that you can safely ignore the log
message.

-- 
Jean Delvare

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Jean Delvare]

> > I now have a VIA Epia EN12000 which shows in dmesg:
> >
> > hwmon-vid: Unknown VRM version of your x86 CPU
> >
> > # cat /proc/cpuinfo
> > processor       : 0
> > vendor_id       : CentaurHauls
> > cpu family      : 6
> > model           : 10
> > model name      : VIA Esther processor 1200MHz

Reading the document "EPIA-EN User's Manual v.1.10" from
http://www.via.com.tw/en/products/mainboards/downloads.jsp?motherboard_id99
there is a list of possible CPU core voltages which reminds me the VRM
table of the Intel Pentium M / Core. Rudolf, what do you think? What
about this patch:

Signed-off-by: Jean Delvare <khali at linux-fr.org>
---
 drivers/hwmon/hwmon-vid.c |    1 +
 1 file changed, 1 insertion(+)

--- linux-2.6.21-rc1.orig/drivers/hwmon/hwmon-vid.c	2007-02-04 19:44:54.000000000 +0100
+++ linux-2.6.21-rc1/drivers/hwmon/hwmon-vid.c	2007-02-27 18:40:05.000000000 +0100
@@ -176,6 +176,7 @@ static struct vrm_model vrm_models[] = {
 	{X86_VENDOR_CENTAUR, 0x6, 0x8, 0x7, 85},	/* Ezra T */
 	{X86_VENDOR_CENTAUR, 0x6, 0x9, 0x7, 85},	/* Nemiah */
 	{X86_VENDOR_CENTAUR, 0x6, 0x9, ANY, 17},	/* C3-M */
+	{X86_VENDOR_CENTAUR, 0x6, 0xA, ANY, 13},	/* Esther */
 	{X86_VENDOR_UNKNOWN, ANY, ANY, ANY, 0}		/* stop here */
 };
 


-- 
Jean Delvare

[lm-sensors] hwmon-vid: Unknown VRM version of your x86 CPU

[Rudolf Marek]

Hello all,

Jean, good catch. I check the Intel datasheet and lowest limit is also 0.7V so 
it must be this.

I also read the VIA datasheet of Eden CPU at 90nm (which core is called Esther)
http://www.via.com.tw/en/downloads/datasheets/processors/eden_90nm170.zip
                                                                                   .
The CPUID family 6 model A core of the VIA Eden is shared with other VIA 
processors. To identify these brand variants, use the MSR instruction to read 
register 1153h and use the Exclusive OR (XOR)
function between bit fields 21:20 and 19:18.

Also:

VIA Eden Type:0  Fam: 6  Model: 10 Stepping: Begins at 9

Btw the datasheet contains also the Table which is identical to Intel one (page 26)

I will check the other processor not in the table and prepare a patch.

Rudolf