Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753878Ab2KLROo (ORCPT ); Mon, 12 Nov 2012 12:14:44 -0500 Received: from smtprelay-b22.telenor.se ([195.54.99.213]:34764 "EHLO smtprelay-b22.telenor.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751795Ab2KLROn (ORCPT ); Mon, 12 Nov 2012 12:14:43 -0500 X-SENDER-IP: [85.230.29.114] X-LISTENER: [smtp.bredband.net] X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AkPYAJEtoVBV5h1yPGdsb2JhbABEilS3fwEBAYECGQEBAQEfAhcNJ4IeAQEEATocIwULCAMYLhQlChoTiAQKugIUjAF9VYEdgnphA5V7hXuDTolr X-IronPort-AV: E=Sophos;i="4.80,763,1344204000"; d="scan'208";a="228286745" From: "Henrik Rydberg" Date: Mon, 12 Nov 2012 18:20:56 +0100 To: Alexandra Chin Cc: Dmitry Torokhov , Linux Kernel , Linux Input , Linus Walleij , Naveen Kumar Gaddipati , Mahesh Srinivasan , Alex Chang , Scott Lin , Christopher Heiny Subject: Re: [PATCH v5] staging: ste_rmi4: Convert to Type-B support Message-ID: <20121112172056.GA1400@polaris.bitmath.org> References: <20121103013623.0f3a1b28168a9a42881b0b00@tw.synaptics.com> <20121108150529.b16707c1c84e6fd862e58b6a@tw.synaptics.com> <20121109192604.GA16972@polaris.bitmath.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1661 Lines: 40 On Mon, Nov 12, 2012 at 04:40:34PM +0000, Alexandra Chin wrote: > Hi Henrik, > > >> Convert to MT-B because Synaptics touch devices are capable > >> of tracking identifiable fingers. > >> > >> Signed-off-by: Alexandra Chin > >> --- > >> Changes from v5: > >> - Incorporated Henrik's review comments > >> *rollback to v3 from v4 > >> *fix odd line break in v3 > >> - Include Alexandra in the list of authors > > > > I am seeing _new_ irrelevant changes in this version... There is no > > need to revert to v3. Just remove the synaptics_rmi4_query_function() > > changes and other uneeded cruft from the v4 patch. No new changes, > > please, and make sure the final patch is tested. > > Those changes are intended to fix line over 80 characters in v3. > The code has been optimized and the logic does not change. I would still prefer if the trivial non-functional changes were deferred to later patches. I realize that the change from 'TOUCHPAD' to 'TOUCHSCREEN' created some problems, but those changes were not called for in the first place. How about leaving those lines be for now, and then change to something more generic in a later patch? The code will obviously work equally well for both touchpad and touchscreen settings, so the defines could be better named anyways. > Patch v5 has been verified with pandaboard. Thank you. Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/