diff mbox

acpi: make ACPI_DOCK a tristate option again

Message ID 4AA0E2BE02000078000136C7@vpn.id2.novell.com (mailing list archive)
State RFC, archived
Headers show

Commit Message

Jan Beulich Sept. 4, 2009, 7:49 a.m. UTC
I cannot follow the explanation in commit
898b054f3eec5921320ae8614b5bdd7b07ea5b43: If other drivers require the
dock module to be loaded before them, they sould simply have a
dependency on dock.ko. In the particular case mentioned (libata), this
already is the case.

However, in order to dock to really be compilable as a module, the
select from libata also needs to be corrected: Boolean sub-options of
tristate options selecting another tristate option should not do the
select themselves, as that forces the selected option to Y. Instead,
a conditional select in the base tristate option is needed.

Signed-off-by: Jan Beulich <jbeulich@novell.com>
Cc: Shaohua Li <shaohua.li@intel.com>

---
 drivers/acpi/Kconfig  |    2 +-
 drivers/acpi/Makefile |    2 +-
 drivers/acpi/osl.c    |    1 +
 drivers/ata/Kconfig   |    2 +-
 4 files changed, 4 insertions(+), 3 deletions(-)




--
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
diff mbox

Patch

--- linux-2.6.31-rc8/drivers/acpi/Kconfig	2009-08-28 14:54:27.000000000 +0200
+++ 2.6.31-rc8-acpi-dock-module/drivers/acpi/Kconfig	2009-08-19 13:28:12.000000000 +0200
@@ -171,7 +171,7 @@  config ACPI_FAN
 	  the module will be called fan.
 
 config ACPI_DOCK
-	bool "Dock"
+	tristate "Dock"
 	depends on EXPERIMENTAL
 	help
 	  This driver supports ACPI-controlled docking stations and removable
--- linux-2.6.31-rc8/drivers/acpi/Makefile	2009-06-10 05:05:27.000000000 +0200
+++ 2.6.31-rc8-acpi-dock-module/drivers/acpi/Makefile	2009-08-19 13:31:37.000000000 +0200
@@ -32,7 +32,6 @@  acpi-$(CONFIG_ACPI_SLEEP)	+= proc.o
 acpi-y				+= bus.o glue.o
 acpi-y				+= scan.o
 acpi-y				+= ec.o
-acpi-$(CONFIG_ACPI_DOCK)	+= dock.o
 acpi-y				+= pci_root.o pci_link.o pci_irq.o pci_bind.o
 acpi-y				+= power.o
 acpi-y				+= system.o event.o
@@ -54,6 +53,7 @@  obj-$(CONFIG_ACPI_CONTAINER)	+= containe
 obj-$(CONFIG_ACPI_THERMAL)	+= thermal.o
 obj-$(CONFIG_ACPI_HOTPLUG_MEMORY) += acpi_memhotplug.o
 obj-$(CONFIG_ACPI_BATTERY)	+= battery.o
+obj-$(CONFIG_ACPI_DOCK)		+= dock.o
 obj-$(CONFIG_ACPI_SBS)		+= sbshc.o
 obj-$(CONFIG_ACPI_SBS)		+= sbs.o
 
--- linux-2.6.31-rc8/drivers/acpi/osl.c	2009-08-28 14:54:27.000000000 +0200
+++ 2.6.31-rc8-acpi-dock-module/drivers/acpi/osl.c	2009-09-04 08:47:50.000000000 +0200
@@ -803,6 +803,7 @@  acpi_status acpi_os_hotplug_execute(acpi
 {
 	return __acpi_os_execute(0, function, context, 1);
 }
+EXPORT_SYMBOL_GPL(acpi_os_hotplug_execute);
 
 void acpi_os_wait_events_complete(void *context)
 {
--- linux-2.6.31-rc8/drivers/ata/Kconfig	2009-08-28 14:54:27.000000000 +0200
+++ 2.6.31-rc8-acpi-dock-module/drivers/ata/Kconfig	2009-08-19 13:28:35.000000000 +0200
@@ -8,6 +8,7 @@  menuconfig ATA
 	depends on BLOCK
 	depends on !(M32R || M68K) || BROKEN
 	select SCSI
+	select ACPI_DOCK if ATA_ACPI
 	---help---
 	  If you want to use a ATA hard disk, ATA tape drive, ATA CD-ROM or
 	  any other ATA device under Linux, say Y and make sure that you know
@@ -29,7 +30,6 @@  config ATA_NONSTANDARD
 config ATA_ACPI
 	bool "ATA ACPI Support"
 	depends on ACPI && PCI
-	select ACPI_DOCK
 	default y
 	help
 	  This option adds support for ATA-related ACPI objects.