Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758871AbXHPM2j (ORCPT ); Thu, 16 Aug 2007 08:28:39 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754598AbXHPM2c (ORCPT ); Thu, 16 Aug 2007 08:28:32 -0400 Received: from ug-out-1314.google.com ([66.249.92.172]:41019 "EHLO ug-out-1314.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754561AbXHPM2b (ORCPT ); Thu, 16 Aug 2007 08:28:31 -0400 Date: Thu, 16 Aug 2007 15:28:23 +0300 From: "Michael S. Tsirkin" To: Henrique de Moraes Holschuh Cc: "Michael S. Tsirkin" , lenb@kernel.org, Hugh Dickins , Linux Kernel Mailing List , Richard Hughes Subject: Re: [PATCH 3/3] ACPI: thinkpad-acpi: change thinkpad-acpi input default and kconfig help Message-ID: <20070816122823.GH5684@mellanox.co.il> Reply-To: "Michael S. Tsirkin" References: <11868017142597-git-send-email-hmh@hmh.eng.br> <11868017143582-git-send-email-hmh@hmh.eng.br> <20070811180602.GA17843@mellanox.co.il> <20070811221625.GB27761@khazad-dum.debian.net> <20070816073053.GA19833@mellanox.co.il> <20070816122506.GA5081@khazad-dum.debian.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070816122506.GA5081@khazad-dum.debian.net> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1988 Lines: 41 > Quoting Henrique de Moraes Holschuh : > Subject: Re: [PATCH 3/3] ACPI: thinkpad-acpi: change thinkpad-acpi input default and kconfig help > > On Thu, 16 Aug 2007, Michael S. Tsirkin wrote: > > > I *can* make the compile-time option a module parameter, though. That > > > wouldn't be much of a problem, and the compile-time option would select the > > > default for that parameter, but it would be easier to override it at run > > > time (you can already do it, but it requires reprogramming various driver > > > parameters using sysfs and the input device IOCTLs). > > > > > > Would a module parameter address enough of your concerns? Kconfig would > > > only set the default for that parameter, then... > > > > I really think modifying defaults it not a great idea. > > What about adding a writeable module parameter? > > Then new HAL would just write to /sysfs/module/thinkpad/parameters/interface_version to > > switch to new behaviour. > > Nah. This thing regulates what the module has to do when it loads and inits > the firmware, This does not have to be at module load time, though, does it? > and the whole point of the new behaviour is to get userspace > to be as hands' off as possible, so that we can auto-tune things better > in-kernel, without a need to worry about HAL and friends overwriting local > admin configurations needlessly. > > I will probably have to do something like the strict versioned interfaces of > wireless extensions, I think. It really is a case of a deprecation of > current userland interfaces done sanely. Right now I am thinking about > giving userland two kernel releases before I change the default interface > level, and one to two years to remove the older ones. -- MST - 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/