From patchwork Wed Dec 16 15:16:23 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tomasz Nowicki X-Patchwork-Id: 7863141 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 989819F387 for ; Wed, 16 Dec 2015 15:37:00 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 8539F20382 for ; Wed, 16 Dec 2015 15:36:59 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 86956202D1 for ; Wed, 16 Dec 2015 15:36:58 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1a9E6h-0001qE-Nt; Wed, 16 Dec 2015 15:35:11 +0000 Received: from casper.infradead.org ([2001:770:15f::2]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1a9E6f-0000kf-SM for linux-arm-kernel@bombadil.infradead.org; Wed, 16 Dec 2015 15:35:10 +0000 Received: from mail-wm0-x22b.google.com ([2a00:1450:400c:c09::22b]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1a9DqY-00023o-Cu for linux-arm-kernel@lists.infradead.org; Wed, 16 Dec 2015 15:18:31 +0000 Received: by mail-wm0-x22b.google.com with SMTP id p187so5706378wmp.0 for ; Wed, 16 Dec 2015 07:18:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=tQUjYncAluGDq/nwJ0yOnGE5FQOgY8GYwsZfs9O8haQ=; b=oPpwjW8vPi6quWxCKG9IkSf7fqUInVxLzi8SaKQfhT4WE/war4WHIfthvXW+XgIh05 m0YW85lZBsD9Yrw1SYMeX/aamVE3jograff2YcjmvsY+KWnb8VJDTwjrj4oDAFesW0Zy bO6wKgwqLOl7gdi1o6I0EuKel8LbIAv1Tz0MD5i0lQbXYklEy6nRQfKcLbCaiSavP1Wh LepdWTpkJetbbxyGaYVVdc92CuXRwkDgcn44Ga5ZfLDcrHKbTgf/b30AKRBQzyWGdIIg mcrGXWTKObHUSPzN+AaGLpoDKXvRFBueyeIzd4Sbos5UyrrxzQgerIhVhBboUhRW6wNE Jmww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=tQUjYncAluGDq/nwJ0yOnGE5FQOgY8GYwsZfs9O8haQ=; b=XdVq8Vxuc4j034X6u3HwVJ+eRT7F0NmmjbqoDhUuzEx7vY6TJGvBfncU5MK16tJPXK WDMy3YpqCOxztySH6GksWeTZmt4FfxVeXctBN1DHNUYAP+k3OBBBSuQZ6zB1vvOh0Nkr /ttjAwbSP+SXCcHR5RQ1AAr0Qj7kKSb04/v/OHoYO7+rVSw2ixxhuQpzo5WjJqpH9vnU 26iqbTrxOD+cDMafgzaXNFXyDaZXVh2fgr7WGP0/urnvzCBeY+hsXudK8wFGX/PtuMln 9CXkjZ3IX+UYDS5kk/TY9vOmKcvmSPVEj/771BEcLolZkIU+Y4+RY/cXvRZdWUDEeIJ2 pwRg== X-Gm-Message-State: ALoCoQnjXqX8G0/NGdU1Qgi5PmwkGMgcrZg9HWxoJkqlEFWn/R7gvaIRGTjzn46Smx9qrj6RiirP6PkAYgarEUCYlpe7JF+JjQ== X-Received: by 10.28.146.197 with SMTP id u188mr12385902wmd.78.1450279082847; Wed, 16 Dec 2015 07:18:02 -0800 (PST) Received: from tn-HP-4.semihalf.local ([80.82.22.190]) by smtp.gmail.com with ESMTPSA id z17sm6438761wjq.1.2015.12.16.07.18.01 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 16 Dec 2015 07:18:02 -0800 (PST) From: Tomasz Nowicki To: bhelgaas@google.com, arnd@arndb.de, will.deacon@arm.com, catalin.marinas@arm.com, rjw@rjwysocki.net, hanjun.guo@linaro.org, Lorenzo.Pieralisi@arm.com, okaya@codeaurora.org, jiang.liu@linux.intel.com, Stefano.Stabellini@eu.citrix.com Subject: [PATCH V2 13/23] x86, ia64, pci: Remove ACPI companion device from platform specific data. Date: Wed, 16 Dec 2015 16:16:23 +0100 Message-Id: <1450278993-12664-14-git-send-email-tn@semihalf.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1450278993-12664-1-git-send-email-tn@semihalf.com> References: <1450278993-12664-1-git-send-email-tn@semihalf.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20151216_151830_469087_4837B185 X-CRM114-Status: GOOD ( 15.07 ) X-Spam-Score: -2.6 (--) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: jchandra@broadcom.com, jcm@redhat.com, linaro-acpi@lists.linaro.org, linux-pci@vger.kernel.org, Liviu.Dudau@arm.com, ddaney@caviumnetworks.com, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, robert.richter@caviumnetworks.com, Suravee.Suthikulpanit@amd.com, msalter@redhat.com, wangyijing@huawei.com, Tomasz Nowicki , tglx@linutronix.de, mw@semihalf.com, linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-4.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED, T_DKIM_INVALID, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Since PCI core code is setting ACPI companion device for us now, platform specific ACPI companion device setting turns out to be dead now. Therefore we can get rid of it, including related companion reference from PCI sysdata structure. Signed-off-by: Tomasz Nowicki --- arch/ia64/include/asm/pci.h | 1 - arch/ia64/pci/pci.c | 16 ---------------- arch/x86/include/asm/pci.h | 3 --- arch/x86/pci/acpi.c | 2 -- arch/x86/pci/irq.c | 10 ---------- 5 files changed, 32 deletions(-) diff --git a/arch/ia64/include/asm/pci.h b/arch/ia64/include/asm/pci.h index 07039d1..5050748 100644 --- a/arch/ia64/include/asm/pci.h +++ b/arch/ia64/include/asm/pci.h @@ -65,7 +65,6 @@ extern int pci_mmap_legacy_page_range(struct pci_bus *bus, #define pci_legacy_write platform_pci_legacy_write struct pci_controller { - struct acpi_device *companion; void *iommu; int segment; int node; /* nearest node with memory or NUMA_NO_NODE for global allocation */ diff --git a/arch/ia64/pci/pci.c b/arch/ia64/pci/pci.c index 8f6ac2f..978d6af 100644 --- a/arch/ia64/pci/pci.c +++ b/arch/ia64/pci/pci.c @@ -301,28 +301,12 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) } info->controller.segment = root->segment; - info->controller.companion = device; info->controller.node = acpi_get_node(device->handle); INIT_LIST_HEAD(&info->io_resources); return acpi_pci_root_create(root, &pci_acpi_root_ops, &info->common, &info->controller); } -int pcibios_root_bridge_prepare(struct pci_host_bridge *bridge) -{ - /* - * We pass NULL as parent to pci_create_root_bus(), so if it is not NULL - * here, pci_create_root_bus() has been called by someone else and - * sysdata is likely to be different from what we expect. Let it go in - * that case. - */ - if (!bridge->dev.parent) { - struct pci_controller *controller = bridge->bus->sysdata; - ACPI_COMPANION_SET(&bridge->dev, controller->companion); - } - return 0; -} - void pcibios_fixup_device_resources(struct pci_dev *dev) { int idx; diff --git a/arch/x86/include/asm/pci.h b/arch/x86/include/asm/pci.h index 4625943..a98c022 100644 --- a/arch/x86/include/asm/pci.h +++ b/arch/x86/include/asm/pci.h @@ -14,9 +14,6 @@ struct pci_sysdata { int domain; /* PCI domain */ int node; /* NUMA node */ -#ifdef CONFIG_ACPI - struct acpi_device *companion; /* ACPI companion device */ -#endif #ifdef CONFIG_X86_64 void *iommu; /* IOMMU private data */ #endif diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c index 56714a9..286e0f5 100644 --- a/arch/x86/pci/acpi.c +++ b/arch/x86/pci/acpi.c @@ -333,7 +333,6 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) struct pci_sysdata sd = { .domain = domain, .node = node, - .companion = root->device }; memcpy(bus->sysdata, &sd, sizeof(sd)); @@ -348,7 +347,6 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) else { info->sd.domain = domain; info->sd.node = node; - info->sd.companion = root->device; bus = acpi_pci_root_create(root, &acpi_pci_root_ops, &info->common, &info->sd); } diff --git a/arch/x86/pci/irq.c b/arch/x86/pci/irq.c index 7032798..cc62226 100644 --- a/arch/x86/pci/irq.c +++ b/arch/x86/pci/irq.c @@ -1153,16 +1153,6 @@ void __init pcibios_irq_init(void) int pcibios_root_bridge_prepare(struct pci_host_bridge *bridge) { - /* - * We pass NULL as parent to pci_create_root_bus(), so if it is not NULL - * here, pci_create_root_bus() has been called by someone else and - * sysdata is likely to be different from what we expect. Let it go in - * that case. - */ - if (!bridge->dev.parent) { - struct pci_sysdata *sd = bridge->bus->sysdata; - ACPI_COMPANION_SET(&bridge->dev, sd->companion); - } bridge->map_irq = pci_map_irq; return 0; }