From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754661AbZCICTw (ORCPT ); Sun, 8 Mar 2009 22:19:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752122AbZCICTn (ORCPT ); Sun, 8 Mar 2009 22:19:43 -0400 Received: from mga01.intel.com ([192.55.52.88]:22235 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751749AbZCICTm (ORCPT ); Sun, 8 Mar 2009 22:19:42 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.38,326,1233561600"; d="scan'208";a="671329644" Subject: Re: [PATCH] acpi: simplify module_param namespace From: yakui_zhao To: Rusty Russell Cc: "lenb@kernel.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" In-Reply-To: <200903071328.28113.rusty@rustcorp.com.au> References: <200903071328.28113.rusty@rustcorp.com.au> Content-Type: text/plain Organization: Intel Open Source Technology Center Date: Mon, 09 Mar 2009 10:20:44 +0800 Message-Id: <1236565244.7060.7.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.22.1 (2.22.1-2.fc9) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2009-03-07 at 10:58 +0800, Rusty Russell wrote: > (I've been holding this for a while: if you really hate it I'll delete it) > > Rather than overriding MODULE_PARAM_PREFIX, build via acpi.o so > KBUILD_MODNAME is set to "acpi". > > This is the logical way to do it, even though acpi cannot be a module > due to these config options being bool. > > Signed-off-by: Rusty Russell > --- > drivers/acpi/Makefile | 10 ++++++---- > drivers/acpi/debug.c | 5 ----- > drivers/acpi/power.c | 4 ---- > drivers/acpi/system.c | 4 ---- > 4 files changed, 6 insertions(+), 17 deletions(-) > > diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile > --- a/drivers/acpi/Makefile > +++ b/drivers/acpi/Makefile > @@ -14,8 +14,13 @@ obj-$(CONFIG_X86) += blacklist.o > # > # ACPI Core Subsystem (Interpreter) > # > -obj-y += osl.o utils.o reboot.o\ > +obj-y += acpi.o \ > acpica/ If so, it can't be guaranteed that the EC driver is initialized before the power resource driver. > + > +# "acpi." module_param namespace. > +acpi-y += osl.o utils.o reboot.o power.o \ > + system.o event.o > +acpi-$(CONFIG_ACPI_DEBUG) += debug.o > > # sleep related files > obj-y += wakeup.o > @@ -51,9 +56,6 @@ obj-$(CONFIG_ACPI_PROCESSOR) += processo > obj-$(CONFIG_ACPI_PROCESSOR) += processor.o > obj-$(CONFIG_ACPI_CONTAINER) += container.o > obj-$(CONFIG_ACPI_THERMAL) += thermal.o > -obj-y += power.o > -obj-y += system.o event.o > -obj-$(CONFIG_ACPI_DEBUG) += debug.o > obj-$(CONFIG_ACPI_NUMA) += numa.o > obj-$(CONFIG_ACPI_HOTPLUG_MEMORY) += acpi_memhotplug.o > obj-$(CONFIG_ACPI_PROCFS_POWER) += cm_sbs.o > diff --git a/drivers/acpi/debug.c b/drivers/acpi/debug.c > --- a/drivers/acpi/debug.c > +++ b/drivers/acpi/debug.c > @@ -12,11 +12,6 @@ > > #define _COMPONENT ACPI_SYSTEM_COMPONENT > ACPI_MODULE_NAME("debug"); > - > -#ifdef MODULE_PARAM_PREFIX > -#undef MODULE_PARAM_PREFIX > -#endif > -#define MODULE_PARAM_PREFIX "acpi." > > struct acpi_dlayer { > const char *name; > diff --git a/drivers/acpi/power.c b/drivers/acpi/power.c > --- a/drivers/acpi/power.c > +++ b/drivers/acpi/power.c > @@ -54,10 +54,6 @@ ACPI_MODULE_NAME("power"); > #define ACPI_POWER_RESOURCE_STATE_ON 0x01 > #define ACPI_POWER_RESOURCE_STATE_UNKNOWN 0xFF > > -#ifdef MODULE_PARAM_PREFIX > -#undef MODULE_PARAM_PREFIX > -#endif > -#define MODULE_PARAM_PREFIX "acpi." > int acpi_power_nocheck; > module_param_named(power_nocheck, acpi_power_nocheck, bool, 000); > > diff --git a/drivers/acpi/system.c b/drivers/acpi/system.c > --- a/drivers/acpi/system.c > +++ b/drivers/acpi/system.c > @@ -33,10 +33,6 @@ > > #define _COMPONENT ACPI_SYSTEM_COMPONENT > ACPI_MODULE_NAME("system"); > -#ifdef MODULE_PARAM_PREFIX > -#undef MODULE_PARAM_PREFIX > -#endif > -#define MODULE_PARAM_PREFIX "acpi." > > #define ACPI_SYSTEM_CLASS "system" > #define ACPI_SYSTEM_DEVICE_NAME "System" > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html