Re: [PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Paweł, I would suggest dropping sentence punctuation in the patch subject line: Input: atmel_mxt_ts: Add support for optional regulators. More below... On Tue, Jul 31, 2018 at 05:18:09PM +0200, Paweł Chmiel wrote: > This patch adds optional regulators, which can be used to power > up

Re: [PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Paweł, I would suggest dropping sentence punctuation in the patch subject line: Input: atmel_mxt_ts: Add support for optional regulators. More below... On Tue, Jul 31, 2018 at 05:18:09PM +0200, Paweł Chmiel wrote: > This patch adds optional regulators, which can be used to power > up

[PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-07-31 Thread Paweł Chmiel
This patch adds optional regulators, which can be used to power up touchscreen. After enabling regulators, we need to wait 150msec. This value is taken from official driver. It was tested on Samsung Galaxy i9000 (based on Samsung S5PV210 SOC). Signed-off-by: Paweł Chmiel --- Changes from v2:

[PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-07-31 Thread Paweł Chmiel
This patch adds optional regulators, which can be used to power up touchscreen. After enabling regulators, we need to wait 150msec. This value is taken from official driver. It was tested on Samsung Galaxy i9000 (based on Samsung S5PV210 SOC). Signed-off-by: Paweł Chmiel --- Changes from v2: