linux-toradex.git/drivers/hwmon/pc87360.c, branch v3.10.43

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: 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: (pc87360) Convert to use devm_ functions

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

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

Cc: Jim Cromie 
Signed-off-by: Guenter Roeck 
Tested-by: Jim Cromie

hwmon: (pc87360) Fix: do not use assignment in if condition

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

Fix checkpatch issue:
ERROR: do not use assignment in if condition

Replace repeated calls to device_create_file() with calls to sysfs_create_group.

Cc: Jean Delvare 
Cc: Jim Cromie 
Signed-off-by: Guenter Roeck

hwmon: (pc87360) Fix checkpatch issues

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

Fixed:
ERROR: code indent should use tabs where possible
ERROR: do not use assignment in if condition
ERROR: "foo * bar" should be "foo *bar"
ERROR: space required after that ',' (ctx:VxV)
ERROR: spaces required around that ':' (ctx:VxE)
ERROR: spaces required around that '==' (ctx:VxV)
WARNING: braces {} are not necessary for single statement blocks
WARNING: line over 80 characters
WARNING: please, no space before tabs
WARNING: please, no spaces at the start of a line
WARNING: simple_strtol is obsolete, use kstrtol instead
WARNING: simple_strtoul is obsolete, use kstrtoul instead

Not or not all fixed (code complexity):
ERROR: Macros with complex values should be enclosed in parenthesis
ERROR: do not use assignment in if condition

Cc: Jim Cromie 
Signed-off-by: Guenter Roeck

hwmon: (pc87360) Fix multi-line comments

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

Cc: Jim Cromie 
Acked-by: Jim Cromie 
Signed-off-by: Guenter Roeck

hwmon: (pc87360) Use pr_fmt and pr_

2011-01-12T20:55:11+00:00

Added #define pr_fmt KBUILD_MODNAME ": " fmt
Converted printks to pr_
Coalesced any long formats
Removed prefixes from formats

[JD: Also convert debug messages]

Signed-off-by: Joe Perches 
Signed-off-by: Jean Delvare

hwmon: (pc87360) Fix device resource declaration

2010-08-14T19:08:48+00:00

It's not OK to call platform_device_add_resources() multiple times
in a row. Despite its name, this functions sets the resources, it
doesn't add them. So we have to prepare an array with all the
resources, and then call platform_device_add_resources() once.

Before this fix, only the last I/O resource would be actually
registered. The other I/O resources were leaked.

Signed-off-by: Jean Delvare 
Cc: Jim Cromie 
Cc: stable@kernel.org