linux-toradex.git/drivers/hwmon/dme1737.c, branch v4.9.43

hwmon: drop owner assignment from platform_drivers

2014-10-20T14:20:36+00:00

A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang

hwmon: (dme1737) Prevent overflow problem when writing large limits

2014-08-06T02:40:15+00:00

On platforms with sizeof(int) < sizeof(long), writing a temperature
limit larger than MAXINT will result in unpredictable limit values
written to the chip. Avoid auto-conversion from long to int to fix
the problem.

Voltage limits, fan minimum speed, pwm frequency, pwm ramp rate, and
other attributes have the same problem, fix them as well.

Zone temperature limits are signed, but were cached as u8, causing
unepected values to be reported for negative temperatures. Cache as
s8 to fix the problem.

vrm is an u8, so the written value needs to be limited to [0, 255].

Signed-off-by: Axel Lin 
[Guenter Roeck: Fix zone temperature cache]
Cc: stable@vger.kernel.org
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: remove use of __devexit

2012-11-28T19:50:34+00:00

CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.

Signed-off-by: Bill Pemberton 
Cc: Hans de Goede 
Cc: Jean Delvare 
Cc: Alistair John Strachan 
Cc: Fenghua Yu 
Cc: Juerg Haefliger 
Cc: Andreas Herrmann 
Cc: Clemens Ladisch 
Cc: Rudolf Marek 
Cc: Jim Cromie 
Cc: "Mark M. Hoffman" 
Cc: Roger Lucas 
Acked-by: Guenter Roeck 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

hwmon: remove use of __devinit

2012-11-28T19:49:40+00:00

CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.

Signed-off-by: Bill Pemberton 
Cc: Hans de Goede 
Cc: Jean Delvare 
Cc: Alistair John Strachan 
Cc: Fenghua Yu 
Cc: Juerg Haefliger 
Cc: Andreas Herrmann 
Cc: Clemens Ladisch 
Cc: Rudolf Marek 
Cc: Jim Cromie 
Cc: "Mark M. Hoffman" 
Cc: Roger Lucas 
Acked-by: Guenter Roeck 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

hwmon: remove use of __devexit_p

2012-11-28T19:48:59+00:00

CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.

Signed-off-by: Bill Pemberton 
Cc: Hans de Goede 
Cc: Jean Delvare 
Cc: Alistair John Strachan 
Cc: Fenghua Yu 
Cc: Juerg Haefliger 
Cc: Andreas Herrmann 
Cc: Clemens Ladisch 
Cc: Rudolf Marek 
Cc: Jim Cromie 
Cc: "Mark M. Hoffman" 
Cc: Roger Lucas 
Acked-by: Guenter Roeck 
Acked-by: Mark Brown 
Signed-off-by: Greg Kroah-Hartman

hwmon: (dme1737) Convert to use devm_ functions

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

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

Cc: Juerg Haefliger 
Signed-off-by: Guenter Roeck

hwmon: (dme1737) Fix checkpatch issues

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

Fixed:
WARNING: braces {} are not necessary for any arm of this statement
WARNING: braces {} are not necessary for single statement blocks
WARNING: simple_strtol is obsolete, use kstrtol instead

Modify multi-line comments to follow Documentation/CodingStyle.

Also: s/#define^I/#define /

Not fixed (false positive):
ERROR: Macros with multiple statements should be enclosed in a do - while loop

Cc: Juerg Haefliger 
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