linux-toradex.git/drivers/hwmon/w83791d.c, branch v3.15.6

hwmon: Correct some typos

2013-10-18T16:12:02+00:00

Signed-off-by: LABBE Corentin 
Signed-off-by: Guenter Roeck

hwmon: Fix checkpatch warning 'quoted string split across lines'

2013-04-08T04:16:40+00:00

Cc: Corentin Labbe 
Cc: Mark M. Hoffman 
Cc: Fenghua Yu 
Cc: Juerg Haefliger 
Cc: Andreas Herrmann 
Cc: Rudolf Marek 
Cc: Jim Cromie 
Cc: Roger Lucas 
Cc: Marc Hulsman 
Signed-off-by: Guenter Roeck

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: Fix chip feature table headers

2012-11-05T20:54:40+00:00

These got broken by recent patches fixing checkpatch warnings in these
drivers. The trick is that the patches themselves looked good, but the
source files after applying them do not. That's why I am not a big fan
of using tabs inside comments.

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

hwmon: Add missing inclusions of

2012-10-10T13:25:56+00:00

Many hwmon drivers use jiffies but omit the inclusion of the header
file. Fix that, and also fix one driver which was including the header
file but didn't need it.

Signed-off-by: Jean Delvare 
Acked-by: Guenter Roeck 
Acked-by: Luca Tettamanti 
Cc: Marc Hulsman 
Cc: Rudolf Marek

hwmon: (w83791d) Drop unnecessary compare statements

2012-09-24T04:08:35+00:00

The following build warnings are seen with -Wextra.

w83791d.c: In function store_temp_target:
w83791d.c:858:2: warning: comparison of unsigned expression < 0 is always false
w83791d.c: In function store_temp_tolerance:
w83791d.c:920:2: warning: comparison of unsigned expression < 0 is always false

For store_temp_target, accept negative numbers and clamp to >= 0.
For store_temp_tolerance, drop the unnecessary comparison.

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

hwmon: (w83791d) Convert to use devm_ functions

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

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

Cc: Marc Hulsman 
Signed-off-by: Guenter Roeck 
Acked-by: Jean Delvare

hwmon: (w83791d) Fix checkpatch issues

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

Fixed:
ERROR: code indent should use tabs where possible
ERROR: do not use assignment in if condition
ERROR: space prohibited after that open parenthesis '('
ERROR: space required after that ',' (ctx:VxV)
WARNING: braces {} are not necessary for single statement blocks
WARNING: simple_strtol is obsolete, use kstrtol instead
WARNING: simple_strtoul is obsolete, use kstrtoul instead

Modify multi-line comments to follow Documentation/CodingStyle.

Not fixed (false positive):
ERROR: Macros with complex values should be enclosed in parenthesis

Cc: Charles Spirakis 
Cc: Marc Hulsman 
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

module_param: make bool parameters really bool (drivers & misc)

2012-01-12T23:02:20+00:00

module_param(bool) used to counter-intuitively take an int.  In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.

It's time to remove the int/unsigned int option.  For this version
it'll simply give a warning, but it'll break next kernel version.

Acked-by: Mauro Carvalho Chehab 
Signed-off-by: Rusty Russell