linux-toradex.git/drivers/hwmon/adm1031.c, branch v3.17

hwmon: (adm1031) Convert to devm_hwmon_device_register_with_groups

2014-08-04T14:01:37+00:00

Use ATTRIBUTE_GROUPS macro and devm_hwmon_device_register_with_groups() to
simplify the code a bit.

Signed-off-by: Axel Lin 
Signed-off-by: Guenter Roeck

hwmon: (adm1031) Avoid forward declaration

2014-08-04T14:01:37+00:00

Reorder functions to avoid forward declaration.

Signed-off-by: Axel Lin 
Signed-off-by: Guenter Roeck

hwmon: (adm1031) Fix writes to limit registers

2014-07-07T12:49:25+00:00

Upper limit for write operations to temperature limit registers
was clamped to a fractional value. However, limit registers do
not support fractional values. As a result, upper limits of 127.5
degrees C or higher resulted in a rounded limit of 128 degrees C.
Since limit registers are signed, this was stored as -128 degrees C.
Clamp limits to (-55, +127) degrees C to solve the problem.

Value on writes to auto_temp[12]_min and auto_temp[12]_max were not
clamped at all, but masked. As a result, out-of-range writes resulted
in a more or less arbitrary limit. Clamp those attributes to (0, 127)
degrees C for more predictable results.

Cc: Axel Lin 
Cc: stable@vger.kernel.org
Reviewed-by: Jean Delvare 
Signed-off-by: Guenter Roeck

Update Jean Delvare's e-mail address

2014-01-29T19:40:08+00:00

Signed-off-by: Jean Delvare

hwmon: Replace SENSORS_LIMIT with clamp_val

2013-01-26T05:03:54+00:00

SENSORS_LIMIT and the generic clamp_val have the same functionality,
and clamp_val is more efficient.

This patch reduces text size by 9052 bytes and bss size by 11624 bytes
for x86_64 builds.

Signed-off-by: Guenter Roeck 
Acked-by: George Joseph 
Acked-by: Jean Delvare

hwmon: (adm1031) Convert to use devm_ functions

2012-07-22T04:48:39+00:00

Convert to use devm_ functions to reduce code size and simplify the code.

Signed-off-by: Guenter Roeck 
Acked-by: Jean Delvare

hwmon: (adm1031) Fix compiler warning

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

Some configurations produce the following compile warning:

drivers/hwmon/adm1031.c: In function 'set_fan_auto_channel':
drivers/hwmon/adm1031.c:292: warning: 'reg' may be used uninitialized in this function

While this is a false positive, it can easily be fixed by overloading the return
value from get_fan_auto_nearest with both register value and error return code
(the register value is never negative). Coincidentially, that also reduces
module size by a few bytes.

Signed-off-by: Guenter Roeck 
Acked-by: Jean Delvare

hwmon: (adm1031) Fix multi-line comments

2012-03-19T01:27:33+00:00

Signed-off-by: Guenter Roeck

hwmon: convert drivers/hwmon/* to use module_i2c_driver()

2012-03-19T01:26:50+00:00

This patch converts the drivers in drivers/hwmon/* to use the
module_i2c_driver() macro which makes the code smaller and a bit simpler.

Signed-off-by: Axel Lin 
Cc: Corentin Labbe 
Cc: Dirk Eibach 
Cc: "Mark M. Hoffman" 
Cc: Steve Glendinning 
Cc: Riku Voipio 
Cc: Guillaume Ligneul 
Cc: David George 
Cc: "Hans J. Koch" 
Cc: Marc Hulsman 
Cc: Rudolf Marek 
Signed-off-by: Guenter Roeck

hwmon: (adm1031) Fix coding style issues

2012-01-16T21:51:48+00:00

Fix almost all coding style issues except for the multi-line macro errors,
which do not really apply since the macros are not multi-line statements
but declarations.

Based on merged patch series from Zac Storer; fixed remaining checkpatch
errors and warnings.

Cc: Zac Storer 
Signed-off-by: Guenter Roeck 
Signed-off-by: Jean Delvare