Re: [RFC/RFT 03/10] i2c: synquacer: remove unused is_suspended flag

From: Ard Biesheuvel
Date: Mon Dec 10 2018 - 16:04:23 EST


On Mon, 10 Dec 2018 at 22:03, Wolfram Sang
<wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote:
>
> This flag was defined and checked but never set a value. Remove it.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>

Acked-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>

> ---
> drivers/i2c/busses/i2c-synquacer.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-synquacer.c b/drivers/i2c/busses/i2c-synquacer.c
> index 2184b7c3580e..d18b0941b71a 100644
> --- a/drivers/i2c/busses/i2c-synquacer.c
> +++ b/drivers/i2c/busses/i2c-synquacer.c
> @@ -144,8 +144,6 @@ struct synquacer_i2c {
> u32 timeout_ms;
> enum i2c_state state;
> struct i2c_adapter adapter;
> -
> - bool is_suspended;
> };
>
> static inline int is_lastmsg(struct synquacer_i2c *i2c)
> @@ -316,9 +314,6 @@ static int synquacer_i2c_doxfer(struct synquacer_i2c *i2c,
> unsigned long timeout;
> int ret;
>
> - if (i2c->is_suspended)
> - return -EBUSY;
> -
> synquacer_i2c_hw_init(i2c);
> bsr = readb(i2c->base + SYNQUACER_I2C_REG_BSR);
> if (bsr & SYNQUACER_I2C_BSR_BB) {
> --
> 2.11.0
>