<feed xmlns='http://www.w3.org/2005/Atom'>
<title>talos-obmc-linux/drivers/i2c, branch dev-4.10</title>
<subtitle>Talos™ II Linux sources for OpenBMC</subtitle>
<id>https://git.raptorcs.com/git/talos-obmc-linux/atom?h=dev-4.10</id>
<link rel='self' href='https://git.raptorcs.com/git/talos-obmc-linux/atom?h=dev-4.10'/>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/'/>
<updated>2017-10-30T23:35:08+00:00</updated>
<entry>
<title>drivers: i2c: fsi: Add proper abort method</title>
<updated>2017-10-30T23:35:08+00:00</updated>
<author>
<name>Edward A. James</name>
<email>eajames@us.ibm.com</email>
</author>
<published>2017-10-30T18:43:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=de178a52f1a46c615b19e504a47dcebec054c45f'/>
<id>urn:sha1:de178a52f1a46c615b19e504a47dcebec054c45f</id>
<content type='text'>
Driver wasn't cleaning up on timeout or in an error situation properly.
Need to do a full reset if we fail in order to re-stablish a good state
of the engine.

OpenBMC-Staging-Count: 1
Signed-off-by: Edward A. James &lt;eajames@us.ibm.com&gt;
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: aspeed: Retain delay/setup/hold values when configuring bus frequency</title>
<updated>2017-08-29T06:51:06+00:00</updated>
<author>
<name>Andrew Jeffery</name>
<email>andrew@aj.id.au</email>
</author>
<published>2017-08-15T07:21:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=9c34c71a6691e008d14f276d521d8233e62451df'/>
<id>urn:sha1:9c34c71a6691e008d14f276d521d8233e62451df</id>
<content type='text'>
In addition to the base, low and high clock configuration, the AC timing
register #1 on the AST2400 houses fields controlling:

1. tBUF: Minimum delay between Stop and Start conditions
2. tHDSTA: Hold time for the Start condition
3. tACST: Setup time for Start and Stop conditions, and hold time for the
   Repeated Start condition

These values are defined in hardware on the AST2500 and therefore don't
need to be set.

aspeed_i2c_init_clk() was performing a direct write of the generated
clock values rather than a read/mask/modify/update sequence to retain
tBUF, tHDSTA and tACST, and therefore cleared the tBUF, tHDSTA and tACST
fields on the AST2400. This resulted in a delay/setup/hold time of 1
base clock, which in some configurations is not enough for some devices
(e.g. the MAX31785 fan controller, with an APB of 48MHz and a desired
bus speed of 100kHz).

Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Reviewed-by: Brendan Higgins &lt;brendanhiggins@google.com&gt;
Tested-by: Brendan Higgins &lt;brendanhiggins@google.com&gt;
Signed-off-by: Wolfram Sang &lt;wsa@the-dreams.de&gt;
(cherry picked from commit 95fd3ad9cd5f8a4bb01215b846a3c8e6adefe21c)
Signed-off-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: aspeed: add proper support fo 24xx clock params</title>
<updated>2017-08-15T03:05:58+00:00</updated>
<author>
<name>Brendan Higgins</name>
<email>brendanhiggins@google.com</email>
</author>
<published>2017-07-28T20:45:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=ecfff44dbe15063648e3863eb2cc783c87832935'/>
<id>urn:sha1:ecfff44dbe15063648e3863eb2cc783c87832935</id>
<content type='text'>
24xx BMCs have larger clock divider granularity which can cause problems
when trying to set them as 25xx clock dividers; this adds clock setting
code specific to 24xx.

This also fixes a potential issue where clock dividers were rounded down
instead of up.

Signed-off-by: Brendan Higgins &lt;brendanhiggins@google.com&gt;
Signed-off-by: Wolfram Sang &lt;wsa@the-dreams.de&gt;
(cherry picked from commit 87b59ff8d1d9d809c014a4e20bc043064b6e0047)
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: fsi: Fix bug on remove() and reset to upstream state</title>
<updated>2017-08-08T12:32:23+00:00</updated>
<author>
<name>Edward A. James</name>
<email>eajames@us.ibm.com</email>
</author>
<published>2017-08-08T03:01:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=e27d476ac884d4f4190456111425a83e58ee8083'/>
<id>urn:sha1:e27d476ac884d4f4190456111425a83e58ee8083</id>
<content type='text'>
Deleting entries in list on remove() even if the list was empty...
Also a few other fixes just to get it in line with what I've sent
upstream.

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>i2c: aspeed: fixed potential null pointer dereference</title>
<updated>2017-08-01T00:14:05+00:00</updated>
<author>
<name>Brendan Higgins</name>
<email>brendanhiggins@google.com</email>
</author>
<published>2017-07-29T01:00:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=7ed1754af0cae68d8fb552cebbf0799fe38a03ed'/>
<id>urn:sha1:7ed1754af0cae68d8fb552cebbf0799fe38a03ed</id>
<content type='text'>
Before I skipped null checks when the master is in the STOP state; this
fixes that.

