From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752272AbbJMG7u (ORCPT ); Tue, 13 Oct 2015 02:59:50 -0400 Received: from mga02.intel.com ([134.134.136.20]:50957 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751764AbbJMG7s convert rfc822-to-8bit (ORCPT ); Tue, 13 Oct 2015 02:59:48 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,676,1437462000"; d="scan'208";a="825607613" From: "Tirdea, Irina" To: Karsten Merker , Bastien Nocera , Dmitry Torokhov , Aleksei Mamlin , "linux-input@vger.kernel.org" , Ian Campbell CC: "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Chen-Yu Tsai Subject: RE: [PATCH RFC V2 0/3] Input: goodix - add axis swapping and axis inversion support Thread-Topic: [PATCH RFC V2 0/3] Input: goodix - add axis swapping and axis inversion support Thread-Index: AQHRArwAfE8WTwc9RkKyk7HBhAT7g55pACow Date: Tue, 13 Oct 2015 06:59:42 +0000 Deferred-Delivery: Tue, 13 Oct 2015 06:59:00 +0000 Message-ID: <1F3AC3675D538145B1661F571FE1805F2F0FE4B0@irsmsx105.ger.corp.intel.com> References: <1444413352-24529-1-git-send-email-merker@debian.org> In-Reply-To: <1444413352-24529-1-git-send-email-merker@debian.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Karsten Merker [mailto:merker@debian.org] > Sent: 09 October, 2015 20:56 > To: Bastien Nocera; Dmitry Torokhov; Tirdea, Irina; Aleksei Mamlin; linux-input@vger.kernel.org; Ian Campbell > Cc: devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; Chen-Yu Tsai; Karsten Merker > Subject: [PATCH RFC V2 0/3] Input: goodix - add axis swapping and axis inversion support > > Hello, > > this is v2 of my "Input: goodix - add axis swapping and axis inversion > support" patchset. > The goodix touchscreen driver has gained device-tree support in kernel > 4.1, but doesn't currently support the touchscreen-swapped-x-y, > touchscreen-inverted-x and touchscreen-inverted-y properties. > > On systems which combine a portrait-mode display with a landscape-mode > touchscreen, such as e.g. the MSI Primo 81 tablet, support for these > features is necessary to have the touchscreen and the display use the > same coordinate system. > > With support for axis inversion, the "rotated_screen" flag in the > driver can also be removed, as "rotated_screen" is just a special case > of x/y axis inversion. > > This patchset sits on top of the "[PATCH v8 0/9] Goodix touchscreen > enhancements" series by Irina Tirdea: > https://www.spinics.net/lists/linux-input/msg41437.html > > I have successfully tested the axis swapping on an (arm-based) MSI > Primo 81 tablet, but I lack appropriate hardware to do a real-world > test of the "rotated_screen" code path, so I would appreciate very > much if somebody with appropriate hardware (WinBook TW100 or TW700) > could give it a try. > > Regards, > Karsten > Hi Karsten, I took a look at your patches and also did a quick test on my setup. Code looks good, I have just one comment I've mentioned on the first patch in the series. Thanks, Irina > Changelog: > > v1: * Initial version (based von v6 of Irina Tirdea's "Goodix > touchscreen enhancements" series). > Reviewed-by: Bastien Nocera > > v2: * Rebase against v8 of Irina Tirdea's "Goodix touchscreen > enhancements" series. > * Fix a typo in the commit message. > * Add an update for the goodix dt bindings documentation > (patch No. 3). > > > Karsten Merker (3): > Input: goodix - add dt axis swapping and axis inversion support > Input: goodix - use "inverted_[xy]" flags instead of "rotated_screen" > Input: goodix - update dt bindings documentation (axis > swapping/inversion) > > .../bindings/input/touchscreen/goodix.txt | 6 ++++ > drivers/input/touchscreen/goodix.c | 33 ++++++++++++++++++---- > 2 files changed, 34 insertions(+), 5 deletions(-) > > -- > 2.1.4