From patchwork Fri Dec 7 06:25:25 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [05/15] PCI/acpiphp: Collapse initialization call chain of "acpiphp" sub-driver Date: Thu, 06 Dec 2012 20:25:25 -0000 From: Myron Stowe X-Patchwork-Id: 204404 Message-Id: <20121207062524.11051.46916.stgit@amt.stowe> To: bhelgaas@google.com Cc: linux-pci@vger.kernel.org, yinghai@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org This patch collapses the initialization call chain of "acpiphp" by calling acpiphp_glue_init() directly within acphphp_init(), removing the intervening init_acpi() call. No functional change. Signed-off-by: Myron Stowe --- drivers/pci/hotplug/acpiphp_core.c | 24 +++++++----------------- 1 files changed, 7 insertions(+), 17 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/pci/hotplug/acpiphp_core.c b/drivers/pci/hotplug/acpiphp_core.c index 011a538..1f4e228 100644 --- a/drivers/pci/hotplug/acpiphp_core.c +++ b/drivers/pci/hotplug/acpiphp_core.c @@ -271,14 +271,6 @@ static int get_adapter_status(struct hotplug_slot *hotplug_slot, u8 *value) return 0; } -static int __init init_acpi(void) -{ - /* initialize internal data structure etc. */ - acpiphp_glue_init(); - - return 0; -} - /** * release_slot - free up the memory used by a slot * @hotplug_slot: slot to free @@ -363,21 +355,19 @@ static int __init acpiphp_init(void) { info(DRIVER_DESC " version: " DRIVER_VERSION "\n"); - if (acpi_pci_disabled) - return 0; + if (!acpi_pci_disabled) + /* initialize internal data structure etc. */ + acpiphp_glue_init(); - /* read all the ACPI info from the system */ - return init_acpi(); + return 0; } static void __exit acpiphp_exit(void) { - if (acpi_pci_disabled) - return; - - /* deallocate internal data structures etc. */ - acpiphp_glue_exit(); + if (!acpi_pci_disabled) + /* deallocate internal data structures etc. */ + acpiphp_glue_exit(); } module_init(acpiphp_init);