OpenBMC-Staging-Count: 1
Signed-off-by: Brendan Higgins &lt;brendanhiggins@google.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: aspeed: Reset to upstream version</title>
<updated>2017-07-26T12:17:41+00:00</updated>
<author>
<name>Joel Stanley</name>
<email>joel@jms.id.au</email>
</author>
<published>2017-07-25T06:51:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=063f2e641487d693eb538439695d420a8ed90cab'/>
<id>urn:sha1:063f2e641487d693eb538439695d420a8ed90cab</id>
<content type='text'>
This sets the i2c device driver to the same state as the one that was
included in the 4.13 merge window.

This is as of 4.13-rc1. There were three commits that added the code:

$ git log --oneline --no-merges drivers/i2c/busses/i2c-aspeed.c drivers/irqchip/irq-aspeed-i2c-ic.c
f9eb91350bb2 i2c: aspeed: added slave support for Aspeed I2C driver
f327c686d3ba i2c: aspeed: added driver for Aspeed I2C
f48e699ddf70 irqchip/aspeed-i2c-ic: Add I2C IRQ controller for Aspeed

So this state is current as of f9eb91350bb2.

OpenBMC-Staging-Count: 1
Tested-by: Andrew Jeffery &lt;andrew@aj.id.au&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: fsi: Add SMBUS emulation</title>
<updated>2017-07-19T13:57:13+00:00</updated>
<author>
<name>Joel Stanley</name>
<email>joel@jms.id.au</email>
</author>
<published>2017-07-06T06:44:43+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=74d230f203a86659338d4d83aa1dc96ccb5d4098'/>
<id>urn:sha1:74d230f203a86659338d4d83aa1dc96ccb5d4098</id>
<content type='text'>
Userspace expects this to be enabled. For example, it is required for
using i2cdetect.

Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
Acked-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>i2c: fsi: Remove unnecessary compatible check</title>
<updated>2017-07-08T02:36:44+00:00</updated>
<author>
<name>Joel Stanley</name>
<email>joel@jms.id.au</email>
</author>
<published>2017-07-06T10:26:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=0799bbb43a7e9b759a5e4e0eb9566dae27ac7b81'/>
<id>urn:sha1:0799bbb43a7e9b759a5e4e0eb9566dae27ac7b81</id>
<content type='text'>
The driver is probed by the FSI bus driver when the compatible matches,
so we do not need to re-check it.

OpenBMC-Staging-Count: 1
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
<entry>
<title>i2c: fsi: Fix devicetree parsing in FSI-attached I2C master algo</title>
<updated>2017-07-08T02:36:44+00:00</updated>
<author>
<name>Edward A. James</name>
<email>eajames@us.ibm.com</email>
</author>
<published>2017-07-07T21:47:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=5d7f539fd63984a0431ff74f26c5585e0f9eadb8'/>
<id>urn:sha1:5d7f539fd63984a0431ff74f26c5585e0f9eadb8</id>
<content type='text'>
Add compatible check for the master device node. Change "port" to "reg"
for the port number property. Check that the port node is available. Add
the device node to the adapter device.

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>i2c: fsi: Remove idx counter</title>
<updated>2017-07-08T02:36:44+00:00</updated>
<author>
<name>Joel Stanley</name>
<email>joel@jms.id.au</email>
</author>
<published>2017-07-08T02:00:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/talos-obmc-linux/commit/?id=8abacd9729e3f1a702c6629ca6b1dc261415b150'/>
<id>urn:sha1:8abacd9729e3f1a702c6629ca6b1dc261415b150</id>
<content type='text'>
It is no longer used.

OpenBMC-Staging-Count: 1
Acked-by: Eddie James &lt;eajames@linux.vnet.ibm.com&gt;
Signed-off-by: Joel Stanley &lt;joel@jms.id.au&gt;
</content>
</entry>
</feed>
