mbox series

[0/3] i2c: exynos5: bus recovery implementation

Message ID 20171130143007.30258-1-a.hajda@samsung.com
Headers show
Series i2c: exynos5: bus recovery implementation | expand

Message

Andrzej Hajda Nov. 30, 2017, 2:30 p.m. UTC 
Hi,

This patchset adds bus recovery functionality to Exynos HSI2C, it is used in
case of transmission timeouts.
Internal tests shows that it significantly improves I2C transmissions.

Andi, could you take a look at it/test, feedback welcome.

Later, I will post also patches to MHL chip which uses this functionality.
MHL chip has known bug of being in incorrect state after hard reset,
I2C bus recovery cures it.

Regards
Andrzej


Andrzej Hajda (3):
  i2c: exynos5: change internal transmission timeout to 100ms
  i2c: exynos5: implement bus recovery functionality
  i2c: exynos5: do not check TRANS_STATUS in case of Exynos7 variant

 drivers/i2c/busses/i2c-exynos5.c | 52 +++++++++++++++++++++++++++++++++-------
 1 file changed, 44 insertions(+), 8 deletions(-)

Comments

Andi Shyti Dec. 7, 2017, 7:36 a.m. UTC | #1 
Hi Andrzej,

> This patchset adds bus recovery functionality to Exynos HSI2C, it is used in
> case of transmission timeouts.
> Internal tests shows that it significantly improves I2C transmissions.
> 
> Andi, could you take a look at it/test, feedback welcome.
> 
> Later, I will post also patches to MHL chip which uses this functionality.
> MHL chip has known bug of being in incorrect state after hard reset,
> I2C bus recovery cures it.
> 
> Regards
> Andrzej
> 
> 
> Andrzej Hajda (3):
>   i2c: exynos5: change internal transmission timeout to 100ms
>   i2c: exynos5: implement bus recovery functionality
>   i2c: exynos5: do not check TRANS_STATUS in case of Exynos7 variant

Thanks for the patches, they look good and I tested them.

You can add for all three

Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
Tested-by: Andi Shyti <andi.shyti@samsung.com>

Thanks,
Andi