Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932905Ab2F1KIJ (ORCPT ); Thu, 28 Jun 2012 06:08:09 -0400 Received: from mail-gg0-f174.google.com ([209.85.161.174]:36819 "EHLO mail-gg0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932777Ab2F1KIF (ORCPT ); Thu, 28 Jun 2012 06:08:05 -0400 MIME-Version: 1.0 In-Reply-To: References: <1340109594-10732-1-git-send-email-benjamin.tissoires@gmail.com> Date: Thu, 28 Jun 2012 12:08:04 +0200 Message-ID: Subject: Re: [PATCH 1/3] HID: hid-multitouch: fix input mode feature command From: Benjamin Tissoires To: Jiri Kosina Cc: Dmitry Torokhov , Henrik Rydberg , Stephane Chatty , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1482 Lines: 42 Hi Jiri, Thanks a lot for that. My apologies. I didn't thanked Henrik and commented his review. So thank you Henrik. (the "break" here is not an optimisation as field->maxusage may be greater than the real report count). I'm currently quite overloaded with the hid over i2c bus, and I'm getting some very goods results. I'm stuck with the acpi part, but the rest of it is working pretty good for a draft. I'll keep you inform soon. Cheers, Benjamin On Thu, Jun 28, 2012 at 10:31 AM, Jiri Kosina wrote: > On Tue, 19 Jun 2012, benjamin.tissoires wrote: > >> From: Benjamin Tissoires >> >> Zytronic panels shows a new way of setting the Input Mode feature. >> This feature is put in the second usage in the HID feature, instead >> of the first, as the majority of the multitouch devices. >> >> This patch adds a detection step when the feature is presented to know >> where the feature is located in the report. We can then trigger the right >> command to the device. This removes the magic number "0" in the function >> mt_set_input_mode. > > I have applied 1/3 and 3/3. For 2/3, Henrik raised concern which I'd like > to have resolved first. > > Thanks! > > -- > Jiri Kosina > SUSE Labs -- 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/