linux-toradex.git/drivers/hwmon, branch v2.6.33.4

hwmon: (sht15) Properly handle the case CONFIG_REGULATOR=n

2010-04-26T14:47:55+00:00

commit c7a78d2c2e2537fd24903e966f34aae50319d587 upstream.

When CONFIG_REGULATOR isn't set, regulator_get_voltage() returns 0.
Properly handle this case by not trusting the value.

Reported-by: Jerome Oufella 
Signed-off-by: Jean Delvare 
Cc: Jonathan Cameron 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

hwmon: (sht15) Fix sht15_calc_temp interpolation function

2010-04-26T14:47:54+00:00

commit 328a2c22abd08911e37fa66f1358f829cecd72e9 upstream.

I discovered two issues.
First the previous sht15_calc_temp() loop did not iterate through the
temppoints array since the (data->supply_uV > temppoints[i - 1].vdd)
test is always true in this direction.

Also the two-points linear interpolation function was returning biased
values due to a stray division by 1000 which shouldn't be there.

[JD: Also change the default value for d1 from 0 to something saner.]

Signed-off-by: Jerome Oufella 
Acked-by: Jonathan Cameron 
Signed-off-by: Jean Delvare 
Signed-off-by: Greg Kroah-Hartman

hwmon: (coretemp) Add missing newline to dev_warn() message

2010-04-01T23:01:55+00:00

commit 4d7a5644e4adfafe76c2bd8ee168e3f3b5dae3a8 upstream.

Add missing newline to dev_warn() message string. This is more of an issue
with older kernels that don't automatically add a newline if it was missing
from the end of the previous line.

Signed-off-by: Dean Nelson 
Signed-off-by: Jean Delvare 
Signed-off-by: Greg Kroah-Hartman

hwmon: Fix off-by-one kind values

2010-03-15T16:06:33+00:00

commit dc71afe5ac7e8d049bb991330518e4c898a7d92e upstream.

Recent changes on the I2C front have left off-by-one array indexes in
3 hwmon drivers. Fix them.

Faulty commit:
e5e9f44c2 i2c: Drop I2C_CLIENT_INSMOD_2 to 8

Reported-by: Dan Carpenter 
Signed-off-by: Jean Delvare 
Cc: Hans de Goede 
Cc: Andre Prendel 
Signed-off-by: Greg Kroah-Hartman

hwmon: (tmp421) Restore missing inputs

2010-03-15T16:06:33+00:00

commit 8d59582a867470a3e0c3eced4a01625ae8dc546b upstream.

An off-by-one error caused some inputs to not be created by the driver
when they should. TMP421 gets only one input instead of two, TMP422
gets two instead of three, etc. Fix the bug by listing explicitly the
number of inputs each device has.

Signed-off-by: Jean Delvare 
Tested-by: Andre Prendel 
Signed-off-by: Greg Kroah-Hartman

hwmon: (tmp421) Fix temperature conversions

2010-03-15T16:06:32+00:00

commit a44908d742a577fb5ccb9a8c082326d4cea234c2 upstream.

The low bits of temperature registers are status bits, they must be
masked out before converting the register values to temperatures.

Signed-off-by: Jean Delvare 
Tested-by: Andre Prendel 
Signed-off-by: Greg Kroah-Hartman

macintosh/hwmon/ams: Fix device removal sequence

2010-03-15T16:06:12+00:00

commit 98ceb75c7c14eada76b0aa9f03a635a735cee3cb upstream.

Some code that is in ams_exit() (the module exit code) should instead
be called when the device (not module) is removed. It probably doesn't
make much of a difference in the PMU case, but in the I2C case it does
matter.

I make no guarantee that my fix isn't racy, I'm not familiar enough
with the ams driver code to tell for sure.

Signed-off-by: Jean Delvare 
Tested-by: Christian Kujau 
Cc: Benjamin Herrenschmidt 
Cc: Stelian Pop 
Cc: Michael Hanselmann 
Signed-off-by: Benjamin Herrenschmidt 
Signed-off-by: Greg Kroah-Hartman

hwmon: (w83781d) Request I/O ports individually for probing

2010-02-05T18:58:36+00:00

Different motherboards have different PNP declarations for
W83781D/W83782D chips. Some declare the whole range of I/O ports (8
ports), some declare only the useful ports (2 ports at offset 5) and
some declare fancy ranges, for example 4 ports at offset 4. To
properly handle all cases, request all ports individually for probing.
After we have determined that we really have a W83781D or W83782D
chip, the useful port range will be requested again, as a single
block.

I did not see a board which needs this yet, but I know of one for lm78
driver and I'd like to keep the logic of these two drivers in sync.

Signed-off-by: Jean Delvare 
Cc: stable@kernel.org

hwmon: (lm78) Request I/O ports individually for probing

2010-02-05T18:58:36+00:00

Different motherboards have different PNP declarations for LM78/LM79
chips. Some declare the whole range of I/O ports (8 ports), some
declare only the useful ports (2 ports at offset 5) and some declare
fancy ranges, for example 4 ports at offset 4. To properly handle all
cases, request all ports individually for probing. After we have
determined that we really have an LM78 or LM79 chip, the useful port
range will be requested again, as a single block.

This fixes the driver on the Olivetti M3000 DT 540, at least.

Signed-off-by: Jean Delvare 
Cc: stable@kernel.org

hwmon: (adt7462) Wrong ADT7462_VOLT_COUNT

2010-02-05T18:58:35+00:00

The #define ADT7462_VOLT_COUNT is wrong, it should be 13 not 12. All the 
for loops that use this as a limit count are of the typical form, "for 
(n = 0; n < ADT7462_VOLT_COUNT; n++)", so to loop through all voltages 
w/o missing the last one it is necessary for the count to be one greater 
than it is.  (Specifically, you will miss the +1.5V 3GPIO input with count 
= 12 vs. 13.)

Signed-off-by: Ray Copeland 
Acked-by: "Darrick J. Wong" 
Signed-off-by: Jean Delvare 
Cc: stable@kernel.org