Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-15 Thread Dmitry Osipenko
15.09.2020 18:55, Wang, Jiada пишет: ... >> Jiada, will you be able to re-work this patch? Please note that the new >> "atmel,mXT1386" DT compatible needs to be added into the >> atmel,maxtouch.txt binding. > > Yes, I can re-work this patch, and add one more change to dts-binding. > > to summariz

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-15 Thread Wang, Jiada
Hi Dmitry On 2020/09/15 6:33, Dmitry Osipenko wrote: 14.09.2020 22:36, Dmitry Torokhov пишет: On Mon, Sep 14, 2020 at 12:33:40PM -0700, Dmitry Torokhov wrote: On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Osipenko wrote: 13.09.2020 19:56, Dmitry Torokhov пишет: Hi Jiada, On Sat, Sep 12,

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Rob Herring
On Mon, Sep 14, 2020 at 4:32 PM Dmitry Torokhov wrote: > > On Tue, Sep 15, 2020 at 12:33:18AM +0300, Dmitry Osipenko wrote: > > 14.09.2020 22:36, Dmitry Torokhov пишет: > > > On Mon, Sep 14, 2020 at 12:33:40PM -0700, Dmitry Torokhov wrote: > > >> On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Os

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Torokhov
On Tue, Sep 15, 2020 at 12:33:18AM +0300, Dmitry Osipenko wrote: > 14.09.2020 22:36, Dmitry Torokhov пишет: > > On Mon, Sep 14, 2020 at 12:33:40PM -0700, Dmitry Torokhov wrote: > >> On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Osipenko wrote: > >>> 13.09.2020 19:56, Dmitry Torokhov пишет: > >>>

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Osipenko
14.09.2020 22:36, Dmitry Torokhov пишет: > On Mon, Sep 14, 2020 at 12:33:40PM -0700, Dmitry Torokhov wrote: >> On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Osipenko wrote: >>> 13.09.2020 19:56, Dmitry Torokhov пишет: Hi Jiada, On Sat, Sep 12, 2020 at 09:55:21AM +0900, Jiada Wang

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Torokhov
On Mon, Sep 14, 2020 at 12:33:40PM -0700, Dmitry Torokhov wrote: > On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Osipenko wrote: > > 13.09.2020 19:56, Dmitry Torokhov пишет: > > > Hi Jiada, > > > > > > On Sat, Sep 12, 2020 at 09:55:21AM +0900, Jiada Wang wrote: > > >> From: Nick Dyer > > >> >

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Torokhov
On Mon, Sep 14, 2020 at 08:29:44PM +0300, Dmitry Osipenko wrote: > 13.09.2020 19:56, Dmitry Torokhov пишет: > > Hi Jiada, > > > > On Sat, Sep 12, 2020 at 09:55:21AM +0900, Jiada Wang wrote: > >> From: Nick Dyer > >> > >> Some maXTouch chips (eg mXT1386) will not respond on the first I2C request >

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Osipenko
13.09.2020 19:56, Dmitry Torokhov пишет: > Hi Jiada, > > On Sat, Sep 12, 2020 at 09:55:21AM +0900, Jiada Wang wrote: >> From: Nick Dyer >> >> Some maXTouch chips (eg mXT1386) will not respond on the first I2C request >> when they are in a sleep state. It must be retried after a delay for the >> c

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Osipenko
14.09.2020 18:50, Andy Shevchenko пишет: ... >> It's more preferred >> to accept patch as-is if it does right thing and then maintainer could >> modify the patch, making cosmetic changes. > > It depends on the maintainer's workflow (which may be different from > maintainer to maintainer). Sure!

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Andy Shevchenko
On Mon, Sep 14, 2020 at 6:26 PM Dmitry Osipenko wrote: > 14.09.2020 16:49, Andy Shevchenko пишет: > > On Sun, Sep 13, 2020 at 3:57 PM Dmitry Osipenko wrote: ... > + ret = i2c_transfer(client->adapter, xfer, ARRAY_SIZE(xfer)); > + if (ret != ARRAY_SIZE(xfer)) { > >> ...> Al

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Dmitry Osipenko
14.09.2020 16:49, Andy Shevchenko пишет: > On Sun, Sep 13, 2020 at 3:57 PM Dmitry Osipenko wrote: >> >> 13.09.2020 11:43, Andy Shevchenko пишет: >>> ... >>> + bool retried = false; >> >>> I thought Dmitry wants that to be retry >> >> In the comment to v2 you suggested to negate the cond

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-14 Thread Andy Shevchenko
On Sun, Sep 13, 2020 at 3:57 PM Dmitry Osipenko wrote: > > 13.09.2020 11:43, Andy Shevchenko пишет: > > ... > > > >> + bool retried = false; > > > I thought Dmitry wants that to be retry > > In the comment to v2 you suggested to negate the condition, Where? I just checked a few messages bef

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-13 Thread Dmitry Torokhov
Hi Jiada, On Sat, Sep 12, 2020 at 09:55:21AM +0900, Jiada Wang wrote: > From: Nick Dyer > > Some maXTouch chips (eg mXT1386) will not respond on the first I2C request > when they are in a sleep state. It must be retried after a delay for the > chip to wake up. Do we know when the chip is in sle

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-13 Thread Dmitry Osipenko
13.09.2020 11:43, Andy Shevchenko пишет: > ... > >> + bool retried = false; > I thought Dmitry wants that to be retry In the comment to v2 you suggested to negate the condition, hence I thought it's YOU who wants it to be retried. The "retried" is a very common form among kernel drivers,

Re: [PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-13 Thread Andy Shevchenko
On Sat, Sep 12, 2020 at 3:55 AM Jiada Wang wrote: > > From: Nick Dyer > > Some maXTouch chips (eg mXT1386) will not respond on the first I2C request > when they are in a sleep state. It must be retried after a delay for the > chip to wake up. > > Signed-off-by: Nick Dyer > [gdavis: Forward port

[PATCH v4 1/1] Input: atmel_mxt_ts - implement I2C retries

2020-09-11 Thread Jiada Wang
From: Nick Dyer Some maXTouch chips (eg mXT1386) will not respond on the first I2C request when they are in a sleep state. It must be retried after a delay for the chip to wake up. Signed-off-by: Nick Dyer [gdavis: Forward port and fix conflicts.] Signed-off-by: George G. Davis [jiada: return