<feed xmlns='http://www.w3.org/2005/Atom'>
<title>BMC/Intel-BMC/linux.git/drivers/hwmon, branch dev-5.8</title>
<subtitle>Intel OpenBMC Linux kernel source tree (mirror)</subtitle>
<id>https://git.radix-linux.su/BMC/Intel-BMC/linux.git/atom?h=dev-5.8</id>
<link rel='self' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/atom?h=dev-5.8'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/'/>
<updated>2020-08-06T00:30:33+00:00</updated>
<entry>
<title>hwmon: (occ) Add new temperature sensor type</title>
<updated>2020-08-06T00:30:33+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.ibm.com</email>
</author>
<published>2020-04-30T20:39:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=dbb7423b601e6c70026b90666c8bf980d6cd9965'/>
<id>urn:sha1:dbb7423b601e6c70026b90666c8bf980d6cd9965</id>
<content type='text'>
The latest version of the On-Chip Controller (OCC) has a different
format for the temperature sensor data. Add a new temperature sensor
version to handle this data.

OpenBMC-Staging-Count: 3
Signed-off-by: Eddie James &lt;eajames@linux.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: Add PECI dimmtemp driver</title>
<updated>2020-08-06T00:30:33+00:00</updated>
<author>
<name>Jae Hyun Yoo</name>
<email>jae.hyun.yoo@linux.intel.com</email>
</author>
<published>2019-12-18T01:28:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=01148274870c27c041fcf5d20c328a1d25c5e9c8'/>
<id>urn:sha1:01148274870c27c041fcf5d20c328a1d25c5e9c8</id>
<content type='text'>
This commit adds PECI dimmtemp hwmon driver.

OpenBMC-Staging-Count: 3
Signed-off-by: Jae Hyun Yoo &lt;jae.hyun.yoo@linux.intel.com&gt;
Reviewed-by: Haiyue Wang &lt;haiyue.wang@linux.intel.com&gt;
Reviewed-by: James Feist &lt;james.feist@linux.intel.com&gt;
Reviewed-by: Vernon Mauery &lt;vernon.mauery@linux.intel.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: Add PECI cputemp driver</title>
<updated>2020-08-06T00:30:33+00:00</updated>
<author>
<name>Jae Hyun Yoo</name>
<email>jae.hyun.yoo@linux.intel.com</email>
</author>
<published>2019-12-18T01:28:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=c8797c03ddd213ba8cbdca8650d03dc98fd6eeee'/>
<id>urn:sha1:c8797c03ddd213ba8cbdca8650d03dc98fd6eeee</id>
<content type='text'>
This commit adds PECI cputemp hwmon driver.

OpenBMC-Staging-Count: 3
Signed-off-by: Jae Hyun Yoo &lt;jae.hyun.yoo@linux.intel.com&gt;
Reviewed-by: Haiyue Wang &lt;haiyue.wang@linux.intel.com&gt;
Reviewed-by: James Feist &lt;james.feist@linux.intel.com&gt;
Reviewed-by: Vernon Mauery &lt;vernon.mauery@linux.intel.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus (max31785): Wrap all I2C accessors in one-shot failure handlers</title>
<updated>2020-08-06T00:30:29+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2018-04-03T14:26:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=8fe05b2b1f29f2d42d65a003643dc109b9a9d70c'/>
<id>urn:sha1:8fe05b2b1f29f2d42d65a003643dc109b9a9d70c</id>
<content type='text'>
The MAX31785(A) has shown erratic behaviour across multiple system
designs, unexpectedly clock stretching and NAKing transactions. Perform
a one-shot retry if necessary for all access attempts.

OpenBMC-Staging-Count: 12
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Tested-by: George Keishing &lt;gkeishin@in.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus (core): One-shot retries for failure to set page</title>
<updated>2020-08-06T00:30:29+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2018-04-03T14:26:53+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=c12883b6f07d58e569f80bfc0ee4f81c1efe3e88'/>
<id>urn:sha1:c12883b6f07d58e569f80bfc0ee4f81c1efe3e88</id>
<content type='text'>
Work around the shonky behaviour seen with the MAX31785 where we fail
to set the page register in some circumstances.

There's no real elegant way to do this. We can propagate the error up,
but that forces us to retry the operation way up the call tree in any
number of places. It also forces callers to split out pmbus_set_page()
from the pmbus_{read,write}_{byte,word}_data() functions in order to
differentiate between a failure to set the page and a failure to read a
register (that might not exist, in which case an error is anticiptated).

