Patch by David Knierim <david_knierim@earthlink.net> 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
This commit is contained in:
Mark M. Hoffman committed 2005-03-10 03:45:34 +00:00
1 parent 49aa0ea2b7
commit c6b95bf7e3
2 files changed
+2 -1

No files matched your search

+1
View File
@@ -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
+1 -1
View File
@@ -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] = {