Re: [RESEND, V2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2021-01-28 Thread Wolfram Sang
On Tue, Jan 26, 2021 at 07:37:00PM +0800, Qii Wang wrote: > Hi Wolfram, > > On Sat, 2021-01-09 at 16:29 +0800, qii.w...@mediatek.com wrote: > > From: Qii Wang > > > > Some i2c device driver indirectly uses I2C driver when it is now > > being suspended. The i2c devices driver is suspended during

Re: [RESEND, V2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2021-01-26 Thread Qii Wang
Hi Wolfram, On Sat, 2021-01-09 at 16:29 +0800, qii.w...@mediatek.com wrote: > From: Qii Wang > > Some i2c device driver indirectly uses I2C driver when it is now > being suspended. The i2c devices driver is suspended during the > NOIRQ phase and this cannot be changed due to other dependencies.

[RESEND, V2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2021-01-09 Thread qii.wang
From: Qii Wang Some i2c device driver indirectly uses I2C driver when it is now being suspended. The i2c devices driver is suspended during the NOIRQ phase and this cannot be changed due to other dependencies. Therefore, we also need to move the suspend handling for the I2C controller driver to

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-22 Thread Qii Wang
Hi sirs: If there is no new comment, I will resent it in 5.11.

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-15 Thread Qii Wang
On Mon, 2020-12-14 at 22:08 +0200, Grygorii Strashko wrote: > > On 14/12/2020 10:48, Qii Wang wrote: > > On Thu, 2020-12-10 at 15:03 +0200, Grygorii Strashko wrote: > >> > >> On 10/12/2020 03:56, Qii Wang wrote: > >>> On Mon, 2020-12-07 at 18:35 +0200, Grygorii Strashko wrote: > > > >

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-14 Thread Grygorii Strashko
On 14/12/2020 10:48, Qii Wang wrote: On Thu, 2020-12-10 at 15:03 +0200, Grygorii Strashko wrote: On 10/12/2020 03:56, Qii Wang wrote: On Mon, 2020-12-07 at 18:35 +0200, Grygorii Strashko wrote: On Thu, 2020-12-03 at 10:01 +0200, Grygorii Strashko wrote: On 03/12/2020 03:25, Qii Wang

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-14 Thread Qii Wang
On Thu, 2020-12-10 at 15:03 +0200, Grygorii Strashko wrote: > > On 10/12/2020 03:56, Qii Wang wrote: > > On Mon, 2020-12-07 at 18:35 +0200, Grygorii Strashko wrote: > >> > >>> > >>> On Thu, 2020-12-03 at 10:01 +0200, Grygorii Strashko wrote: > > On 03/12/2020 03:25, Qii Wang wrote: >

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-10 Thread Grygorii Strashko
On 10/12/2020 03:56, Qii Wang wrote: On Mon, 2020-12-07 at 18:35 +0200, Grygorii Strashko wrote: On Thu, 2020-12-03 at 10:01 +0200, Grygorii Strashko wrote: On 03/12/2020 03:25, Qii Wang wrote: On Wed, 2020-12-02 at 16:35 +0100, Wolfram Sang wrote: Hi, Some i2c device driver

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-09 Thread Qii Wang
On Mon, 2020-12-07 at 18:35 +0200, Grygorii Strashko wrote: > > > > > On Thu, 2020-12-03 at 10:01 +0200, Grygorii Strashko wrote: > >> > >> On 03/12/2020 03:25, Qii Wang wrote: > >>> On Wed, 2020-12-02 at 16:35 +0100, Wolfram Sang wrote: > Hi, > > > Some i2c device driver

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-07 Thread Grygorii Strashko
On 07/12/2020 09:33, Qii Wang wrote: Hi: Thank you very much for your patience review. There are two main purposes of this patch: 1.i2c_mark_adapter_suspended_mark_adapter_resumed Avoid accessing the adapter while it is suspended by marking it suspended during suspend. This allows the I2C

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-06 Thread Qii Wang
Hi: Thank you very much for your patience review. There are two main purposes of this patch: 1.i2c_mark_adapter_suspended_mark_adapter_resumed Avoid accessing the adapter while it is suspended by marking it suspended during suspend. This allows the I2C core to catch this, and print a warning.

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-03 Thread Grygorii Strashko
On 03/12/2020 03:25, Qii Wang wrote: On Wed, 2020-12-02 at 16:35 +0100, Wolfram Sang wrote: Hi, Some i2c device driver indirectly uses I2C driver when it is now being suspended. The i2c devices driver is suspended during the NOIRQ phase and this cannot be changed due to other dependencies.

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-02 Thread Qii Wang
On Wed, 2020-12-02 at 16:35 +0100, Wolfram Sang wrote: > Hi, > > > Some i2c device driver indirectly uses I2C driver when it is now > > being suspended. The i2c devices driver is suspended during the > > NOIRQ phase and this cannot be changed due to other dependencies. > > Therefore, we also need

Re: [v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-12-02 Thread Wolfram Sang
Hi, > Some i2c device driver indirectly uses I2C driver when it is now > being suspended. The i2c devices driver is suspended during the > NOIRQ phase and this cannot be changed due to other dependencies. > Therefore, we also need to move the suspend handling for the I2C > controller driver to

[v2] i2c: mediatek: Move suspend and resume handling to NOIRQ phase

2020-11-18 Thread qii.wang
From: Qii Wang Some i2c device driver indirectly uses I2C driver when it is now being suspended. The i2c devices driver is suspended during the NOIRQ phase and this cannot be changed due to other dependencies. Therefore, we also need to move the suspend handling for the I2C controller driver to