All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH][2.6.0-test2]adm1021 i2c driver bug fix
@ 2003-08-06 20:17 Rusty Lynch
  0 siblings, 0 replies; only message in thread
From: Rusty Lynch @ 2003-08-06 20:17 UTC (permalink / raw)
  To: Philip Edelbrock, Frodo Looijaard; +Cc: lkml

While initializing the adm1021 device, the driver is performing a conversion 
from fixed point to Celcius on values that were declaired as Celcius.  On 
my Dell Precision 220 this results in a shutdown after a couple of minutes
running.

This patch was made against the 2.6.0-test2 tree, and just removes the 
conversion.

    --rustyl

--- drivers/i2c/chips/adm1021.c.orig	2003-08-06 13:04:25.000000000 -0700
+++ drivers/i2c/chips/adm1021.c	2003-08-06 13:09:43.000000000 -0700
@@ -356,13 +356,13 @@
 {
 	/* Initialize the adm1021 chip */
 	adm1021_write_value(client, ADM1021_REG_TOS_W,
-			    TEMP_TO_REG(adm1021_INIT_TOS));
+			    adm1021_INIT_TOS);
 	adm1021_write_value(client, ADM1021_REG_THYST_W,
-			    TEMP_TO_REG(adm1021_INIT_THYST));
+			    adm1021_INIT_THYST);
 	adm1021_write_value(client, ADM1021_REG_REMOTE_TOS_W,
-			    TEMP_TO_REG(adm1021_INIT_REMOTE_TOS));
+			    adm1021_INIT_REMOTE_TOS);
 	adm1021_write_value(client, ADM1021_REG_REMOTE_THYST_W,
-			    TEMP_TO_REG(adm1021_INIT_REMOTE_THYST));
+			    adm1021_INIT_REMOTE_THYST);
 	/* Enable ADC and disable suspend mode */
 	adm1021_write_value(client, ADM1021_REG_CONFIG_W, 0);
 	/* Set Conversion rate to 1/sec (this can be tinkered with) */




^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2003-08-06 20:31 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-08-06 20:17 [PATCH][2.6.0-test2]adm1021 i2c driver bug fix Rusty Lynch

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.