<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/linux.git/drivers/hwmon/f75375s.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>2013-10-13T23:16:27+00:00</updated>
<entry>
<title>hwmon: (f75375s) Don't crash the kernel unnecessarily</title>
<updated>2013-10-13T23:16:27+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2013-09-14T07:43:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=39b103b4d2e6f006dbd425cf71c46914bd31ce80'/>
<id>urn:sha1:39b103b4d2e6f006dbd425cf71c46914bd31ce80</id>
<content type='text'>
The f75375s driver crashes the kernel if it detects an an internal
implementation error. While the detected conditions suggest that
there is a bug in the code, the condition is not fatal.
Replace BUG() with WARN().

Cc: Riku Voipio &lt;riku.voipio@iki.fi&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: use dev_get_platdata()</title>
<updated>2013-08-12T05:10:39+00:00</updated>
<author>
<name>Jingoo Han</name>
<email>jg1.han@samsung.com</email>
</author>
<published>2013-07-30T08:13:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/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: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: (f75375s) Convert to use devm_ functions</title>
<updated>2012-09-24T04:08:31+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-06-02T16:58:05+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=05639bcb4b0d509090026e836addaa3b28a14d14'/>
<id>urn:sha1:05639bcb4b0d509090026e836addaa3b28a14d14</id>
<content type='text'>
Convert to use devm_ functions to reduce code size and simplify the code.

Cc: Riku Voipio &lt;riku.voipio@iki.fi&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (f75375s) Fix warning message seen in some configurations</title>
<updated>2012-04-01T17:24:34+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2012-03-24T15:38:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=6394011d65b7e74666f07cca98747601cf8298fa'/>
<id>urn:sha1:6394011d65b7e74666f07cca98747601cf8298fa</id>
<content type='text'>
In some configurations, BUG() does not result in an endless loop but returns to
the caller. This results in the following compiler warning:

drivers/hwmon/f75375s.c: In function 'duty_mode_enabled':
drivers/hwmon/f75375s.c:280: warning: control reaches end of non-void function
drivers/hwmon/f75375s.c: In function 'auto_mode_enabled':
drivers/hwmon/f75375s.c:295: warning: control reaches end of non-void function

Fix the warning by returning something sensible after BUG().

Cc: Nikolaus Schulz &lt;schulz@macnetix.de&gt;
Cc: Riku Voipio &lt;riku.voipio@iki.fi&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Acked-by: Jean Delvare &lt;khali@linux-fr.org&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: convert drivers/hwmon/* to use module_i2c_driver()</title>
<updated>2012-03-19T01:26:50+00:00</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@gmail.com</email>
</author>
<published>2012-01-20T07:38:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=f0967eea80ec2a19a4fe1ad27e3ff1b22c79a3c7'/>
<id>urn:sha1:f0967eea80ec2a19a4fe1ad27e3ff1b22c79a3c7</id>
<content type='text'>
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 &lt;axel.lin@gmail.com&gt;
Cc: Corentin Labbe &lt;corentin.labbe@geomatys.fr&gt;
Cc: Dirk Eibach &lt;eibach@gdsys.de&gt;
Cc: "Mark M. Hoffman" &lt;mhoffman@lightlink.com&gt;
Cc: Steve Glendinning &lt;steve.glendinning@smsc.com&gt;
Cc: Riku Voipio &lt;riku.voipio@iki.fi&gt;
Cc: Guillaume Ligneul &lt;guillaume.ligneul@gmail.com&gt;
Cc: David George &lt;david.george@ska.ac.za&gt;
Cc: "Hans J. Koch" &lt;hjk@hansjkoch.de&gt;
Cc: Marc Hulsman &lt;m.hulsman@tudelft.nl&gt;
Cc: Rudolf Marek &lt;r.marek@assembler.cz&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (f75375s) Catch some attempts to write to r/o registers</title>
<updated>2012-03-02T20:02:23+00:00</updated>
<author>
<name>Nikolaus Schulz</name>
<email>mail@microschulz.de</email>
</author>
<published>2012-02-28T21:15:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=15d1ad0cc9d2d3f549afddbcdbc9c3637f0d1331'/>
<id>urn:sha1:15d1ad0cc9d2d3f549afddbcdbc9c3637f0d1331</id>
<content type='text'>
It makes no sense to attempt to manually configure the fan in auto mode,
or set the duty cycle directly in closed loop mode.  The corresponding
registers are then read-only.  If the user tries it nonetheless, error out
with EINVAL instead of silently doing nothing.

Signed-off-by: Nikolaus Schulz &lt;mail@microschulz.de&gt;
[guenter.roeck@ericsson.com: Minor formatting cleanup]
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (f75375s) Properly map the F75387 automatic modes to pwm_enable</title>
<updated>2012-03-02T19:56:43+00:00</updated>
<author>
<name>Nikolaus Schulz</name>
<email>schulz@macnetix.de</email>
</author>
<published>2012-02-28T21:15:53+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=b17d6561acc16265b65b1e0d27b649829b61a7e3'/>
<id>urn:sha1:b17d6561acc16265b65b1e0d27b649829b61a7e3</id>
<content type='text'>
The F75387 supports automatic fan control using either PWM duty cycle or
RPM speed values.  Make the driver detect the latter mode, and expose the
different modes in sysfs as per pwm_enable, so that the user can switch
between them.

The interpretation of the pwm_enable attribute for the F75387 is adjusted
to be a superset of those values used for similar Fintek chips which do
not support automatic duty mode, with 2 mapping to automatic speed mode,
and moving automatic duty mode to the new value 4.

Toggling the duty mode via pwm_enable is currently denied for the F75387,
as the chip then simply reinterprets the fan configuration register values
according to the new mode, switching between RPM and PWM units, which
makes this a dangerous operation.

This patch introduces a new pwm mode into the driver. This is necessary
because the new mode (automatic pwm mode, 4) may already be enabled by the
BIOS, and the driver should not break existing functionality. This was seen
on at least one board.

Signed-off-by: Nikolaus Schulz &lt;mail@microschulz.de&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (f75375s) Make pwm*_mode writable for the F75387</title>
<updated>2012-02-28T22:44:02+00:00</updated>
<author>
<name>Nikolaus Schulz</name>
<email>mail@microschulz.de</email>
</author>
<published>2012-02-28T21:15:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=edeea102857e33b5e9b17a3a2640da390732a693'/>
<id>urn:sha1:edeea102857e33b5e9b17a3a2640da390732a693</id>
<content type='text'>
Signed-off-by: Nikolaus Schulz &lt;mail@microschulz.de&gt;
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
<entry>
<title>hwmon: (f75375s) Fix writes to the pwm* attribute for the F75387</title>
<updated>2012-02-28T22:39:47+00:00</updated>
<author>
<name>Nikolaus Schulz</name>
<email>mail@microschulz.de</email>
</author>
<published>2012-02-28T21:15:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=331255d35d6f517020485aee38dbb8b8dfaa1642'/>
<id>urn:sha1:331255d35d6f517020485aee38dbb8b8dfaa1642</id>
<content type='text'>
For the F75387, the register holding the current PWM duty cycle value is
r/o; changing it requires writing to the fan expect register instead.

Signed-off-by: Nikolaus Schulz &lt;mail@microschulz.de&gt;
[guenter.roeck@ericsson.com: Simplified function parameters]
Signed-off-by: Guenter Roeck &lt;guenter.roeck@ericsson.com&gt;
</content>
</entry>
</feed>
