From c6b95bf7e3bd527a79a2bbc8b517396cf5900347 Mon Sep 17 00:00:00 2001 From: "Mark M. Hoffman" Date: Thu, 10 Mar 2005 03:45:34 +0000 Subject: [PATCH] Patch by David Knierim to fix IN10 (+5V) scaling > I have been hunting down an issue related to the IN10 input of the > LM93. I have observed that there is a rather large error between the > actual voltage and the voltage level displayed by sensors(1) (~ .12V). > > Please find attached a small patch that fixes this problem. The data > sheet is very ambiguous and I and another engineer made the same mistake > calculating the voltage per DAC step the first time we read the spec sheet. > > The confusion stems from an entry in the "maximum voltage" column that > is higher than the value in the "absolute maximum range" column. For 5V > only, the value in the "register reading at maximum voltage" is for the > voltage in the "absolute maximum range", NOT the "maximum voltage". Whew! git-svn-id: http://lm-sensors.org/svn/lm-sensors/trunk@2924 7894878c-1315-0410-8ee3-d5d059ff63e0 --- CHANGES | 1 + kernel/chips/lm93.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index 5c88fb2f..1f857ec0 100644 --- a/CHANGES +++ b/CHANGES @@ -35,6 +35,7 @@ ask CVS about it: Limit I2C address range Support VID on the IT8712F Module lm90: Fix MAX6657, MAX6658 and MAX6659 detection + Module lm93: Fix IN10 (+5V) scaling Module mic74: New Module w83781d: Revert AS99127F temp2 and temp3 computation change Skip final initialization step for AS99127F chips diff --git a/kernel/chips/lm93.c b/kernel/chips/lm93.c index aaa64955..93ab6dd4 100644 --- a/kernel/chips/lm93.c +++ b/kernel/chips/lm93.c @@ -689,7 +689,7 @@ static const unsigned long lm93_vin_val_min[16] = { }; static const unsigned long lm93_vin_val_max[16] = { 1236, 1236, 1236, 1600, 2000, 2000, 1600, 1600, - 4400, 6667, 3333, 2625, 1312, 1312, 1236, 3600, + 4400, 6500, 3333, 2625, 1312, 1312, 1236, 3600, }; /* static const unsigned long lm93_vin_val_nom[16] = {