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

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

hwmon: (w83627ehf) mark const init data with initconst instead of initdata

2012-04-01T17:24:36+00:00

As long as there is no other non-const variable marked __initdata in the
same compilation unit it doesn't hurt. If there were one however
compilation would fail with

	error: $variablename causes a section type conflict

because a section containing const variables is marked read only and so
cannot contain non-const variables.

Signed-off-by: Uwe Kleine-König 
Cc: Jean Delvare 
Cc: Guenter Roeck 
Cc: lm-sensors@lm-sensors.org
Signed-off-by: Guenter Roeck

hwmon: (acpi_power_meter) fix lockdep spew due to non-static lock class