From: Benjamin Tissoires <benjamin.tissoires@enac.fr> To: Jiri Kosina <jkosina@suse.cz> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>, Henrik Rydberg <rydberg@euromail.se>, Stephane Chatty <chatty@lii-enac.fr>, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 6/6] hid-multitouch: class MT_CLS_STANTUM is redundant with MT_CLS_CONFIDENCE Date: Fri, 20 May 2011 12:32:23 +0200 [thread overview] Message-ID: <BANLkTim1nGEbBo2Zf6gfTzAAZJK5vmpy6A@mail.gmail.com> (raw) In-Reply-To: <alpine.LNX.2.00.1105201031560.28291@pobox.suse.cz> On Fri, May 20, 2011 at 10:35, Jiri Kosina <jkosina@suse.cz> wrote: > On Thu, 19 May 2011, Benjamin Tissoires wrote: > >> Stantum devices used to work with MT_CLS_STANTUM but MT_CLS_CONFIDENCE >> is exactly the same. This patch switches them to this generic class, >> and remove the unused MT_CLS_STANTUM. >> >> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@enac.fr> >> --- >> drivers/hid/hid-multitouch.c | 9 +++------ >> 1 files changed, 3 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c >> index feeb0b7..65b92d2 100644 >> --- a/drivers/hid/hid-multitouch.c >> +++ b/drivers/hid/hid-multitouch.c >> @@ -88,7 +88,6 @@ struct mt_class { >> #define MT_CLS_DUAL_INRANGE_CONTACTNUMBER 3 >> #define MT_CLS_CYPRESS 4 >> #define MT_CLS_EGALAX 5 >> -#define MT_CLS_STANTUM 6 >> #define MT_CLS_3M 7 >> #define MT_CLS_CONFIDENCE 8 >> #define MT_CLS_CONFIDENCE_MINUS_ONE 9 > > Benjamin, > > is it intentional that you are leaving the hole in the numbering here? > > I don't think there would be any issue with re-numbering 7-10, would it? Well, the first time I tried to renumber those classes (to keep them alphabetically sorted), I've been told not to do it. That's why I let the hole in this case. There won't be any issue in renumbering those classes (and we could sort them). Maybe I can just send a new patch on top of this to sort them. Cheers, Benjamin PS: and thank you very much for applying so quickly the last patches I sent. > > -- > Jiri Kosina > SUSE Labs >
WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Tissoires <benjamin.tissoires@enac.fr> To: Jiri Kosina <jkosina@suse.cz> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>, Henrik Rydberg <rydberg@euromail.se>, Stephane Chatty <chatty@lii-enac.fr>, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 6/6] hid-multitouch: class MT_CLS_STANTUM is redundant with MT_CLS_CONFIDENCE Date: Fri, 20 May 2011 12:32:23 +0200 [thread overview] Message-ID: <BANLkTim1nGEbBo2Zf6gfTzAAZJK5vmpy6A@mail.gmail.com> (raw) In-Reply-To: <alpine.LNX.2.00.1105201031560.28291@pobox.suse.cz> On Fri, May 20, 2011 at 10:35, Jiri Kosina <jkosina@suse.cz> wrote: > On Thu, 19 May 2011, Benjamin Tissoires wrote: > >> Stantum devices used to work with MT_CLS_STANTUM but MT_CLS_CONFIDENCE >> is exactly the same. This patch switches them to this generic class, >> and remove the unused MT_CLS_STANTUM. >> >> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@enac.fr> >> --- >> drivers/hid/hid-multitouch.c | 9 +++------ >> 1 files changed, 3 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c >> index feeb0b7..65b92d2 100644 >> --- a/drivers/hid/hid-multitouch.c >> +++ b/drivers/hid/hid-multitouch.c >> @@ -88,7 +88,6 @@ struct mt_class { >> #define MT_CLS_DUAL_INRANGE_CONTACTNUMBER 3 >> #define MT_CLS_CYPRESS 4 >> #define MT_CLS_EGALAX 5 >> -#define MT_CLS_STANTUM 6 >> #define MT_CLS_3M 7 >> #define MT_CLS_CONFIDENCE 8 >> #define MT_CLS_CONFIDENCE_MINUS_ONE 9 > > Benjamin, > > is it intentional that you are leaving the hole in the numbering here? > > I don't think there would be any issue with re-numbering 7-10, would it? Well, the first time I tried to renumber those classes (to keep them alphabetically sorted), I've been told not to do it. That's why I let the hole in this case. There won't be any issue in renumbering those classes (and we could sort them). Maybe I can just send a new patch on top of this to sort them. Cheers, Benjamin PS: and thank you very much for applying so quickly the last patches I sent. > > -- > Jiri Kosina > SUSE Labs > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-05-20 10:32 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-05-19 12:18 [PATCH 0/6] Support for multiple devices Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 1/6] hid-multitouch: add support for ActionStar panels Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 2/6] hid-multitouch: add support for CVTouch panels Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 3/6] hid-multitouch: add support for GoodTouch panels Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 4/6] hid-multitouch: add support for Touch International panels Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 5/6] hid-multitouch: add support for Unitec panels Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-19 12:18 ` [PATCH 6/6] hid-multitouch: class MT_CLS_STANTUM is redundant with MT_CLS_CONFIDENCE Benjamin Tissoires 2011-05-19 12:18 ` Benjamin Tissoires 2011-05-20 8:35 ` Jiri Kosina 2011-05-20 10:32 ` Benjamin Tissoires [this message] 2011-05-20 10:32 ` Benjamin Tissoires 2011-05-20 11:37 ` Jiri Kosina 2011-05-20 13:10 ` Benjamin Tissoires 2011-05-23 9:22 ` [PATCH 0/6] Support for multiple devices Henrik Rydberg
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=BANLkTim1nGEbBo2Zf6gfTzAAZJK5vmpy6A@mail.gmail.com \ --to=benjamin.tissoires@enac.fr \ --cc=chatty@lii-enac.fr \ --cc=dmitry.torokhov@gmail.com \ --cc=jkosina@suse.cz \ --cc=linux-input@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=rydberg@euromail.se \ /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: linkBe 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.