Re: [PATCH dev-5.2 0/2] i2c: aspeed: Add H/W timeout support

[Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>]

Hi Joel,

On 9/4/2019 3:54 PM, Joel Stanley wrote:
> Hi Jae,
> 
> On Wed, 4 Sep 2019 at 20:08, Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> wrote:
>>
>> In case of multi-master environment, if a peer master incorrectly handles
>> a bus in the middle of a transaction, I2C hardware hangs in slave state
>> and it can't escape from the slave state, so this commit adds slave
>> inactive timeout support to recover the bus in the case.
>>
>> By applying this change, SDA data-low and SCL clock-low timeout feature
>> also could be enabled which was disabled previously.
> 
> Please consider sending your RFC patches to the upstream list. You
> have a big backlog of patches now.

Thanks for the reminding. I can't send the RFC patches yet because QEMU
H/W model isn't ready yet. I'm still waiting for the fix from Cedric.

Thanks,
Jae

> 
> Cheers,
> 
> Joel
> 
>>
>> Jae Hyun Yoo (2):
>>    dt-bindings: i2c: aspeed: add hardware timeout support
>>    i2c: aspeed: add slave inactive timeout support
>>
>>   .../devicetree/bindings/i2c/i2c-aspeed.txt    |  2 +
>>   drivers/i2c/busses/i2c-aspeed.c               | 79 +++++++++++++++++--
>>   2 files changed, 75 insertions(+), 6 deletions(-)
>>
>> --
>> 2.23.0
>>