-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org]
Sent: 27 October, 2015 1:33
To: Karsten Merker
Cc: Bastien Nocera; Tirdea, Irina; Aleksei Mamlin; linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Mark Rutland; Purdila, Octavian; linux-
kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v9 0/9] Goodix touchscreen enhancements

On Mon, Oct 26, 2015 at 07:21:12PM +0100, Karsten Merker wrote:

quoted

On Mon, Oct 26, 2015 at 04:06:29PM +0100, Bastien Nocera wrote:

quoted

On Mon, 2015-10-12 at 18:24 +0300, Irina Tirdea wrote:

quoted

quoted

v9 only adds GPIOLIB dependency in Kconfig for patch 2:
"Input: goodix - reset device at init". There are no other code
changes from v8.

Thanks for testing these changes, Bastien and Aleksei!

Karsten, there is no need to rebase your series on top of v9.

Are we waiting on anything else before merging this? I'd like it to be
scheduled to be merged so I can start focusing on the subsequent and
dependent patches for that same driver.

Hello,

AFAICS there is one open point (cf.
http://www.spinics.net/lists/linux-input/msg41567.html) which
Irina wanted to address in a v10 of the patchset (cf.
http://www.spinics.net/lists/linux-input/msg41642.html).

There is also the whole thing about insane handling of named gpios in
ACPI layer, which stops me from merging the reset code since these gpios
should be marked as optional and we should stop ignoring errors coming
from gpiolib.

The ACPI layer change is quite complex, since it includes changing the drivers
that use the gpio API before removing the fallback to indexed ACPI.
Not sure that will not also break current drivers that already count on this
fallback. Unfortunately, I do not have the time right now to get involved in
fixing the ACPI core myself.

Dmitry, is there anything I can do in the driver to get these patches merged?
I could go back to using indexed gpios and add an additional property
to specify if irq can be used as output or not (as suggested in one of the
previous reviews).

Thanks,
Irina

Thanks.

--
Dmitry

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html