Thread (29 messages) 29 messages, 6 authors, 2015-12-18

Re: [PATCH v11 6/8] Input: goodix - add support for ESD

From: Dmitry Torokhov <hidden>
Date: 2015-11-20 17:20:51
Also in: linux-devicetree, lkml

On Fri, Nov 20, 2015 at 7:44 AM, Rob Herring [off-list ref] wrote:
On Thu, Nov 19, 2015 at 02:26:39PM +0200, Irina Tirdea wrote:
quoted
Add ESD (Electrostatic Discharge) protection mechanism.
[...]
quoted
This is based on Goodix datasheets for GT911 and GT9271 and on Goodix
driver gt9xx.c for Android (publicly available in Android kernel
trees for various devices).

Signed-off-by: Irina Tirdea <redacted>
For the binding: Acked-by: Rob Herring [off-list ref]
You should not have the "For the binding:" part here. It was just a note
so it was clear what part I looked at.

It is preferred to split DT bindings to separate patches for this
reason.
It however does not make sense if one wants to research when and why
something was changed. We do not split patches to .h files from .c
either.

Whenever I can I merge the driver and dt binding changes together if
they have been posted and reviewed as separate patches. The
Acks/Reviewed are mostly important for maintainers anyway.

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help