Hello,

Kevin Hilman wrote:
> From: Rajendra Nayak <[email protected]>
>
> Because of OMAP off-mode, powerdomain can go off when I2C is idle.
> Save enough state, and do a re-init for each transfer.
>
> Additional save/restore state added by Jagadeesh Bhaskar Pakaravoor
> (SYSC_REG) and Aaro Koskinen (wakeup sources.)
>
> Signed-off-by: Jouni Hogander <[email protected]>
> Signed-off-by: Rajendra Nayak <[email protected]>
> Cc: Jagadeesh Bhaskar Pakaravoor <[email protected]>
> Cc: Aaro Koskinen <[email protected]>
> Signed-off-by: Kevin Hilman <[email protected]>

This patch introduces a compiler warning:

  CC      drivers/i2c/busses/i2c-omap.o
drivers/i2c/busses/i2c-omap.c: In function 'omap_i2c_init':
drivers/i2c/busses/i2c-omap.c:303: warning: unused variable 'v'

So the declaration of "v" should be also removed here:

> @@ -289,21 +302,22 @@ static int omap_i2c_init(struct omap_i2c_dev *dev)
>            } else if (dev->rev >= OMAP_I2C_REV_ON_3430) {
>                    u32 v;
>
> -                  v = SYSC_AUTOIDLE_MASK;
> -                  v |= SYSC_ENAWAKEUP_MASK;
> -                  v |= (SYSC_IDLEMODE_SMART <<
> +                  dev->syscstate = SYSC_AUTOIDLE_MASK;
> +                  dev->syscstate |= SYSC_ENAWAKEUP_MASK;
> +                  dev->syscstate |= (SYSC_IDLEMODE_SMART <<

A.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to