All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jiri Kosina <jkosina@suse.cz>
To: Benjamin Tissoires <benjamin.tissoires@enac.fr>
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 13:37:45 +0200 (CEST)	[thread overview]
Message-ID: <alpine.LNX.2.00.1105201336280.28291@pobox.suse.cz> (raw)
In-Reply-To: <BANLkTim1nGEbBo2Zf6gfTzAAZJK5vmpy6A@mail.gmail.com>

On Fri, 20 May 2011, Benjamin Tissoires wrote:

> > 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.

OK. Well, I don't really care personally much, I was just wondering why 
leaving it unnecessarily sparse.

I have applied the whole series now, thanks.

> PS: and thank you very much for applying so quickly the last patches I 
> sent.

I am preparing the tree for merge window push, so I rather have everything 
sorted out in the upcoming day or two.

Thanks,

-- 
Jiri Kosina
SUSE Labs

  reply	other threads:[~2011-05-20 11:37 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
2011-05-20 10:32       ` Benjamin Tissoires
2011-05-20 11:37       ` Jiri Kosina [this message]
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=alpine.LNX.2.00.1105201336280.28291@pobox.suse.cz \
    --to=jkosina@suse.cz \
    --cc=benjamin.tissoires@enac.fr \
    --cc=chatty@lii-enac.fr \
    --cc=dmitry.torokhov@gmail.com \
    --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: 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.