<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux/drivers/hwmon/smsc47m1.c, branch v4.9</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
</subtitle>
<id>https://git.shady.money/linux/atom?h=v4.9</id>
<link rel='self' href='https://git.shady.money/linux/atom?h=v4.9'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/'/>
<updated>2014-10-20T14:20:36Z</updated>
<entry>
<title>hwmon: drop owner assignment from platform_drivers</title>
<updated>2014-10-20T14:20:36Z</updated>
<author>
<name>Wolfram Sang</name>
<email>wsa@the-dreams.de</email>
</author>
<published>2014-10-20T14:20:36Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=2a1ed077189a38bccf39cb00d8dca96d20a49463'/>
<id>urn:sha1:2a1ed077189a38bccf39cb00d8dca96d20a49463</id>
<content type='text'>
A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang &lt;wsa@the-dreams.de&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Avoid forward declaration</title>
<updated>2014-08-04T14:01:39Z</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@ingics.com</email>
</author>
<published>2014-07-17T11:15:36Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=e70198acf84690b3d8ed5d1b924942c1dd85d20d'/>
<id>urn:sha1:e70198acf84690b3d8ed5d1b924942c1dd85d20d</id>
<content type='text'>
Reorder functions to avoid forward declaration.

Signed-off-by: Axel Lin &lt;axel.lin@ingics.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>Update Jean Delvare's e-mail address</title>
<updated>2014-01-29T19:40:08Z</updated>
<author>
<name>Jean Delvare</name>
<email>khali@linux-fr.org</email>
</author>
<published>2014-01-29T19:40:08Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=7c81c60f3789a082e141d7a013392af5f78db16a'/>
<id>urn:sha1:7c81c60f3789a082e141d7a013392af5f78db16a</id>
<content type='text'>
Signed-off-by: Jean Delvare &lt;khali@linux-fr.org&gt;
</content>
</entry>
<entry>
<title>hwmon: use dev_get_platdata()</title>
<updated>2013-08-12T05:10:39Z</updated>
<author>
<name>Jingoo Han</name>
<email>jg1.han@samsung.com</email>
</author>
<published>2013-07-30T08:13:06Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=a8b3a3a53f9a814e9938ea9cc179086ff5c0a387'/>
<id>urn:sha1:a8b3a3a53f9a814e9938ea9cc179086ff5c0a387</id>
<content type='text'>
Use the wrapper function for retrieving the platform data instead of
accessing dev-&gt;platform_data directly.

Signed-off-by: Jingoo Han &lt;jg1.han@samsung.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: Replace SENSORS_LIMIT with clamp_val</title>
<updated>2013-01-26T05:03:54Z</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2013-01-09T16:09:34Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=2a844c148e1f714ebf42cb96e1b172ce394c36c9'/>
<id>urn:sha1:2a844c148e1f714ebf42cb96e1b172ce394c36c9</id>
<content type='text'>
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 &lt;linux@roeck-us.net&gt;
Acked-by: George Joseph &lt;george.joseph@fairview5.com&gt;
Acked-by: Jean Delvare &lt;khali@linux-fr.org&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Convert to use devm_ functions</title>
<updated>2012-07-22T04:48:43Z</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-06-02T19:04:07Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=08ad7c9522c22eea1912d41ee9418154b8760ea9'/>
<id>urn:sha1:08ad7c9522c22eea1912d41ee9418154b8760ea9</id>
<content type='text'>
Convert to use devm_ functions to reduce code size and simplify the code.

Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Acked-by: Jean Delvare &lt;khali@linux-fr.org&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Fix compiler warning</title>
<updated>2012-04-09T19:17:38Z</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-03-28T15:55:12Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=1d0045ee4a220872b65147b5b290e4a4852386d9'/>
<id>urn:sha1:1d0045ee4a220872b65147b5b290e4a4852386d9</id>
<content type='text'>
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 &lt;linux@roeck-us.net&gt;
Reviewed-by: Robert Coulson &lt;robert.coulson@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Fix checkpatch issues</title>
<updated>2012-03-19T01:27:09Z</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-01-15T06:27:00Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=85a0c0d1a17cd83f4c2cec09c3ae69ed210f23b2'/>
<id>urn:sha1:85a0c0d1a17cd83f4c2cec09c3ae69ed210f23b2</id>
<content type='text'>
Fixed:
ERROR: code indent should use tabs where possible
ERROR: do not use assignment in if condition
ERROR: space required after that ',' (ctx:VxV)
ERROR: spaces required around that '&lt;=' (ctx:VxV)
ERROR: spaces required around that '==' (ctx:VxV)
ERROR: spaces required around that '&gt;=' (ctx:VxV)
ERROR: trailing statements should be on next line
ERROR: trailing whitespace
WARNING: simple_strtol is obsolete, use kstrtol instead
WARNING: static const char * array should probably be static const char * const

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: Jean Delvare &lt;khali@linux-fr.org&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Fix: do not use assignment in if condition</title>
<updated>2012-03-19T01:27:08Z</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-01-17T01:15:02Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=7e612685a3a3f0c1159f4562421e91a6c55d024e'/>
<id>urn:sha1:7e612685a3a3f0c1159f4562421e91a6c55d024e</id>
<content type='text'>
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 &lt;khali@linux-fr.org&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (smsc47m1) Use pr_fmt and pr_&lt;level&gt;</title>
<updated>2011-01-08T18:55:27Z</updated>
<author>
<name>Joe Perches</name>
<email>joe@perches.com</email>
</author>
<published>2010-10-20T06:51:49Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=512504e9f5489dbe85c80fd2c1b44ab5a8e0d5fa'/>
<id>urn:sha1:512504e9f5489dbe85c80fd2c1b44ab5a8e0d5fa</id>
<content type='text'>
Added #define pr_fmt KBUILD_MODNAME ": " fmt
Converted printks to pr_&lt;level&gt;
Coalesced any long formats
Removed prefixes from formats

Signed-off-by: Joe Perches &lt;joe@perches.com&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
</feed>
