Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1535995

Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend

Show all headers | View raw

Hi David,

Am Montag, 5. Dezember 2016, 16:02:59 CET schrieb David Wu:
> During suspend there may still be some i2c access happening.
> And if we don't keep i2c irq ON, there may be i2c access timeout if
> i2c is in irq mode of operation.

can you describe the issue you're trying to fix a bit more please?

I.e. I'd think the i2c-core does suspend i2c-client devices first, so that
these should be able to finish up their ongoing transfers and not start any
new ones instead?

Your irq can still happen slightly after the system started going to actually
sleep, so to me it looks like you just widened the window where irqs can be
handled. Especially as your irq could also just simply stem from the start
state, so you cannot even be sure if your transaction actually is finished.

So to me it looks like the i2c-connected device driver should be fixed instead?

In the past I solved this for example in the zforce_ts driver [0] to
prevent i2c transfers from happening during suspend.


Heiko

[0] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/input/touchscreen/zforce_ts.c


> 
> Signed-off-by: David Wu <david.wu@rock-chips.com>
> ---
>  drivers/i2c/busses/i2c-rk3x.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c
> index df22066..67af32a 100644
> --- a/drivers/i2c/busses/i2c-rk3x.c
> +++ b/drivers/i2c/busses/i2c-rk3x.c
> @@ -1261,7 +1261,7 @@ static int rk3x_i2c_probe(struct platform_device
> *pdev) }
> 
>  	ret = devm_request_irq(&pdev->dev, irq, rk3x_i2c_irq,
> -			       0, dev_name(&pdev->dev), i2c);
> +			       IRQF_NO_SUSPEND, dev_name(&pdev->dev), i2c);
>  	if (ret < 0) {
>  		dev_err(&pdev->dev, "cannot request IRQ\n");
>  		return ret;

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] i2c: rk3x: keep i2c irq ON in suspend David Wu <david.wu@rock-chips.com> - 2016-12-05 11:10 +0100
  Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend Heiko Stuebner <heiko@sntech.de> - 2016-12-05 12:00 +0100
    Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend "David.Wu" <david.wu@rock-chips.com> - 2016-12-06 09:20 +0100
      Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend Doug Anderson <dianders@chromium.org> - 2016-12-06 17:40 +0100
        Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend "David.Wu" <david.wu@rock-chips.com> - 2016-12-07 04:40 +0100
          Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend Grygorii Strashko <grygorii.strashko@ti.com> - 2016-12-07 17:30 +0100
            Re: [PATCH] i2c: rk3x: keep i2c irq ON in suspend "David.Wu" <david.wu@rock-chips.com> - 2016-12-08 17:00 +0100

csiph-web