Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-26 Thread Dmitry Osipenko
25.05.2020 17:51, Wang, Jiada пишет: > Hello Dmitry ... > > Thanks for detailed information to help me boot ubuntu on acer tab a500, > now I am able to boot it with ubuntu and reproduced the issue with v11 > patch-set. > > I will start to investigate the root cause, > from now on, my update patch

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-25 Thread Wang, Jiada
Hello Dmitry On 2020/05/17 22:26, Dmitry Osipenko wrote: 17.05.2020 16:08, Dmitry Osipenko пишет: 17.05.2020 06:32, Wang, Jiada пишет: Hello Dmitry On 2020/05/14 13:53, Dmitry Osipenko wrote: 13.05.2020 08:07, Wang, Jiada пишет: Hello Dmitry On 2020/05/12 8:13, Dmitry Osipenko wrote: 11.0

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-17 Thread Dmitry Osipenko
17.05.2020 16:08, Dmitry Osipenko пишет: > 17.05.2020 06:32, Wang, Jiada пишет: >> Hello Dmitry >> >> On 2020/05/14 13:53, Dmitry Osipenko wrote: >>> 13.05.2020 08:07, Wang, Jiada пишет: Hello Dmitry On 2020/05/12 8:13, Dmitry Osipenko wrote: > 11.05.2020 05:05, Wang, Jiada пишет

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-17 Thread Dmitry Osipenko
17.05.2020 06:32, Wang, Jiada пишет: > Hello Dmitry > > On 2020/05/14 13:53, Dmitry Osipenko wrote: >> 13.05.2020 08:07, Wang, Jiada пишет: >>> Hello Dmitry >>> >>> On 2020/05/12 8:13, Dmitry Osipenko wrote: 11.05.2020 05:05, Wang, Jiada пишет: > Hello Dmitry > > Thanks for your c

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-16 Thread Wang, Jiada
Hello Dmitry On 2020/05/14 13:53, Dmitry Osipenko wrote: 13.05.2020 08:07, Wang, Jiada пишет: Hello Dmitry On 2020/05/12 8:13, Dmitry Osipenko wrote: 11.05.2020 05:05, Wang, Jiada пишет: Hello Dmitry Thanks for your comment and test, can you let me know which platform (board) you are using

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-13 Thread Dmitry Osipenko
13.05.2020 08:07, Wang, Jiada пишет: > Hello Dmitry > > On 2020/05/12 8:13, Dmitry Osipenko wrote: >> 11.05.2020 05:05, Wang, Jiada пишет: >>> Hello Dmitry >>> >>> Thanks for your comment and test, >>> >>> can you let me know which platform (board) you are using for test, >>> and DTS changes if yo

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-12 Thread Wang, Jiada
Hello Dmitry On 2020/05/12 8:13, Dmitry Osipenko wrote: 11.05.2020 05:05, Wang, Jiada пишет: Hello Dmitry Thanks for your comment and test, can you let me know which platform (board) you are using for test, and DTS changes if you have added any. That's this device-tree [1] without any extra

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-11 Thread Dmitry Osipenko
11.05.2020 05:05, Wang, Jiada пишет: > Hello Dmitry > > Thanks for your comment and test, > > can you let me know which platform (board) you are using for test, > and DTS changes if you have added any. That's this device-tree [1] without any extra changes. [1] https://patchwork.ozlabs.org/proje

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-10 Thread Wang, Jiada
Hello Dmitry Thanks for your comment and test, can you let me know which platform (board) you are using for test, and DTS changes if you have added any. Thanks, Jiada On 2020/05/11 10:05, Dmitry Osipenko wrote: 08.05.2020 08:56, Jiada Wang пишет: The path of enabling the IRQ in the probe fun

Re: [PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-10 Thread Dmitry Osipenko
08.05.2020 08:56, Jiada Wang пишет: > The path of enabling the IRQ in the probe function is not safe in level > triggered operation, if it was already powered up and there is a message > waiting on the device (eg finger down) because the object table has not yet > been read. This forces the ISR int

[PATCH v11 33/56] Input: atmel_mxt_ts - delay enabling IRQ when not using regulators

2020-05-07 Thread Jiada Wang
The path of enabling the IRQ in the probe function is not safe in level triggered operation, if it was already powered up and there is a message waiting on the device (eg finger down) because the object table has not yet been read. This forces the ISR into a hard loop. Delay enabling the interrupt