PCI: acpiphp: merge acpiphp_debug and debug
authorYinghai Lu <yinghai@kernel.org>
Sat, 23 Jun 2012 07:42:03 +0000 (00:42 -0700)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 10 Jul 2012 23:02:37 +0000 (17:02 -0600)
Should not have two, just remove debug, and use module_param_named
instead.

Also change acpiphp_debug to bool.

Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/hotplug/acpiphp.h
drivers/pci/hotplug/acpiphp_core.c

index 6b58ed0432e970da6033e8f30d7cec037eb4951b..a1afb5b39ad4d03b075a02326796b4e9a8045949 100644 (file)
@@ -205,6 +205,6 @@ extern u8 acpiphp_get_latch_status (struct acpiphp_slot *slot);
 extern u8 acpiphp_get_adapter_status (struct acpiphp_slot *slot);
 
 /* variables */
-extern int acpiphp_debug;
+extern bool acpiphp_debug;
 
 #endif /* _ACPIPHP_H */
index aa41631e9e02139db31203f8b960bc3b46769207..96316b74969f6e7588243e5b36839647319df077 100644 (file)
@@ -47,8 +47,7 @@
 /* name size which is used for entries in pcihpfs */
 #define SLOT_NAME_SIZE  21              /* {_SUN} */
 
-static bool debug;
-int acpiphp_debug;
+bool acpiphp_debug;
 
 /* local variables */
 static int num_slots;
@@ -62,7 +61,7 @@ MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);
 MODULE_LICENSE("GPL");
 MODULE_PARM_DESC(debug, "Debugging mode enabled or not");
-module_param(debug, bool, 0644);
+module_param_named(debug, acpiphp_debug, bool, 0644);
 
 /* export the attention callback registration methods */
 EXPORT_SYMBOL_GPL(acpiphp_register_attention);
@@ -379,8 +378,6 @@ static int __init acpiphp_init(void)
        if (acpi_pci_disabled)
                return 0;
 
-       acpiphp_debug = debug;
-
        /* read all the ACPI info from the system */
        return init_acpi();
 }