linux.git/drivers/hwmon, branch v3.4-rc5

hwmon: (fam15h_power) Fix pci_device_id array

2012-04-26T15:40:08+00:00

pci_match_id() takes an *array* of IDs which must be properly zero-
terminated.

Reported-by: Ben Hutchings 
Cc: stable@vger.kernel.org # 3.0+: 00250ec hwmon: fam15h_power: fix bogus values
Cc: stable@vger.kernel.org # 3.0+
Signed-off-by: Guenter Roeck 
Acked-by: Jean Delvare

hwmon: (ad7314) Fix build warning

2012-04-23T01:22:53+00:00

The following build warning is seen in some configurations.

drivers/hwmon/ad7314.c: In function 'ad7314_show_temperature':
drivers/hwmon/ad7314.c:70: warning: 'data' may be used uninitialized in this function

Fix by overloading the return value from ad7314_spi_read with both data and
error code (the returned data is really u16 and needs to be converted into a
signed value anyway).

Signed-off-by: Guenter Roeck 
Cc: Jonathan Cameron 
Acked-by: Jean Delvare

hwmon: (ads1015) Fix build warning

2012-04-18T18:15:50+00:00

The following build warning is seen in some configurations.

drivers/hwmon/ads1015.c: In function 'show_in':
drivers/hwmon/ads1015.c:129: warning: 'in' may be used uninitialized in this function

Fix by separating the register read function from the code converting the result
into mV.

Signed-off-by: Guenter Roeck 
Cc: Dirk Eibach 
Reviewed-by: Robert Coulson

hwmon: fam15h_power: fix bogus values with current BIOSes

2012-04-18T18:15:40+00:00

Newer BKDG[1] versions recommend a different initialization value for
the running average range register in the northbridge. This improves
the power reading by avoiding counter saturations resulting in bogus
values for anything below about 80% of TDP power consumption.
Updated BIOSes will have this new value set up from the beginning,
but meanwhile we correct this value ourselves.
This needs to be done on all northbridges, even on those where the
driver itself does not register at.

This fixes the driver on all current machines to provide proper
values for idle load.

[1]
http://support.amd.com/us/Processor_TechDocs/42301_15h_Mod_00h-0Fh_BKDG.pdf
Chapter 3.8: D18F5xE0 Processor TDP Running Average (p. 452)

Signed-off-by: Andre Przywara 
Acked-by: Jean Delvare 
[guenter.roeck@ericsson.com: Removed unnecessary return statement]
Signed-off-by: Guenter Roeck 
Cc: stable@vger.kernel.org # 3.0+

hwmon: (pmbus_core) Fix compiler warning

2012-04-09T19:17:38+00:00

Some configurations produce the following compiler warning:

drivers/hwmon/pmbus/pmbus_core.c: In function 'pmbus_show_boolean':
drivers/hwmon/pmbus/pmbus_core.c:752: warning: 'val' may be used uninitialized in this function

While this is a false positive, it can easily be fixed by overloading the return
value from pmbus_get_boolean with both val and error return code (val is a
boolean and thus never negative).

Signed-off-by: Guenter Roeck 
Reviewed-by: Robert Coulson

hwmon: (smsc47m1) Fix compiler warning

2012-04-09T19:17:38+00:00

Some configurations produce the following compiler warning:

drivers/hwmon/smsc47m1.c: In function 'sm_smsc47m1_init':
drivers/hwmon/smsc47m1.c:938: warning: 'address' may be used uninitialized in this function

While this is a false positive, it can easily be fixed by overloading the return
value from smsc47m1_find with both address and error return code (the address
is an unsigned short and thus never negative). This also reduces module size by
a few bytes (46 bytes for x86_64).

Signed-off-by: Guenter Roeck 
Reviewed-by: Robert Coulson

hwmon: (acpi_power_meter) Fix compiler warning seen in some configurations

2012-04-09T19:17:37+00:00

In some configurations, BUG() does not result in an endless loop but returns
to the caller. This results in the following compiler warning:

drivers/hwmon/acpi_power_meter.c: In function 'show_str':
drivers/hwmon/acpi_power_meter.c:380: warning: 'val' may be used uninitialized in this function

Fix the warning by setting val to an empty string after BUG().

Signed-off-by: Guenter Roeck 
Reviewed-by: Robert Coulson

hwmon: (smsc47b397) Fix compiler warning

2012-04-09T19:17:36+00:00

Some configurations produce the following compiler warning:

drivers/hwmon/smsc47b397.c: In function 'smsc47b397_init':
drivers/hwmon/smsc47b397.c:385: warning: 'address' may be used uninitialized in this function

While this is a false positive, it can easily be fixed by overloading the return
value from smsc47b397_find with both address and error return code (the address
is an unsigned short and thus never negative). This also reduces module size by
a few bytes (64 bytes for x86_64).

Cc: Mark M. Hoffman 
Signed-off-by: Guenter Roeck 
Reviewed-by: Robert Coulson 
Acked-by: Jean Delvare

hwmon: (ad7314) Adds missing spi_dev initialization

2012-04-04T00:08:28+00:00

This driver was recently moved from IIO (where it worked) to hwmon (where
it doesn't.) This breakage occured because the hwmon version neglected to
correctly initialize a reference to spi_dev in its drvdata. The result is a
segfault every time the temperature is queried.

Signed-off-by: Graeme Smecher 
Cc: stable@vger.kernel.org # 3.2+
Signed-off-by: Guenter Roeck

hwmon: (k10temp) Add support for AMD Trinity CPUs

2012-04-01T17:25:56+00:00

The on-chip northbridge's temperature sensor of the upcoming
AMD Trinity CPUs works the same as for the previous CPUs.
Since it has a different PCI-ID, we just add the new one to the list
supported by k10temp.
This allows to use the k10temp driver on those CPUs.

Signed-off-by: Andre Przywara 
Cc: stable@vger.kernel.org # 3.0+
Signed-off-by: Guenter Roeck