From patchwork Mon Jan 28 10:00:51 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Bolle X-Patchwork-Id: 2054731 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 1397D3FD1A for ; Mon, 28 Jan 2013 10:06:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751638Ab3A1KGD (ORCPT ); Mon, 28 Jan 2013 05:06:03 -0500 Received: from cpsmtpb-ews06.kpnxchange.com ([213.75.39.9]:63192 "EHLO cpsmtpb-ews06.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751577Ab3A1KGC (ORCPT ); Mon, 28 Jan 2013 05:06:02 -0500 X-Greylist: delayed 308 seconds by postgrey-1.27 at vger.kernel.org; Mon, 28 Jan 2013 05:06:02 EST Received: from cpsps-ews26.kpnxchange.com ([10.94.84.192]) by cpsmtpb-ews06.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Mon, 28 Jan 2013 10:59:40 +0100 Received: from CPSMTPM-TLF103.kpnxchange.com ([195.121.3.6]) by cpsps-ews26.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Mon, 28 Jan 2013 10:59:40 +0100 Received: from [192.168.1.104] ([212.123.139.93]) by CPSMTPM-TLF103.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Mon, 28 Jan 2013 11:00:51 +0100 Message-ID: <1359367251.883.13.camel@x61.thuisdomein> Subject: [PATCH] PCI: pciehp: hide ENTRY messages behind ctrl_dbg() From: Paul Bolle To: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 28 Jan 2013 11:00:51 +0100 X-Mailer: Evolution 3.4.4 (3.4.4-2.fc17) Mime-Version: 1.0 X-OriginalArrivalTime: 28 Jan 2013 10:00:52.0017 (UTC) FILETIME=[5B007A10:01CDFD3E] X-RcptDomain: vger.kernel.org Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org In each suspend and resume cycle my laptop prints these messages at KERN_INFO level: pciehp 0000:00:1c.1:pcie04: pciehp_suspend ENTRY pciehp 0000:00:1c.0:pcie04: pciehp_suspend ENTRY and pciehp 0000:00:1c.0:pcie04: pciehp_resume ENTRY pciehp 0000:00:1c.1:pcie04: pciehp_resume ENTRY Messages like these are probably only useful while debugging pciehp, so let's hide them behind the ctrl_dbg() macro. Signed-off-by: Paul Bolle --- 0) Compile tested only! 1) Perhaps a better solution is to drop these messages entirely. drivers/pci/hotplug/pciehp_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/pci/hotplug/pciehp_core.c b/drivers/pci/hotplug/pciehp_core.c index 939bd1d..ed1702d 100644 --- a/drivers/pci/hotplug/pciehp_core.c +++ b/drivers/pci/hotplug/pciehp_core.c @@ -293,7 +293,8 @@ static void pciehp_remove(struct pcie_device *dev) #ifdef CONFIG_PM static int pciehp_suspend (struct pcie_device *dev) { - dev_info(&dev->device, "%s ENTRY\n", __func__); + struct controller *ctrl = get_service_data(dev); + ctrl_dbg(ctrl, "%s ENTRY\n", __func__); return 0; } @@ -303,8 +304,8 @@ static int pciehp_resume (struct pcie_device *dev) struct slot *slot; u8 status; - dev_info(&dev->device, "%s ENTRY\n", __func__); ctrl = get_service_data(dev); + ctrl_dbg(ctrl, "%s ENTRY\n", __func__); /* reinitialize the chipset's event detection logic */ pcie_enable_notification(ctrl);