mirror of
https://github.com/torvalds/linux
synced 2024-10-15 07:47:34 +00:00
i2c: exynos5: use 'time_left' variable with wait_for_completion_timeout()
There is a confusing pattern in the kernel to use a variable named 'timeout' to store the result of wait_for_completion_timeout() causing patterns like: timeout = wait_for_completion_timeout(...) if (!timeout) return -ETIMEDOUT; with all kinds of permutations. Use 'time_left' as a variable to make the code self explaining. Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
This commit is contained in:
parent
4804a8b786
commit
9b238f0d56
|
@ -763,7 +763,7 @@ static bool exynos5_i2c_poll_irqs_timeout(struct exynos5_i2c *i2c,
|
|||
static int exynos5_i2c_xfer_msg(struct exynos5_i2c *i2c,
|
||||
struct i2c_msg *msgs, int stop)
|
||||
{
|
||||
unsigned long timeout;
|
||||
unsigned long time_left;
|
||||
int ret;
|
||||
|
||||
i2c->msg = msgs;
|
||||
|
@ -775,13 +775,13 @@ static int exynos5_i2c_xfer_msg(struct exynos5_i2c *i2c,
|
|||
exynos5_i2c_message_start(i2c, stop);
|
||||
|
||||
if (!i2c->atomic)
|
||||
timeout = wait_for_completion_timeout(&i2c->msg_complete,
|
||||
EXYNOS5_I2C_TIMEOUT);
|
||||
else
|
||||
timeout = exynos5_i2c_poll_irqs_timeout(i2c,
|
||||
time_left = wait_for_completion_timeout(&i2c->msg_complete,
|
||||
EXYNOS5_I2C_TIMEOUT);
|
||||
else
|
||||
time_left = exynos5_i2c_poll_irqs_timeout(i2c,
|
||||
EXYNOS5_I2C_TIMEOUT);
|
||||
|
||||
if (timeout == 0)
|
||||
if (time_left == 0)
|
||||
ret = -ETIMEDOUT;
|
||||
else
|
||||
ret = i2c->state;
|
||||
|
|
Loading…
Reference in a new issue