OpenBMC-Staging-Count: 12
Cc: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Cc: Matt Spinler &lt;mspinler@linux.vnet.ibm.com&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Reviewed-by: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Tested-by: George Keishing &lt;gkeishin@in.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus (core): Use driver callbacks in pmbus_get_fan_rate()</title>
<updated>2020-08-06T00:30:29+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2018-04-03T14:26:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=7d116119050ad6d8a93dd910583dd0f8ca6c8ebb'/>
<id>urn:sha1:7d116119050ad6d8a93dd910583dd0f8ca6c8ebb</id>
<content type='text'>
The driver may have overridden the pmbus_read_byte_data() callback, so
make sure we use that to achieve expected behaviour.

This helps in the MAX31785 case where we may need to perform a one-shot
retry of transfers in the face of a failure.

OpenBMC-Staging-Count: 12
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Reviewed-by: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Tested-by: George Keishing &lt;gkeishin@in.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus (max31785): Add support for devicetree configuration</title>
<updated>2020-08-06T00:30:29+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2018-04-03T14:26:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=3523bcc116e011692a4f51c64cedac7176e64a2e'/>
<id>urn:sha1:3523bcc116e011692a4f51c64cedac7176e64a2e</id>
<content type='text'>
OpenBMC-Staging-Count: 12
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Tested-by: George Keishing &lt;gkeishin@in.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon: (drivetemp) Avoid SCT usage on Toshiba DT01ACA family drives</title>
<updated>2020-07-18T15:11:44+00:00</updated>
<author>
<name>Maciej S. Szmigiero</name>
<email>mail@maciej.szmigiero.name</email>
</author>
<published>2020-07-18T12:32:10+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=c66ef39eb27fe123ee05082b90eb2985c33c7715'/>
<id>urn:sha1:c66ef39eb27fe123ee05082b90eb2985c33c7715</id>
<content type='text'>
It has been observed that Toshiba DT01ACA family drives have
WRITE FPDMA QUEUED command timeouts and sometimes just freeze until
power-cycled under heavy write loads when their temperature is getting
polled in SCT mode. The SMART mode seems to be fine, though.

Let's make sure we don't use SCT mode for these drives then.

While only the 3 TB model was actually caught exhibiting the problem let's
play safe here to avoid data corruption and extend the ban to the whole
family.

Fixes: 5b46903d8bf3 ("hwmon: Driver for disk and solid state drives with temperature sensors")
Cc: stable@vger.kernel.org
Signed-off-by: Maciej S. Szmigiero &lt;mail@maciej.szmigiero.name&gt;
Link: https://lore.kernel.org/r/0cb2e7022b66c6d21d3f189a12a97878d0e7511b.1595075458.git.mail@maciej.szmigiero.name
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (scmi) Fix potential buffer overflow in scmi_hwmon_probe()</title>
<updated>2020-07-15T14:08:21+00:00</updated>
<author>
<name>Cristian Marussi</name>
<email>cristian.marussi@arm.com</email>
</author>
<published>2020-07-15T12:13:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=3ce17cd2b94907f6d91b81b32848044b84c97606'/>
<id>urn:sha1:3ce17cd2b94907f6d91b81b32848044b84c97606</id>
<content type='text'>
SMATCH detected a potential buffer overflow in the manipulation of
hwmon_attributes array inside the scmi_hwmon_probe function:

drivers/hwmon/scmi-hwmon.c:226
 scmi_hwmon_probe() error: buffer overflow 'hwmon_attributes' 6 &lt;= 9

Fix it by statically declaring the size of the array as the maximum
possible as defined by hwmon_max define.

Signed-off-by: Cristian Marussi &lt;cristian.marussi@arm.com&gt;
Reviewed-by: Sudeep Holla &lt;sudeep.holla@arm.com&gt;
Link: https://lore.kernel.org/r/20200715121338.GA18761@e119603-lin.cambridge.arm.com
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
<entry>
<title>hwmon: (nct6775) Accept PECI Calibration as temperature source for NCT6798D</title>
<updated>2020-07-14T21:38:50+00:00</updated>
<author>
<name>Guenter Roeck</name>
<email>linux@roeck-us.net</email>
</author>
<published>2020-07-14T21:31:11+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/BMC/Intel-BMC/linux.git/commit/?id=8a03746c8baf82e1616f05a1a716d34378dcf780'/>
<id>urn:sha1:8a03746c8baf82e1616f05a1a716d34378dcf780</id>
<content type='text'>
Stefan Dietrich reports invalid temperature source messages on Asus Formula
XII Z490.

nct6775 nct6775.656: Invalid temperature source 28 at index 0,
		source register 0x100, temp register 0x73

Debugging suggests that temperature source 28 reports the CPU temperature.
Let's assume that temperature sources 28 and 29 reflect "PECI Agent {0,1}
Calibration", similar to other chips of the series.

Reported-by: Stefan Dietrich &lt;roots@gmx.de&gt;
Cc: Stefan Dietrich &lt;roots@gmx.de&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
</content>
</entry>
</feed>
