From patchwork Wed Jun 9 18:15:34 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Leann Ogasawara X-Patchwork-Id: 55117 X-Patchwork-Delegate: leann.ogasawara@canonical.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from chlorine.canonical.com (chlorine.canonical.com [91.189.94.204]) by ozlabs.org (Postfix) with ESMTP id 4E3C81007D3 for ; Thu, 10 Jun 2010 04:15:53 +1000 (EST) Received: from localhost ([127.0.0.1] helo=chlorine.canonical.com) by chlorine.canonical.com with esmtp (Exim 4.69) (envelope-from ) id 1OMPoU-0006aA-8D; Wed, 09 Jun 2010 19:15:42 +0100 Received: from adelie.canonical.com ([91.189.90.139]) by chlorine.canonical.com with esmtp (Exim 4.69) (envelope-from ) id 1OMPoS-0006Yl-QC for kernel-team@lists.ubuntu.com; Wed, 09 Jun 2010 19:15:40 +0100 Received: from hutte.canonical.com ([91.189.90.181]) by adelie.canonical.com with esmtp (Exim 4.69 #1 (Debian)) id 1OMPoP-0000ez-Uc; Wed, 09 Jun 2010 19:15:38 +0100 Received: from c-76-105-148-120.hsd1.or.comcast.net ([76.105.148.120] helo=[192.168.1.4]) by hutte.canonical.com with esmtpsa (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from ) id 1OMPoP-0007b8-Hh; Wed, 09 Jun 2010 19:15:37 +0100 Subject: [RFC] [Maverick] UBUNTU: SAUCE: Add MODULE_ALIAS for Dell WMI module From: Leann Ogasawara To: Mario Limonciello Date: Wed, 09 Jun 2010 11:15:34 -0700 Message-ID: <1276107334.1221.556.camel@emiko> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Cc: kernel-team X-BeenThere: kernel-team@lists.ubuntu.com X-Mailman-Version: 2.1.9 Precedence: list List-Id: Kernel team discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kernel-team-bounces@lists.ubuntu.com Errors-To: kernel-team-bounces@lists.ubuntu.com Hi Mario, We've been temporarily carrying an Ubuntu SAUCE patch (inlined at the bottom), originally authored by yourself. In the commit message you'd noted that upstream was not in favor of applying this patch but instead would prefer to wait for the WMI sysfs interface to be incorporated. It seems that this has now been added as of 2.6.33-rc1: commit 1caab3c1a90be3aa4ec3599409d8fe044b077478 Author: Matthew Garrett Date: Wed Nov 4 14:17:53 2009 -0500 wmi: Add support for module autoloading I assume it is now safe for us to drop the temporary patch we've been carrying that you had originally authored? I see that acer-wmi already dropped similar bits it was carrying: commit 4d5f177caf1ef78f5e7eaee75b66b4d534445682 Author: Thomas Renninger Date: Fri Dec 18 15:29:22 2009 +0100 acer-wmi, msi-wmi: Remove needless DMI MODULE_ALIAS Now that we have WMI autoloading the DMI matching is not needed anymore. Thanks, Leann >From a0d5c1898b6ad79e5eb1d26edaf17cbf7f48ace8 Mon Sep 17 00:00:00 2001 From: Tim Gardner Date: Wed, 13 May 2009 11:38:45 -0600 Subject: [PATCH] UBUNTU: SAUCE: Add MODULE_ALIAS for Dell WMI module >From Mario Limonciello: Until the sysfs WMI interface is ready, there is no way to automatically determine when to load dell-wmi. The equivalent module for Acer laptops, acer-wmi has a workaround in place to match on the DMI tables for the machine. This type of solution works on for Dell as well and gets the module loading when necessary. Upstream is not in favor of applying this patch, but rather waiting for the sysfs interface to be ready. Until it is, this provides a solution to ensure machines that support WMI hotkeys are functional. OriginalAuthor: mario_limonciello@dell.com Signed-off-by: Tim Gardner --- drivers/platform/x86/dell-wmi.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c index 66f53c3..830be3c 100644 --- a/drivers/platform/x86/dell-wmi.c +++ b/drivers/platform/x86/dell-wmi.c @@ -44,6 +44,10 @@ static int acpi_video; MODULE_ALIAS("wmi:"DELL_EVENT_GUID); +/* Temporary workaround until the WMI sysfs interface goes in. + Borrowed from acer-wmi */ +MODULE_ALIAS("dmi:*:*Dell*:*:"); + struct key_entry { char type; /* See KE_* below */ u16 code;