linux-kernel - Re: [PATCH] HID: add support for PenMount dual-touch panel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LNX.2.00.1104211115570.12035@pobox.suse.cz>
Date:	Thu, 21 Apr 2011 11:17:04 +0200 (CEST)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Henrik Rydberg <rydberg@...omail.se>
Cc:	Benjamin Tissoires <benjamin.tissoires@...c.fr>,
	Richard Nauber <richard.nauber@...glemail.com>,
	PenMount <penmount.touch@...il.com>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Stephane Chatty <chatty@...-enac.fr>,
	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
	PenMount <salt@...t.com.tw>
Subject: Re: [PATCH] HID: add support for PenMount dual-touch panel

On Wed, 20 Apr 2011, Henrik Rydberg wrote:

> > In fact, I have a second patch just on top of this one. It has been
> > tested by Ryan with PenMount devices.
> 
> Great, and good timing. :-)
> 
> > I was waiting for Penmount to be included first as it does not impact
> > any other drivers.
> 
> I would prefer the opposite order, since it is cleaner and means one
> less device to test for the major change.

I agree with Henrik on this.

Once you are done with testing the last_field_index change, it'd be great 
if you could respin the patches so that the first one perfoms this change, 
and the second one adds PenMount support on top of it.

I'll then apply it. Thanks,

-- 
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/