Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752783Ab1CUKNu (ORCPT ); Mon, 21 Mar 2011 06:13:50 -0400 Received: from mail-ww0-f44.google.com ([74.125.82.44]:34856 "EHLO mail-ww0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752704Ab1CUKNs convert rfc822-to-8bit (ORCPT ); Mon, 21 Mar 2011 06:13:48 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=V+wGAM5kk3V4uYStwVG333xLoHafesnbqFxxO3Z32718p+YNrk/S4vSGI7b9B80F9Z nS1Nwu/5zOzpWXIBUeQFQNVg6uh729XltvqmCm4DzHeUzk38VfwPG4pw7lVEgQQmdIrq j+tafOK+vrvyVrdI5rU7Esd0h9YPm9uHyzoU0= MIME-Version: 1.0 In-Reply-To: <20110321090533.GE3784@polaris.bitmath.org> References: <1300454876-5017-1-git-send-email-benjamin.tissoires@enac.fr> <1300454876-5017-6-git-send-email-benjamin.tissoires@enac.fr> <20110321090533.GE3784@polaris.bitmath.org> Date: Mon, 21 Mar 2011 11:13:46 +0100 X-Google-Sender-Auth: Vr-Ms4wPN8ZV99h9NwzxPTmwn4c Message-ID: Subject: Re: [PATCH v3 5/5] hid-multitouch: migrate 3M PCT touch screens to hid-multitouch From: Benjamin Tissoires To: Henrik Rydberg Cc: Dmitry Torokhov , Jiri Kosina , Stephane Chatty , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1468 Lines: 42 On Mon, Mar 21, 2011 at 10:05, Henrik Rydberg wrote: > Hi Benjamin, > > On Fri, Mar 18, 2011 at 02:27:56PM +0100, Benjamin Tissoires wrote: >> This patch merges the hid-3m-pct driver into hid-multitouch. >> To keep devices working the same way they used to with hid-3m-pct, >> we need to add two signal/noise ratios for width and height. >> We also need to work on width/height to send proper >> ABS_MT_ORIENTATION flag. >> >> Importing 3M into hid-multitouch also solved the bug in which >> devices handling width and height in their report descriptors >> did not show ABS_MT_TOUCH_MAJOR and ABS_MT_TOUCH_MINOR. >> >> Signed-off-by: Benjamin Tissoires >> Reviewed-by: St?phane Chatty >> --- > > The patch _looks_ good, but I have problems getting this to work on a > 3M2256. What devices were this one tested on so far? Hi Henrik, I tested it against 3M1968. I'll try to have it again to make more testing. What symptoms do you have? For the current time, I'll redo the commit message of the previous one and then eventually we can slip this one for the next round. And many thanks for the review of the patch series. Cheers, Benjamin > > 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/