All of lore.kernel.org
 help / color / mirror / Atom feed
From: JJ Ding <jj_ding@emc.com.tw>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: "Seth Forshee" <seth.forshee@canonical.com>,
	"Aaron Huang" <aaron_huang@emc.com.tw>,
	"Tom Lin" <tom_lin@emc.com.tw>,
	"Éric Piel" <E.A.B.Piel@tudelft.nl>,
	"Daniel Kurtz" <djkurtz@chromium.org>,
	"Chase Douglas" <chase.douglas@canonical.com>,
	"Henrik Rydberg" <rydberg@euromail.se>,
	"Alessandro Rubini" <rubini@cvml.unipv.it>,
	linux-input@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 0/8] Input: elantech: add support for newer hardware
Date: Wed, 31 Aug 2011 17:43:59 +0800	[thread overview]
Message-ID: <874o0xdig0.fsf@emc.com.tw> (raw)
In-Reply-To: <1314606539-24722-1-git-send-email-jj_ding@emc.com.tw>

Hi Dmitry,

What do you think about the series?
Do you think these patches all right for upstream?
Or do you have any other concerns about the chages?
Please let me know and I will fix it.
Thnak you.

jj

On Mon, 29 Aug 2011 16:28:51 +0800, JJ Ding <jj_ding@emc.com.tw> wrote:
> Hi Lists,
> 
> This is a series of patches to add support for newer generations of ELAN
> touchpad.
> 
> Patches #1~#6 are essentially fixes and cleanups that pave the way for
> patch #7 and patch #8, which really adds code for v3 and v4 hardware.
> 
> Any comments, suggestoins are welcome.
> 
> Note:
> Please review patch #8, which adds support for the latest touchpads from ELAN.
> This is a true multi-touch capable touchpad that can track up to 5 fingers.
> Support is implemented using MT-B protocol. I used synaptics and multitouch X
> driver to test this driver, both worked fine.
> 
> Thank you very much.
> 
> Changes since v1:
> - drop EC workaround patch.
> - add patch #2, make ST and MT have the same range, as Dmitry suggested.
> - add patch #4, remove ETP_EDGE_FUZZ_V2, as Éric suggested.
> - rename all packet checking functions so they are consistent.
> - some code cleanup, and don't report ABS_[XY] when fingers == 0, as Seth suggested.
> - add document for v3 debounce, and now check debounce in elantech_packet_check_v3.
> 
> Changes since v2:
> - fix v3 hardware packet checking (reported by Seth and fix by Dmitry.)
> 
> Changes since v3:
> - fix a tiny style problem (useless space on end-of-line, reported by Seth)
> - v3 hardware can track 2 fingers well, so do not set INPUT_PROP_SEMI_MT on v3
>   (As suggested earlier by Éric.)
> - add patch #8, which adds support for v4 hardware.
> 
> JJ Ding (8):
>   Input: elantech - correct x, y value range for v2 hardware
>   Input: elantech - get rid of ETP_2FT_* in elantech.h
>   Input: elantech - use firmware provided x, y ranges
>   Input: elantech - remove ETP_EDGE_FUZZ_V2
>   Input: elantech - packet checking for v2 hardware
>   Input: elantech - clean up elantech_init
>   Input: elantech - add v3 hardware support
>   Input: elantech - add v4 hardware support
> 
>  Documentation/input/elantech.txt |  295 +++++++++++++++++-
>  drivers/input/mouse/elantech.c   |  614 ++++++++++++++++++++++++++++++++-----
>  drivers/input/mouse/elantech.h   |   56 +++-
>  3 files changed, 849 insertions(+), 116 deletions(-)
> 
> -- 
> 1.7.4.1
> 

  parent reply	other threads:[~2011-08-31  9:42 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-29  8:28 [PATCH v4 0/8] Input: elantech: add support for newer hardware JJ Ding
2011-08-29  8:28 ` JJ Ding
2011-08-29  8:28 ` [PATCH v4 1/8] Input: elantech - correct x, y value range for v2 hardware JJ Ding
2011-08-29  8:28 ` [PATCH v4 2/8] Input: elantech - get rid of ETP_2FT_* in elantech.h JJ Ding
2011-08-29  8:28 ` [PATCH v4 3/8] Input: elantech - use firmware provided x, y ranges JJ Ding
2011-08-29  8:28 ` [PATCH v4 4/8] Input: elantech - remove ETP_EDGE_FUZZ_V2 JJ Ding
2011-08-29  8:28   ` JJ Ding
2011-08-29  8:28 ` [PATCH v4 5/8] Input: elantech - packet checking for v2 hardware JJ Ding
2011-08-29  8:28 ` [PATCH v4 6/8] Input: elantech - clean up elantech_init JJ Ding
2011-08-29  8:28 ` [PATCH v4 7/8] Input: elantech - add v3 hardware support JJ Ding
2011-08-29  8:28 ` [PATCH v4 8/8] Input: elantech - add v4 " JJ Ding
2011-08-30 13:50   ` Tom _Lin
2011-08-31 12:50   ` Éric Piel
2011-08-31 12:50     ` Éric Piel
2011-09-01  1:31     ` JJ Ding
2011-09-01  1:31       ` JJ Ding
2011-08-31  9:43 ` JJ Ding [this message]
2011-08-31 21:10   ` [PATCH v4 0/8] Input: elantech: add support for newer hardware Dmitry Torokhov
2011-09-01  1:38     ` JJ Ding
2011-08-31 12:54 ` Éric Piel
2011-08-31 12:54   ` Éric Piel
2011-09-01  1:39   ` JJ Ding
2011-09-01  1:39     ` JJ Ding

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874o0xdig0.fsf@emc.com.tw \
    --to=jj_ding@emc.com.tw \
    --cc=E.A.B.Piel@tudelft.nl \
    --cc=aaron_huang@emc.com.tw \
    --cc=chase.douglas@canonical.com \
    --cc=djkurtz@chromium.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rubini@cvml.unipv.it \
    --cc=rydberg@euromail.se \
    --cc=seth.forshee@canonical.com \
    --cc=tom_lin@emc.com.tw \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.