Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751657Ab2BVHmY (ORCPT ); Wed, 22 Feb 2012 02:42:24 -0500 Received: from mail-gx0-f202.google.com ([209.85.161.202]:52201 "EHLO mail-gx0-f202.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750943Ab2BVHmX (ORCPT ); Wed, 22 Feb 2012 02:42:23 -0500 Authentication-Results: mr.google.com; spf=pass (google.com: domain of cywang@google.com designates 10.100.243.25 as permitted sender) smtp.mail=cywang@google.com MIME-Version: 1.0 From: Chung-yih Wang To: Alessandro Rubini Cc: Dmitry Torokhov , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, djkurtz@chromium.org, Chung-yih Wang Subject: [PATCH] CHROMIUM: Input: synaptics - filter out the events with low z values Date: Wed, 22 Feb 2012 15:41:43 +0800 Message-Id: <1329896503-28394-1-git-send-email-cywang@chromium.org> X-Mailer: git-send-email 1.7.7.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2572 Lines: 68 The single touch path imposes a minimum z value (with hysterisis) before registering BTN_TOUCH. Apply the same (hard coded) threshold when deciding how many fingers to report in the semi-mt path. This patch improves performance of the Google Cr-48 chromebook's extremely sensitive Synaptics profile sensor touchpad by filtering out touch events for hovering fingers. Note: We continue to use the same hard coded threshold value used in the single touch case as it appears this works just as well on these multitouch profile sensor pads as on whatever pads it was originally discovered. Signed-off-by: Chung-Yih Wang --- drivers/input/mouse/synaptics.c | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c index 8081a0a..746dbcc 100644 --- a/drivers/input/mouse/synaptics.c +++ b/drivers/input/mouse/synaptics.c @@ -568,17 +568,22 @@ static void synaptics_report_semi_mt_slot(struct input_dev *dev, int slot, } } +static bool finger_touched(const struct synaptics_hw_state *hw) +{ + return (hw->z > 30); +} + static void synaptics_report_semi_mt_data(struct input_dev *dev, const struct synaptics_hw_state *a, const struct synaptics_hw_state *b, int num_fingers) { - if (num_fingers >= 2) { + if ((num_fingers >= 2) && finger_touched(a) && finger_touched(b)) { synaptics_report_semi_mt_slot(dev, 0, true, min(a->x, b->x), min(a->y, b->y)); synaptics_report_semi_mt_slot(dev, 1, true, max(a->x, b->x), max(a->y, b->y)); - } else if (num_fingers == 1) { + } else if ((num_fingers == 1) && finger_touched(a)) { synaptics_report_semi_mt_slot(dev, 0, true, a->x, a->y); synaptics_report_semi_mt_slot(dev, 1, false, 0, 0); } else { @@ -1040,8 +1045,10 @@ static void synaptics_process_packet(struct psmouse *psmouse) * BTN_TOUCH has to be first as mousedev relies on it when doing * absolute -> relative conversion */ - if (hw.z > 30) input_report_key(dev, BTN_TOUCH, 1); - if (hw.z < 25) input_report_key(dev, BTN_TOUCH, 0); + if (finger_touched(&hw)) + input_report_key(dev, BTN_TOUCH, 1); + if (hw.z < 25) + input_report_key(dev, BTN_TOUCH, 0); if (num_fingers > 0) { input_report_abs(dev, ABS_X, hw.x); -- 1.7.7.3 -- 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/