On 2007/04/28 01:29, Stuart Henderson wrote:
> it looks like you have W83627EHF, it seems from list posts that
> there are versions with various IDs (0x88, 0xa1, 0xc1). The 0xa1
> devices are already supported, it's possibly just a case of

ah, hang on, I just found the datasheet. there are two different
chip id registers, the 0xa1 is for the other one. it looks like the
0xa1 -> 0x88 change is likely to be the correct fix unless someone
already has an W83627EHF detecting correctly.

> adding the other IDs. You could try replacing 0xa1 with 0x88
> in /sys/dev/ic/lm78var.h and report back whether hw.sensors
> matches up with the bios readout.

here is a patch to do this.

Index: dev/ic/lm78var.h
===================================================================
RCS file: /cvs/src/sys/dev/ic/lm78var.h,v
retrieving revision 1.11
diff -u -p -r1.11 lm78var.h
--- dev/ic/lm78var.h    22 Mar 2007 16:55:31 -0000      1.11
+++ dev/ic/lm78var.h    28 Apr 2007 00:42:46 -0000
@@ -111,8 +111,8 @@
 #define WB_CHIPID_W83791SD     0x72
 #define WB_CHIPID_W83792D      0x7a
 #define WB_CHIPID_W83637HF     0x80
+#define WB_CHIPID_W83627EHF    0x88
 #define WB_CHIPID_W83627THF    0x90
-#define WB_CHIPID_W83627EHF    0xa1
 
 /* Config bits */
 #define WB_CONFIG_VMR9         0x01

Reply via email to