From patchwork Fri Dec 1 14:08:12 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Rafael J. Wysocki" X-Patchwork-Id: 843506 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-pci-owner@vger.kernel.org; receiver=) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3ypGMy6Zm2z9t9m for ; Sat, 2 Dec 2017 01:08:46 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751311AbdLAOIo (ORCPT ); Fri, 1 Dec 2017 09:08:44 -0500 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:62295 "EHLO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751105AbdLAOIn (ORCPT ); Fri, 1 Dec 2017 09:08:43 -0500 Received: from 79.184.254.73.ipv4.supernova.orange.pl (79.184.254.73) (HELO aspire.rjw.lan) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.82) id 610f5ee532ea72e3; Fri, 1 Dec 2017 15:08:41 +0100 From: "Rafael J. Wysocki" To: Linux PCI Cc: LKML , Bjorn Helgaas , x86 , Dave Hansen Subject: [PATCH] x86 / PCI: Make broadcom_postcore_init() check acpi_disabled Date: Fri, 01 Dec 2017 15:08:12 +0100 Message-ID: <3186627.pxZj1QbYNg@aspire.rjw.lan> In-Reply-To: <1593230.4rVA5fLXq5@aspire.rjw.lan> References: <1593230.4rVA5fLXq5@aspire.rjw.lan> MIME-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org From: Rafael J. Wysocki acpi_os_get_root_pointer() may return a valid address even if acpi_disabled is set, but the host bridge information from the ACPI tables is not going to be used in that case and the Broadcom host bridge initialization should not be skipped then, so make broadcom_postcore_init() check acpi_disabled too to avoid this issue. Reported-by: Dave Hansen Fixes: 6361d72b04d1 (x86/PCI: read Broadcom CNB20LE host bridge info before PCI scan) Signed-off-by: Rafael J. Wysocki --- Resend with the correct address of the x86 list. Sorry for the duplication. --- arch/x86/pci/broadcom_bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/arch/x86/pci/broadcom_bus.c =================================================================== --- linux-pm.orig/arch/x86/pci/broadcom_bus.c +++ linux-pm/arch/x86/pci/broadcom_bus.c @@ -97,7 +97,7 @@ static int __init broadcom_postcore_init * We should get host bridge information from ACPI unless the BIOS * doesn't support it. */ - if (acpi_os_get_root_pointer()) + if (!acpi_disabled && acpi_os_get_root_pointer()) return 0; #endif