From patchwork Fri Dec 7 06:25:25 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Myron Stowe X-Patchwork-Id: 1848891 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 3F6C4DF2EE for ; Fri, 7 Dec 2012 06:28:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1946005Ab2LGGZ3 (ORCPT ); Fri, 7 Dec 2012 01:25:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:61930 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1946003Ab2LGGZ2 (ORCPT ); Fri, 7 Dec 2012 01:25:28 -0500 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id qB76PPMO010488 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Fri, 7 Dec 2012 01:25:25 -0500 Received: from amt.stowe ([10.3.113.3]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id qB76PPcf010453; Fri, 7 Dec 2012 01:25:25 -0500 From: Myron Stowe Subject: [PATCH 05/15] PCI/acpiphp: Collapse initialization call chain of "acpiphp" sub-driver To: bhelgaas@google.com Cc: linux-pci@vger.kernel.org, yinghai@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 06 Dec 2012 23:25:25 -0700 Message-ID: <20121207062524.11051.46916.stgit@amt.stowe> In-Reply-To: <20121207062454.11051.12739.stgit@amt.stowe> References: <20121207062454.11051.12739.stgit@amt.stowe> User-Agent: StGIT/0.14.3 MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@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);