<feed xmlns='http://www.w3.org/2005/Atom'>
<title>talos-obmc-linux/drivers, branch dev-5.0</title>
<subtitle>Talos™ II Linux sources for OpenBMC</subtitle>
<id>https://git.raptorcs.com/git/talos-obmc-linux/atom?h=dev-5.0</id>
<link rel='self' href='https://git.raptorcs.com/git/talos-obmc-linux/atom?h=dev-5.0'/>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/'/>
<updated>2019-04-19T02:29:09+00:00</updated>
<entry>
<title>hwmon: (pmbus/isl68137): Update ISL68137 to upstream</title>
<updated>2019-04-19T02:29:09+00:00</updated>
<author>
<name>Patrick Venture</name>
<email>venture@google.com</email>
</author>
<published>2019-04-18T14:36:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=b1b37a25644213428f4bab4c427581dd923e35da'/>
<id>urn:sha1:b1b37a25644213428f4bab4c427581dd923e35da</id>
<content type='text'>
Apply differences from hwmon review to bring the driver code to the same
point.  There were some key differences regarding attribute placement
from the review, beyond the normal cleanup.

OpenBMC-Staging-Count: 1
Signed-off-by: Patrick Venture &lt;venture@google.com&gt;
[AJ: Rework subject]
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>pmbus: support for custom sysfs attributes</title>
<updated>2019-04-19T02:27:48+00:00</updated>
<author>
<name>krzysztof.adamski@nokia.com</name>
<email>krzysztof.adamski@nokia.com</email>
</author>
<published>2019-04-18T14:36:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=d7e7edaf5887c6c01c44d1c18609705847965e06'/>
<id>urn:sha1:d7e7edaf5887c6c01c44d1c18609705847965e06</id>
<content type='text'>
This patch makes it possible to pass custom struct attribute_group array
via the pmbus_driver_info struct so that those can be added to the
attribute groups passed to hwmon_device_register_with_groups().

This makes it possible to register custom sysfs attributes by PMBUS
drivers similar to how you can do this with most other busses/classes.

Signed-off-by: Krzysztof Adamski &lt;krzysztof.adamski@nokia.com&gt;
Signed-off-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>Revert "mtd: spi-nor: aspeed: use memcpy_fromio() to capture the optimization buffer"</title>
<updated>2019-04-19T00:56:52+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2019-04-19T00:51:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=9cd6f6c377d4eb61293f0385e2ffbcb2c731dbed'/>
<id>urn:sha1:9cd6f6c377d4eb61293f0385e2ffbcb2c731dbed</id>
<content type='text'>
This reverts commit cb8980080ccb7879c5a8ef0e3efa783c9288d907.

From Milton:

&gt; NACK
&gt;
&gt; [The] justifcation is false.  The routine reads the whole buffer
&gt; because it calls the _rep routine and takes the size.
&gt;
&gt; In addition, the comment just before aspeed_smc_read_from_ahb
&gt; tells why memcpy_fromio and memcpy_toio are broken on 32 bit
&gt; arm, and this is still the case judging from the recent bug
&gt; reportfrom a Nuvoton user [1].
&gt;
&gt; [1] https://github.com/openbmc/openbmc/issues/3521

OpenBMC-Staging-Count: 1
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon (occ): Add temp sensor value check</title>
<updated>2019-04-18T01:45:19+00:00</updated>
<author>
<name>Alexander Soldatov</name>
<email>a.soldatov@yadro.com</email>
</author>
<published>2019-04-17T18:03:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=ed068475e807a70e1c6dfaa4782ed4d252967ed8'/>
<id>urn:sha1:ed068475e807a70e1c6dfaa4782ed4d252967ed8</id>
<content type='text'>
The occ driver supports two formats for the temp sensor value.

The OCC firmware for P8 supports only the first format, for which
no range checking or error processing is performed in the driver.
Inspecting the OCC sources for P8 reveals that OCC may send
a special value 0xFFFF to indicate that a sensor read timeout
has occurred, see

https://github.com/open-power/occ/blob/master_p8/src/occ/cmdh/cmdh_fsp_cmds.c#L395

That situation wasn't handled in the driver. This patch adds invalid
temp value check for the sensor data format 1 and handles it the same
way as it is done for the format 2, where EREMOTEIO is reported for
this case.

