Module Name:    src
Committed By:   jmcneill
Date:           Fri Nov 21 22:31:09 UTC 2014

Modified Files:
        src/sys/dev/i2c: axp22x.c

Log Message:
Change internal temperature base from -267.7C to -243.7C. The AXP223
datasheet says that the value is between -267.7C and 165.8C in steps of 0.1C,
but this doesn't make sense for a 12-bit field. If we take the maximum value
and subtract 4095 we get -243.7C, effectively adding +24C to the reported
temp. On my A31 board, this changes a chilly 15.3C to a more reasonable 39.3C.


To generate a diff of this commit:
cvs rdiff -u -r1.1 -r1.2 src/sys/dev/i2c/axp22x.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/i2c/axp22x.c
diff -u src/sys/dev/i2c/axp22x.c:1.1 src/sys/dev/i2c/axp22x.c:1.2
--- src/sys/dev/i2c/axp22x.c:1.1	Sun Oct 12 23:58:42 2014
+++ src/sys/dev/i2c/axp22x.c	Fri Nov 21 22:31:09 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: axp22x.c,v 1.1 2014/10/12 23:58:42 jmcneill Exp $ */
+/* $NetBSD: axp22x.c,v 1.2 2014/11/21 22:31:09 jmcneill Exp $ */
 
 /*-
  * Copyright (c) 2014 Jared D. McNeill <jmcne...@invisible.ca>
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: axp22x.c,v 1.1 2014/10/12 23:58:42 jmcneill Exp $");
+__KERNEL_RCSID(0, "$NetBSD: axp22x.c,v 1.2 2014/11/21 22:31:09 jmcneill Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -108,8 +108,8 @@ axp22x_sensors_refresh(struct sysmon_env
 	if (error) {
 		edata->state = ENVSYS_SINVALID;
 	} else {
-		/* between -267.7C and 165.8C, step +0.1C */
-		edata->value_cur = (((buf[0] << 4) | (buf[1] & 0xf)) - 2677)
+		/* between -243.7C and 165.8C, step +0.1C */
+		edata->value_cur = (((buf[0] << 4) | (buf[1] & 0xf)) - 2437)
 				   * 100000 + 273150000;
 		edata->state = ENVSYS_SVALID;
 	}

Reply via email to