<feed xmlns='http://www.w3.org/2005/Atom'>
<title>BMC/Intel-BMC/linux.git/drivers/hwmon, branch dev-4.10</title>
<subtitle>Intel OpenBMC Linux kernel source tree (mirror)</subtitle>
<id>https://git.radix-linux.su/BMC/Intel-BMC/linux.git/atom?h=dev-4.10</id>
<link rel='self' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/atom?h=dev-4.10'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/'/>
<updated>2018-01-17T20:55:08+00:00</updated>
<entry>
<title>hwmon: (w83773g) Fix fault detection and reporting</title>
<updated>2018-01-17T20:55:08+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2018-01-17T04:04:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=3def25ca7f3641339439668ac929f9e6d7ffe3ff'/>
<id>urn:sha1:3def25ca7f3641339439668ac929f9e6d7ffe3ff</id>
<content type='text'>
Smatch reports:

drivers/hwmon/w83773g.c:105
	get_fault() warn: shift has higher precedence than mask

Code analysis shows that the code is indeed wrong.
Fix it, and while we are at it, drop unnecessary typecast.

OpenBMC-Staging-Count: 1
Fixes: 86a10c802362 ("hwmon: Add W83773G driver")
Cc: Lei YU &lt;mine260309@gmail.com&gt;
Reviewed-by: Lei YU &lt;mine260309@gmail.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: Add W83773G driver</title>
<updated>2018-01-17T20:55:05+00:00</updated>
<author>
<name>Lei YU</name>
<email>mine260309@gmail.com</email>
</author>
<published>2018-01-17T04:04:00+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=649c3383044249d278a52e907021955c33d62de4'/>
<id>urn:sha1:649c3383044249d278a52e907021955c33d62de4</id>
<content type='text'>
Nuvoton W83773G is a hardware monitor IC providing one local
temperature and two remote temperature sensors.

OpenBMC-Staging-Count: 1
Signed-off-by: Lei YU &lt;mine260309@gmail.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (pmbus/ir35221) Remove unnecessary scaling</title>
<updated>2018-01-16T17:54:30+00:00</updated>
<author>
<name>Samuel Mendoza-Jonas</name>
<email>sam@mendozajonas.com</email>
</author>
<published>2018-01-15T05:58:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=784f8179391e32a89ea69ba6b619f19d757e7f0e'/>
<id>urn:sha1:784f8179391e32a89ea69ba6b619f19d757e7f0e</id>
<content type='text'>
The ir35221 datasheet describes specific scaling factors for a number of
commands which the current driver applies when reading.
However now that the ir35221 has been tested on machines with more
easily verifiable readings these descriptions have turned out to be
superfluous and reading each command according to the linear format is
sufficient.

OpenBMC-Staging-Count: 1
Signed-off-by: Samuel Mendoza-Jonas &lt;sam@mendozajonas.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon (occ): Fix integer overflow in multiplication</title>
<updated>2018-01-16T17:54:30+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.vnet.ibm.com</email>
</author>
<published>2017-12-19T19:56:33+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=4a2c3dea18a9adabe189e2afc89d0324f85331f9'/>
<id>urn:sha1:4a2c3dea18a9adabe189e2afc89d0324f85331f9</id>
<content type='text'>
Power values were overflowing INT_MAX when being converted to
microwatts, even though the storage was sufficiently large (unsigned 64
bit). Change literals to unsigned long long. Also change tmep storage to u32
to avoid overflows at 65000 millidegrees.

OpenBMC-Staging-Count: 1
Signed-off-by: Edward A. James &lt;eajames@us.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon (pmbus): cffps: Add led class device for power supply fault led</title>
<updated>2018-01-16T17:54:28+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.vnet.ibm.com</email>
</author>
<published>2018-01-11T18:47:17+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=03fbaa2d57b579c68f82b4ff2a0e22a50ad071c8'/>
<id>urn:sha1:03fbaa2d57b579c68f82b4ff2a0e22a50ad071c8</id>
<content type='text'>
This power supply device doesn't correctly manage it's own fault led.
Add an led class device and register it so that userspace can manage
power supply fault led as necessary.

