Skip to contentSkip to conversation

coretemp: Add Penryn CPU to coretemp

PATCH In review Rudolf Marek 3 participants 3 messages Jan 17, 2008 – Feb 18, 2008

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 coretemp: Add Penryn CPU to coretemp

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello,

This patch adds support for family 0x17, which has Penryn Core. It should also cover the 8 cores Xeons.

Can someone test please? I think it should work.

Thanks,
Rudolf
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFHj+ms3J9wPJqZRNURAqKoAJwLGJSgU2k1ke0Gn3550qMaWfIyGQCgimyV Fl8HRd1y4ecNQ7SqSMx3RL0=
=8BPU
-----END PGP SIGNATURE-----

drivers/hwmon/coretemp.c +2 -2
Index: linux-2.6.24-rc7/drivers/hwmon/coretemp.c
===================================================================
@@ -414,10 +414,10 @@
 	for_each_online_cpu(i) {
 		struct cpuinfo_x86 *c = &cpu_data(i);
 
-		/* check if family 6, models e, f, 16 */
+		/* check if family 6, models 0xe, 0xf, 0x16, 0x17 */
 		if ((c->cpuid_level < 0) || (c->x86 != 0x6) ||
 		    !((c->x86_model == 0xe) || (c->x86_model == 0xf) ||
-			(c->x86_model == 0x16))) {
+			(c->x86_model == 0x16) || (c->x86_model == 0x17))) {
 
 			/* supported CPU not found, but report the unknown
 			   family 6 CPU */
Documentation/hwmon/coretemp +2 -1
Index: linux-2.6.24-rc7/Documentation/hwmon/coretemp
===================================================================
@@ -4,9 +4,10 @@
 Supported chips:
   * All Intel Core family
     Prefix: 'coretemp'
-    CPUID: family 0x6, models 0xe, 0xf, 0x16
+    CPUID: family 0x6, models 0xe, 0xf, 0x16, 0x17
     Datasheet: Intel 64 and IA-32 Architectures Software Developer's Manual
                Volume 3A: System Programming Guide
+               http://softwarecommunity.intel.com/Wiki/Mobility/720.htm
 
 Author: Rudolf Marek
 

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Acked-by on v1 1/1
#1 lore raw

Hi Rudolf,

Quoted 4 lines · Rudolf Marek

This patch adds support for family 0x17, which has Penryn Core. It should also cover the 8 cores Xeons.

Can someone test please? I think it should work.

I only have a T2600 (Core Duo), I confirm that there is no regression with this patch.

Quoted 2 lines

Review: looks OK to me.

Signature · 6 lines
Jean Delvare

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
#2 lore raw
In reply to Rudolf Marek

Hi Rudolf:

Quoted 12 lines · Jean Delvare

Hi Rudolf,

Quoted 4 lines · Rudolf Marek

This patch adds support for family 0x17, which has Penryn Core. It should also cover the 8 cores Xeons.

Can someone test please? I think it should work.

I only have a T2600 (Core Duo), I confirm that there is no regression with this patch.

Quoted 1 line
Quoted 5 lines
Quoted 1 line

Review: looks OK to me.

Applied to hwmon-2.6.git/testing, thanks.

Signature · 8 lines
Mark M. Hoffman
[email protected]


_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors