Re: [PATCH v1 03/63] Input: atmel_mxt_ts - only read messages in mxt_acquire_irq() when necessary

2019-08-21 Thread Jiada Wang
Hi On 2019/08/22 2:54, Dmitry Torokhov wrote: On Wed, Aug 21, 2019 at 10:26:31PM +0900, Jiada Wang wrote: Hi Dmitry On 2019/08/17 2:16, Dmitry Torokhov wrote: On Fri, Aug 16, 2019 at 05:28:52PM +0900, Jiada Wang wrote: From: Nick Dyer The workaround of reading all messages until an

Re: [PATCH v1 03/63] Input: atmel_mxt_ts - only read messages in mxt_acquire_irq() when necessary

2019-08-21 Thread Dmitry Torokhov
On Wed, Aug 21, 2019 at 10:26:31PM +0900, Jiada Wang wrote: > Hi Dmitry > > On 2019/08/17 2:16, Dmitry Torokhov wrote: > > On Fri, Aug 16, 2019 at 05:28:52PM +0900, Jiada Wang wrote: > > > From: Nick Dyer > > > > > > The workaround of reading all messages until an invalid is received is a > > >

Re: [PATCH v1 03/63] Input: atmel_mxt_ts - only read messages in mxt_acquire_irq() when necessary

2019-08-21 Thread Jiada Wang
Hi Dmitry On 2019/08/17 2:16, Dmitry Torokhov wrote: On Fri, Aug 16, 2019 at 05:28:52PM +0900, Jiada Wang wrote: From: Nick Dyer The workaround of reading all messages until an invalid is received is a way of forcing the CHG line high, which means that when using edge-triggered interrupts

Re: [PATCH v1 03/63] Input: atmel_mxt_ts - only read messages in mxt_acquire_irq() when necessary

2019-08-16 Thread Dmitry Torokhov
On Fri, Aug 16, 2019 at 05:28:52PM +0900, Jiada Wang wrote: > From: Nick Dyer > > The workaround of reading all messages until an invalid is received is a > way of forcing the CHG line high, which means that when using > edge-triggered interrupts the interrupt can be acquired. > > With

[PATCH v1 03/63] Input: atmel_mxt_ts - only read messages in mxt_acquire_irq() when necessary

2019-08-16 Thread Jiada Wang
From: Nick Dyer The workaround of reading all messages until an invalid is received is a way of forcing the CHG line high, which means that when using edge-triggered interrupts the interrupt can be acquired. With level-triggered interrupts the workaround is unnecessary. Also, most recent