From patchwork Thu Mar 12 17:35:40 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 5997701 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@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 208F29F380 for ; Thu, 12 Mar 2015 17:35:47 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 5FE3020373 for ; Thu, 12 Mar 2015 17:35:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D12C22037D for ; Thu, 12 Mar 2015 17:35:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161020AbbCLRfn (ORCPT ); Thu, 12 Mar 2015 13:35:43 -0400 Received: from mail-ob0-f169.google.com ([209.85.214.169]:38303 "EHLO mail-ob0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030318AbbCLRfm (ORCPT ); Thu, 12 Mar 2015 13:35:42 -0400 Received: by obcuz6 with SMTP id uz6so15521655obc.5 for ; Thu, 12 Mar 2015 10:35:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=subject:to:from:cc:date:message-id:in-reply-to:references :user-agent:mime-version:content-type:content-transfer-encoding; bh=B3B79OgrRv0BbeZexcwKUahMeI7a5apRYkPmhZNZN9g=; b=WjTV8oe+AXYHuJpqoLIDcHsWu+8QTyZ7fLomnL4K3z561QAsREwUdlEpaxp/mGiEEk ++WHx2Gajh4y18igkNnmCg3hnIvoPIR36gYmheetKXyVAWzcZR81AjtOptICvKtvKzqj qEtrLni3r1WB6F3FS17rMVpC9FSDqUWmbcguB4ccgaN6cPNdZbRwJgu4IhRKee90MMJV d4MhcojBbwKJUrxsyROGc6buc+EvzCDAMOX5vn6whlVMXNU40htXvsIm0Jivah6Da8qt bvyR9INM9VrZyqFcgALup6gk7m+4WVomoyC2g0MuSCBDwrBTlV7OqsSXzX3Rzh65mKzw QRFg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:subject:to:from:cc:date:message-id:in-reply-to :references:user-agent:mime-version:content-type :content-transfer-encoding; bh=B3B79OgrRv0BbeZexcwKUahMeI7a5apRYkPmhZNZN9g=; b=FKJiFNP+szbggF398fbTq2hXsZpKMd7wBCZigt3kuNOdpgh7voQq7lsT9SX55NXB2k 2KbJvV2QbTC0Z5NfgyLSjJmgUKlWTQl8jAJUnBgDUWBtY0lzSIon25Vnbtl4sBOOEfXS TOg4y90tk/EJtxAW5GkhT5bNjLnH51Ti5hDZuXtyRE9JB77X8SN0vcd5Xp2tVmNKu5Zv mzU89dVxs1AbAM0tRgIN5ieJ3e2snh4qPFpWO41/aqOhx6aF3z4cRNfo512KuTdqQ4LE ECc7qtP9syQoRH9+vizqSkjCBoM5PPvpDGbBTQ3GMyUOVrvj+Ft1AJcbpaE4YxH93g+I YUXw== X-Gm-Message-State: ALoCoQnv7qd7p1nXxGstT9hJLE0vNVcmFLeQtGID5KP5dZPGa8+fgEgemtiuZ6iIBVPs+aaYDsMY X-Received: by 10.60.132.144 with SMTP id ou16mr35141591oeb.36.1426181741784; Thu, 12 Mar 2015 10:35:41 -0700 (PDT) Received: from localhost ([146.7.4.76]) by mx.google.com with ESMTPSA id ku2sm4932684obc.12.2015.03.12.10.35.41 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Thu, 12 Mar 2015 10:35:41 -0700 (PDT) Subject: [PATCH v1 3/4] PCI: Show driver, BAR#, and resource on pci_ioremap_bar() failure To: linux-pci@vger.kernel.org From: Bjorn Helgaas Cc: linux-acpi@vger.kernel.org Date: Thu, 12 Mar 2015 12:35:40 -0500 Message-ID: <20150312173540.1052.39833.stgit@bhelgaas-glaptop2.roam.corp.google.com> In-Reply-To: <20150312173201.1052.28341.stgit@bhelgaas-glaptop2.roam.corp.google.com> References: <20150312173201.1052.28341.stgit@bhelgaas-glaptop2.roam.corp.google.com> User-Agent: StGit/0.16 MIME-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,RCVD_IN_DNSWL_HI,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 Use dev_warn() to complain about a pci_ioremap_bar() failure so we can include the driver name, BAR number, and the resource itself. We could use dev_WARN() to also get the backtrace as we did previously, but I think that's more information than we need. Signed-off-by: Bjorn Helgaas --- drivers/pci/pci.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 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/pci.c b/drivers/pci/pci.c index 81f06e8dcc04..a6d191ad9743 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -126,15 +126,16 @@ EXPORT_SYMBOL_GPL(pci_bus_max_busnr); #ifdef CONFIG_HAS_IOMEM void __iomem *pci_ioremap_bar(struct pci_dev *pdev, int bar) { + struct resource *res = &pdev->resource[bar]; + /* * Make sure the BAR is actually a memory resource, not an IO resource */ - if (!(pci_resource_flags(pdev, bar) & IORESOURCE_MEM)) { - WARN_ON(1); + if (!(res->flags & IORESOURCE_MEM)) { + dev_warn(&pdev->dev, "can't ioremap BAR %d: %pR\n", bar, res); return NULL; } - return ioremap_nocache(pci_resource_start(pdev, bar), - pci_resource_len(pdev, bar)); + return ioremap_nocache(res->start, resource_size(res)); } EXPORT_SYMBOL_GPL(pci_ioremap_bar); #endif