From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752982Ab0JKH2N (ORCPT ); Mon, 11 Oct 2010 03:28:13 -0400 Received: from cantor2.suse.de ([195.135.220.15]:34276 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750739Ab0JKH2L (ORCPT ); Mon, 11 Oct 2010 03:28:11 -0400 Date: Mon, 11 Oct 2010 09:28:10 +0200 Message-ID: From: Takashi Iwai To: Dmitry Torokhov Cc: Henrik Rydberg , Chase Douglas , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/3] Input: synaptics - multitouch and multifinger support In-Reply-To: <20101010205932.GB18455@core.coreip.homeip.net> References: <1286549880-32580-1-git-send-email-chase.douglas@canonical.com> <4CB17019.7070002@euromail.se> <20101010205932.GB18455@core.coreip.homeip.net> User-Agent: Wanderlust/2.15.6 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.7 Emacs/23.1 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Sun, 10 Oct 2010 13:59:32 -0700, Dmitry Torokhov wrote: > > On Sun, Oct 10, 2010 at 09:49:45AM +0200, Henrik Rydberg wrote: > > Takashi, Chase, > > > > talk about Heinz effect. :-) > > > > Rather than taking any of the patches, I was wondering if we could get a single > > patch including all the later findings and considerations on what devices should > > have the multitouch mode. Also, unless there is a really really good reason for > > it, without kernel parameters. After all, these patches should only add new > > functionality without regressions. > > Yes, I think we should limit number of module options (and even more so > Kconfig options). I believe that either MT support is ready for prime-time > and then it should be enabled unconditionally, or it is not ready and > should be kept as patches. his also means that we should not introduce > regressions in ST (even when emulated in MT mode). Right, but it's pretty hard to know whether it works really for all machines, simply because the current MT-detection is nothing but a pure guess work. As mentioned in another mail, there are devices that have no this bit but reports multi-finger detections without tracking. So, there are definitely other things to be checked. Thus, I'm inclined to add a module option, at least, for *disabling* MT for non-working machines. Also, another question is whether it's safe to enable MT even for user-space that don't understand MT. Takashi