ACK: [PATCH 1/1 SRU Xenial] i2c: designware: Use transfer timeout from ioctl I2C_TIMEOUT

Shrirang Bagul shrirang.bagul at canonical.com
Thu Sep 21 07:30:00 UTC 2017


On Thu, 2017-09-21 at 13:03 +0800, Kai-Heng Feng wrote:
> From: Weifeng Voon <weifeng.voon at intel.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1718578
> 
> This allows applications to set the transfer timeout in 10ms increments via
> ioctl I2C_TIMEOUT.
> 
> Signed-off-by: Weifeng Voon <weifeng.voon at intel.com>
> Acked-by: Jarkko Nikula <jarkko.nikula at linux.intel.com>
> Acked-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> Signed-off-by: Wolfram Sang <wsa at the-dreams.de>
> (cherry picked from commit d0bcd8df9aea2bcdbfcb074d408bdc7136031bc5)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
Acked-by: Shrirang Bagul <shrirang.bagul at canonical.com>

> ---
>  drivers/i2c/busses/i2c-designware-core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/busses/i2c-designware-core.c b/drivers/i2c/busses/i2c-
> designware-core.c
> index 6974a99034c3..c331340f3bcc 100644
> --- a/drivers/i2c/busses/i2c-designware-core.c
> +++ b/drivers/i2c/busses/i2c-designware-core.c
> @@ -664,7 +664,7 @@ i2c_dw_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[],
> int num)
>  	i2c_dw_xfer_init(dev);
>  
>  	/* wait for tx to complete */
> -	if (!wait_for_completion_timeout(&dev->cmd_complete, HZ)) {
> +	if (!wait_for_completion_timeout(&dev->cmd_complete, adap->timeout)) {
>  		dev_err(dev->dev, "controller timed out\n");
>  		/* i2c_dw_init implicitly disables the adapter */
>  		i2c_dw_init(dev);
> -- 
> 2.14.1
> 

> 




More information about the kernel-team mailing list