OpenBMC-Staging-Count: 1
Signed-off-by: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (pmbus) cffps: Add debugfs entries</title>
<updated>2018-01-16T17:54:28+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.vnet.ibm.com</email>
</author>
<published>2018-01-11T18:47:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=1c8894815663c3cac90b0632db72f8be5eca9e50'/>
<id>urn:sha1:1c8894815663c3cac90b0632db72f8be5eca9e50</id>
<content type='text'>
Add debugfs entries for additional power supply data, including part
number, serial number, FRU number, firmware revision, ccin, and the
input history of the power supply. The input history is 10 minutes of
input power data in the form of twenty 30-second packets. Each packet
contains average and maximum power for that 30 second period.

OpenBMC-Staging-Count: 1
Signed-off-by: Edward A. James &lt;eajames@us.ibm.com&gt;
[groeck: Fixed endianness problem]
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (pmbus) cffps: Add PMBUS_SKIP_STATUS_CHECK</title>
<updated>2018-01-16T17:54:27+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.vnet.ibm.com</email>
</author>
<published>2018-01-11T18:47:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=dd710e71ee53b95f4cdacdd364b648263aea251a'/>
<id>urn:sha1:dd710e71ee53b95f4cdacdd364b648263aea251a</id>
<content type='text'>
This power supply device regularly fails to read VOUT_MODE due to the
CML bit going high. This results in an incorrect exponent used for the
voltage data, and therefore the power supply reports incorrect voltage.
Work around this by setting the pmbus flag to skip the CML check.

OpenBMC-Staging-Count: 1
Signed-off-by: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Fixes: f69316d62c70 ("hwmon: (pmbus) Add IBM Common Form Factor (CFF) ...")
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (pmbus) Export pmbus device debugfs directory entry</title>
<updated>2018-01-16T17:54:27+00:00</updated>
<author>
<name>Edward A. James</name>
<email>eajames@us.ibm.com</email>
</author>
<published>2017-12-11T21:32:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=e151e191cdba5114b877dd5789e9580313b85e1c'/>
<id>urn:sha1:e151e191cdba5114b877dd5789e9580313b85e1c</id>
<content type='text'>
Pmbus client drivers, if they want to use debugfs, should use the same
root directory as the pmbus debugfs entries are using. Therefore, export
the device dentry for the pmbus client.

OpenBMC-Staging-Count: 1
Signed-off-by: Edward A. James &lt;eajames@us.ibm.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (pmbus/core) Prevent unintentional setting of page to 0xFF</title>
<updated>2018-01-16T17:54:26+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.vnet.ibm.com</email>
</author>
<published>2018-01-11T18:47:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=3339bdef88d3d546bca653c6e6fa101283c252ff'/>
<id>urn:sha1:3339bdef88d3d546bca653c6e6fa101283c252ff</id>
<content type='text'>
The pmbus core may call read/write word data functions with a page value
of -1, intending to perform the operation without setting the page.
However, the read/write word data functions accept only unsigned 8-bit
page numbers, and therefore cannot check for negative page number to
avoid setting the page. This results in setting the page number to 0xFF.
This may result in errors or undefined behavior of some devices
(specifically the ir35221, which allows the page to be set to 0xFF,
but some subsequent operations to read registers may fail).

Switch the pmbus_set_page page parameter to an integer and perform the
check for negative page there. Make read/write functions consistent in
accepting an integer page number parameter.

OpenBMC-Staging-Count: 1
Signed-off-by: Edward A. James &lt;eajames@us.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus: core: Switch PAGE failure dev_warn() to dev_dbg() for serenity</title>
<updated>2017-10-30T03:28:17+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2017-10-30T00:27:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=1b10a88dfa59ea1609c21acd50681a62decdabac'/>
<id>urn:sha1:1b10a88dfa59ea1609c21acd50681a62decdabac</id>
<content type='text'>
Reduce kernel log spam from PMBus core failing a PAGE command to the
MAX37185 (and any other device). It's a hardware-design or driver
related fault, and is not anything the user can fix. Therefore reduce
it from warn to debug.

OpenBMC-Staging-Count: 1
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Acked-by: Andrew Geissler &lt;geissonator@gmail.com&gt;
</content>
</entry>
</feed>