Signed-off-by: Alexander Soldatov &lt;a.soldatov@yadro.com&gt;
Signed-off-by: Alexander Amelkin &lt;a.amelkin@yadro.com&gt;
Reviewed-by: Alexander Amelkin &lt;a.amelkin@yadro.com&gt;
Cc: Edward A. James &lt;eajames@us.ibm.com&gt;
Cc: Joel Stanley &lt;joel@jms.id.au&gt;
Reviewed-by: Eddie James &lt;eajames@linux.ibm.com&gt;
</content>
</entry>
<entry>
<title>mtd: spi-nor: aspeed: add support for the 4B opcodes</title>
<updated>2019-04-18T01:41:23+00:00</updated>
<author>
<name>Cédric Le Goater</name>
<email>clg@kaod.org</email>
</author>
<published>2019-04-17T13:39:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=280a5e83425e0e597a3765037a03cd03e80f3111'/>
<id>urn:sha1:280a5e83425e0e597a3765037a03cd03e80f3111</id>
<content type='text'>
Switch the default controller value to use the read mode in order to
customize the command and use SPINOR_OP_READ_4B (0x13) when the chip
supports 4B opcodes.

OpenBMC-Staging-Count: 1
Signed-off-by: Cédric Le Goater &lt;clg@kaod.org&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>mtd: spi-nor: aspeed: use memcpy_fromio() to capture the optimization buffer</title>
<updated>2019-04-18T01:41:20+00:00</updated>
<author>
<name>Cédric Le Goater</name>
<email>clg@kaod.org</email>
</author>
<published>2019-04-17T13:39:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=cb8980080ccb7879c5a8ef0e3efa783c9288d907'/>
<id>urn:sha1:cb8980080ccb7879c5a8ef0e3efa783c9288d907</id>
<content type='text'>
aspeed_smc_read_from_ahb() only reads the first word which is not what
we want. We want to capture a CALIBRATE_BUF_SIZE size window of the
flash contents to optimize the read.

OpenBMC-Staging-Count: 1
Signed-off-by: Cédric Le Goater &lt;clg@kaod.org&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>mtd: spi-nor: aspeed: clarify 4BYTE address mode mask</title>
<updated>2019-04-18T01:41:15+00:00</updated>
<author>
<name>Cédric Le Goater</name>
<email>clg@kaod.org</email>
</author>
<published>2019-04-17T13:39:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=0006809fb4170784c680d95226d8c8f9ae0379fc'/>
<id>urn:sha1:0006809fb4170784c680d95226d8c8f9ae0379fc</id>
<content type='text'>
OpenBMC-Staging-Count: 1
Signed-off-by: Cédric Le Goater &lt;clg@kaod.org&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>mtd: spi-nor: aspeed: introduce a aspeed_smc_default_read() helper</title>
<updated>2019-04-18T01:41:12+00:00</updated>
<author>
<name>Cédric Le Goater</name>
<email>clg@kaod.org</email>
</author>
<published>2019-04-17T13:39:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=85b9912a9d34e0d6809b6f97e1869ffea9c80d92'/>
<id>urn:sha1:85b9912a9d34e0d6809b6f97e1869ffea9c80d92</id>
<content type='text'>
OpenBMC-Staging-Count: 1
Signed-off-by: Cédric Le Goater &lt;clg@kaod.org&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon (occ): Prevent sysfs error attribute from returning error</title>
<updated>2019-04-18T01:41:09+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.ibm.com</email>
</author>
<published>2019-04-16T15:51:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=39879fe4857e5ac8f19a351f57c3e6a206b05af4'/>
<id>urn:sha1:39879fe4857e5ac8f19a351f57c3e6a206b05af4</id>
<content type='text'>
The error sysfs attribute returns the stored error state of the OCC and
doesn't depend on the OCC poll response. Therefore, split the error
attribute into it's own function to avoid failing out of the function if
the poll response fails.

OpenBMC-Staging-Count: 1
Signed-off-by: Eddie James &lt;eajames@linux.ibm.com&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>hwmon (occ): Store error condition for rate-limited polls</title>
<updated>2019-04-18T01:41:05+00:00</updated>
<author>
<name>Eddie James</name>
<email>eajames@linux.ibm.com</email>
</author>
<published>2019-04-16T15:51:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=2ed835e9a1dc6e38a05b1755142e657af38e2b71'/>
<id>urn:sha1:2ed835e9a1dc6e38a05b1755142e657af38e2b71</id>
<content type='text'>
The OCC driver limits the rate of sending poll commands to the OCC. If a
user reads a hwmon entry after a poll response resulted in an error and
is rate-limited, the error is invisible to the user. Fix this by storing
the last error and returning that in the rate-limited case.

OpenBMC-Staging-Count: 1
Signed-off-by: Eddie James &lt;eajames@linux.ibm.com&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
</feed>
