<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/linux.git/drivers/hwmon/asc7621.c, branch linux-4.16.y</title>
<subtitle>Linux kernel stable tree (mirror)</subtitle>
<id>https://git.radix-linux.su/kernel/linux.git/atom?h=linux-4.16.y</id>
<link rel='self' href='https://git.radix-linux.su/kernel/linux.git/atom?h=linux-4.16.y'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/'/>
<updated>2017-10-30T01:36:03+00:00</updated>
<entry>
<title>hwmon: (asc7621) remove redundant assignment to newval</title>
<updated>2017-10-30T01:36:03+00:00</updated>
<author>
<name>Colin Ian King</name>
<email>colin.king@canonical.com</email>
</author>
<published>2017-10-18T12:10:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=a3bdc5b5bd369fcab8306b80d2c740e332183d20'/>
<id>urn:sha1:a3bdc5b5bd369fcab8306b80d2c740e332183d20</id>
<content type='text'>
The setting of newval to zero is redundant as the following if/else
stanzas will always update newval to a new value. Remove the
redundant setting, cleans up clang build warning:

drivers/hwmon/asc7621.c:582:2: warning: Value stored to 'newval' is
never read

Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (asc7621) make several arrays static const</title>
<updated>2017-08-30T13:31:49+00:00</updated>
<author>
<name>Colin Ian King</name>
<email>colin.king@canonical.com</email>
</author>
<published>2017-08-30T08:32:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=6930125858c71a62a964437c360f188e96879438'/>
<id>urn:sha1:6930125858c71a62a964437c360f188e96879438</id>
<content type='text'>
Don't populate the arrays on the stack, instead make them static.
Makes the object code smaller by over 950 bytes:

Before:
   text	   data	    bss	    dec	    hex	filename
  26144	  18768	    352	  45264	   b0d0	drivers/hwmon/asc7621.o

After:
   text	   data	    bss	    dec	    hex	filename
  25029	  18928	    352	  44309	   ad15	drivers/hwmon/asc7621.o

Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (asc7621) Make map tables const</title>
<updated>2014-08-04T14:01:38+00:00</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@ingics.com</email>
</author>
<published>2014-07-15T11:15:01+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=7a7176aa4bf49685ddbef9adbdaf7383dc62ceed'/>
<id>urn:sha1:7a7176aa4bf49685ddbef9adbdaf7383dc62ceed</id>
<content type='text'>
Map tables are never changed, so it is safe to make them const.

Signed-off-by: Axel Lin &lt;axel.lin@ingics.com&gt;
Acked-by: George Joseph &lt;george.joseph@fairview5.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: Avoid initializing the same field twice</title>
<updated>2014-04-04T16:01:34+00:00</updated>
<author>
<name>Jean Delvare</name>
<email>jdelvare@suse.de</email>
</author>
<published>2014-04-04T16:01:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=6cf0a91ebe11cee1df871619af4eb58c9af32666'/>
<id>urn:sha1:6cf0a91ebe11cee1df871619af4eb58c9af32666</id>
<content type='text'>
All hwmon drivers allocate their data structure with some form of
kzalloc, so setting data fields to zero explicitly is a waste of time.

Signed-off-by: Jean Delvare &lt;jdelvare@suse.de&gt;
Reviewed-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: Remove unnecessary semicolons</title>
<updated>2013-10-13T23:16:25+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2013-03-29T20:54:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=a0393713530c49697e49ce0456c039228ab7facb'/>
<id>urn:sha1:a0393713530c49697e49ce0456c039228ab7facb</id>
<content type='text'>
Semicolons after closing } of conditional blocks are not needed
and can be removed.

Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Reviewed-by: Jean Delvare &lt;khali@linux-fr.org&gt;
</content>
</entry>
<entry>
<title>hwmon: Fix CamelCase checkpatch warnings</title>
<updated>2013-04-08T04:16:40+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2013-03-13T23:40:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=088ce2ac9ebac5c74faf4d39083627875fa6f0f0'/>
<id>urn:sha1:088ce2ac9ebac5c74faf4d39083627875fa6f0f0</id>
<content type='text'>
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:54+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2013-01-09T16:09:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/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: (asc7621) Convert to use devm_ functions</title>
<updated>2012-07-22T04:48:40+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-06-02T16:58:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=35bb95a17a1c04ec9620553867f29a336e868707'/>
<id>urn:sha1:35bb95a17a1c04ec9620553867f29a336e868707</id>
<content type='text'>
Convert to use devm_ functions to reduce code size and simplify the code.

Cc: George Joseph &lt;george.joseph@fairview5.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Acked-by: George Joseph &lt;george.joseph@fairview5.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (asc7621) Fix multi-line comments</title>
<updated>2012-03-19T01:26:54+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-01-19T19:02:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=3c56b066fb62e4aee95e4b3acff35296765a25d1'/>
<id>urn:sha1:3c56b066fb62e4aee95e4b3acff35296765a25d1</id>
<content type='text'>
Cc: George Joseph &lt;george.joseph@fairview5.com&gt;
Acked-by: George Joseph &lt;george.joseph@fairview5.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: replaced strict_str* with kstr*</title>
<updated>2012-01-05T16:19:34+00:00</updated>
<author>
<name>Frans Meulenbroeks</name>
<email>fransmeulenbroeks@gmail.com</email>
</author>
<published>2012-01-04T19:58:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=179c4fdb565dd2157e5dfe89318b74868e3b523d'/>
<id>urn:sha1:179c4fdb565dd2157e5dfe89318b74868e3b523d</id>
<content type='text'>
replaced strict_strtol with kstrtol and
replaced strict_strtuol with kstrtuol

This satisfies checkpatch -f
Compile tested only: no warnings or errors given

Signed-off-by: Frans Meulenbroeks &lt;fransmeulenbroeks@gmail.com&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
</feed>
