linux-toradex.git/drivers/hwmon, branch v3.1.9

hwmon: (jz4740) fix signedness bug

2011-12-21T20:58:31+00:00

commit 0b57d7602b68f7b2786b2f0e22da39cbd4139a95 upstream.

wait_for_completion_interruptible_timeout() may return negative value.
In this case, checking if (t > 0)  will return true if t is unsigned.

Signed-off-by: Axel Lin 
Acked-by: Lars-Peter Clausen 
Signed-off-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

hwmon: (w83627ehf) Fix broken driver init

2011-11-11T17:44:14+00:00

commit bfa02b0da66965caf46e441270af87edda4fea14 upstream.

Commit 2265cef2 (hwmon: (w83627ehf) Properly report PECI and AMD-SI
sensor types) results in kernel panic if data->temp_label was not
initialized.
The problem was found with chip W83627DHG-P.

Add check if data->temp->label was set before use.

Based on incomplete patch by Alexander Beregalov.

Reported-by: Alexander Beregalov 
Tested-by: Alexander Beregalov 
Signed-off-by: Guenter Roeck 
Signed-off-by: Jean Delvare 
Signed-off-by: Greg Kroah-Hartman

hwmon: (w83627ehf) Properly report PECI and AMD-SI sensor types

2011-11-11T17:44:14+00:00

commit 2265cef2751b3441df91f85e0107f9f549e5b711 upstream.

When temperature sources are PECI or AMD-SI agents, it makes no sense
to report their type as diode or thermistor. Instead we must report
their digital nature.

Signed-off-by: Jean Delvare 
Acked-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

hwmon: (coretemp) Fix for non-SMP builds

2011-11-11T17:44:13+00:00

commit 2aba6cac2a84f3b80e11a680c34d55e7739b474d upstream.

The definition of TO_ATTR_NO in the non-SMP case is wrong. As the SMP
definition resolves to the correct value, just use this for both
cases.

Without this fix the temperature attributes are named temp0_* instead
of temp2_*, so libsensors won't pick them. Broken since kernel 3.0.

Signed-off-by: Jean Delvare 
Tested-by: Phil Sutter 
Acked-by: Durgadoss R 
Acked-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

hwmon: (w83627ehf) Fix negative 8-bit temperature values

2011-10-20T14:17:08+00:00

Since 8-bit temperature values are now handled in 16-bit struct
members, values have to be cast to s8 for negative temperatures to be
properly handled. This is broken since kernel version 2.6.39
(commit bce26c58df86599c9570cee83eac58bdaae760e4.)

Signed-off-by: Jean Delvare 
Cc: Guenter Roeck 
Cc: stable@kernel.org	# 2.6.39+
Signed-off-by: Guenter Roeck

hwmon: (w83627ehf) Properly report thermal diode sensors

2011-10-13T23:51:29+00:00

The w83627ehf driver is improperly reporting thermal diode sensors as
type 2, instead of 3. This caused "sensors" and possibly other
monitoring tools to report these sensors as "transistor" instead of
"thermal diode".

Furthermore, diode subtype selection (CPU vs. external) is only
supported by the original W83627EHF/EHG. All later models only support
CPU diode type, and some (NCT6776F) don't even have the register in
question so we should avoid reading from it.

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

hwmon: (coretemp) Avoid leaving around dangling pointer

2011-09-28T15:19:21+00:00

Storing the struct temp_data pointer allocated from create_core_data()
when returning an error has the potential of leaving around a pointer
to freed memory. Reset it to NULL for error returns.

Reported-by: Jan Beulich 
Signed-off-by: Guenter Roeck 
Acked-by: Jean Delvare

hwmon: (coretemp) Fixup platform device ID change

2011-09-28T15:19:20+00:00

With recent change "hwmon: (coretemp) don't use kernel assigned CPU
number as platform device ID", the microcode check is now running on
random CPU. Fix that by checking the microcode before creating the
platform device rather than at probe time.

Also avoid calling TO_PHYS_ID(cpu) twice in the same function, it's
expensive.

Signed-off-by: Jean Delvare 
Cc: Jan Beulich 
Cc: Guenter Roeck 
Signed-off-by: Guenter Roeck

hwmon: (coretemp) remove struct platform_data * parameter from create_core_data()

2011-09-23T17:39:11+00:00

The only caller of the function obtained the pointer solely for the
purpose of passing it to this function, while it can be easily
determined from the struct platform_device * parameter also passed.

Signed-off-by: Jan Beulich 
Signed-off-by: Guenter Roeck

hwmon: (coretemp) constify static data

2011-09-23T17:39:10+00:00

These arrays won't ever be written to, so protect them from
unintentional modification.

Signed-off-by: Jan Beulich 
Signed-off-by: Guenter Roeck