From patchwork Wed Dec 5 20:58:06 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 1843191 Return-Path: X-Original-To: patchwork-linux-wireless@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 9A4343FC71 for ; Wed, 5 Dec 2012 21:05:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753975Ab2LEVFd (ORCPT ); Wed, 5 Dec 2012 16:05:33 -0500 Received: from mail-lb0-f202.google.com ([209.85.217.202]:47100 "EHLO mail-lb0-f202.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753858Ab2LEVFd (ORCPT ); Wed, 5 Dec 2012 16:05:33 -0500 Received: by mail-lb0-f202.google.com with SMTP id s4so224474lbc.1 for ; Wed, 05 Dec 2012 13:05:31 -0800 (PST) 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=RA4hQcAjlyMUAnKCMi/hFgO4WL+7ViRpODap3UptT8Y=; b=MxXFubhpQC0TmswArUTQbAu77iiVL0BTMvSo1QOLH2GigTHlNmYgTjARG663EtX6I4 NV8cCvmKbUzZ7PDgi8uB5MWNDI8Re/FVyf9Xj3CwbeLw/cS3lT1nI7IQMymJAJXHjAFv 0PdmIxnRB0BO04+hbkV9mKPOHmkrUGkyo8HNvc95JdgG1K4saheReMF9zfQf8IJpharo jjeDUlEGXnWn7/SFxVP11aeXTMlP73xzbdbtbRWQwWaE4cCeigcSLMLBiz8MegSBRG4a yN4ljDmZei/UWvF39jTZYCxJVJV29NVVZs2pzbkcUfy7M3CfQDsrMkyV5lTvHrnnylER CpHw== X-Google-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 :x-gm-message-state; bh=RA4hQcAjlyMUAnKCMi/hFgO4WL+7ViRpODap3UptT8Y=; b=SRwf+34sDG5bZvi8F7nEvPWiI3bAagma8iug5seXAKpBXlxalYuKCym737OMk4ppLD +T+yA5HAZXI5mIz1xQhiCZiyTl+nUbjBKKrakbfSEE2LKA9zf+xL2geDHL46SfBxPmL5 3Hd/NzinFCBBLK22XI8v9VSuVDJdx0hUHHIPcYIBMYGPf3R1aE1fUKLIAGEi1mztJG/3 AdcbafVKVSX+HS04VONgeGuC2iidmSctLvHXwT+L92YbjjKDjbPbxgp602cn9khorNGx Yc52Gj0hled2h5zgahzGsKo85NhvTqBlgbxcRJITBt6XUBnR/uNqMD0BrYNI4oCtySE0 XuBg== Received: by 10.14.173.136 with SMTP id v8mr19283975eel.2.1354741087357; Wed, 05 Dec 2012 12:58:07 -0800 (PST) Received: from hpza10.eem.corp.google.com ([74.125.121.33]) by gmr-mx.google.com with ESMTPS id g9si765218eeo.1.2012.12.05.12.58.07 (version=TLSv1/SSLv3 cipher=AES128-SHA); Wed, 05 Dec 2012 12:58:07 -0800 (PST) Received: from bhelgaas.mtv.corp.google.com (bhelgaas.mtv.corp.google.com [172.17.131.112]) by hpza10.eem.corp.google.com (Postfix) with ESMTP id 009D5200059; Wed, 5 Dec 2012 12:58:07 -0800 (PST) Received: from bhelgaas.mtv.corp.google.com (unknown [IPv6:::1]) by bhelgaas.mtv.corp.google.com (Postfix) with ESMTP id 4AEB2180280; Wed, 5 Dec 2012 12:58:06 -0800 (PST) Subject: [PATCH 09/12] iwlegacy: collapse wrapper for pcie_capability_read_word() To: linux-pci@vger.kernel.org From: Bjorn Helgaas Cc: linux-wireless@vger.kernel.org, Stanislaw Gruszka , "John W. Linville" Date: Wed, 05 Dec 2012 13:58:06 -0700 Message-ID: <20121205205806.13851.18014.stgit@bhelgaas.mtv.corp.google.com> In-Reply-To: <20121205205724.13851.50508.stgit@bhelgaas.mtv.corp.google.com> References: <20121205205724.13851.50508.stgit@bhelgaas.mtv.corp.google.com> User-Agent: StGit/0.15 MIME-Version: 1.0 X-Gm-Message-State: ALoCoQm2lMqX/zyGsro5ZMjTyn9yAvkmP663yCzHPLFaFejogq0D7KUNO0L3CKEdL6jJPwTqNF5aPNH2WsrSD7nyJDBapmb0EZFqe3eLdQ7UDGU5SYb1cqFsK07DgieddTQT3eIkN/arAoOQfD9QTwi+bEvGNGWoX4fotAeSeDhmuN26CwDzs9+nuC65HXkuD/ahQWl9JW8P+LDX9Zg+keymvljhGzsucw== Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org il_pcie_link_ctl() has only one call site and no longer provides any useful abstraction, so collapse it into the caller. Signed-off-by: Bjorn Helgaas CC: "John W. Linville" CC: Stanislaw Gruszka CC: linux-wireless@vger.kernel.org Acked-by: Stanislaw Gruszka --- drivers/net/wireless/iwlegacy/common.c | 5 +++-- drivers/net/wireless/iwlegacy/common.h | 8 -------- 2 files changed, 3 insertions(+), 10 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" 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/net/wireless/iwlegacy/common.c b/drivers/net/wireless/iwlegacy/common.c index 318ed3c..1811507 100644 --- a/drivers/net/wireless/iwlegacy/common.c +++ b/drivers/net/wireless/iwlegacy/common.c @@ -1183,8 +1183,9 @@ EXPORT_SYMBOL(il_power_update_mode); void il_power_initialize(struct il_priv *il) { - u16 lctl = il_pcie_link_ctl(il); + u16 lctl; + pcie_capability_read_word(il->pci_dev, PCI_EXP_LNKCTL, &lctl); il->power_data.pci_pm = !(lctl & PCI_CFG_LINK_CTRL_VAL_L0S_EN); il->power_data.debug_sleep_level_override = -1; @@ -4233,7 +4234,7 @@ il_apm_init(struct il_priv *il) * power savings, even without L1. */ if (il->cfg->set_l0s) { - lctl = il_pcie_link_ctl(il); + pcie_capability_read_word(il->pci_dev, PCI_EXP_LNKCTL, &lctl); if ((lctl & PCI_CFG_LINK_CTRL_VAL_L1_EN) == PCI_CFG_LINK_CTRL_VAL_L1_EN) { /* L1-ASPM enabled; disable(!) L0S */ diff --git a/drivers/net/wireless/iwlegacy/common.h b/drivers/net/wireless/iwlegacy/common.h index b4bb813..c9a5022 100644 --- a/drivers/net/wireless/iwlegacy/common.h +++ b/drivers/net/wireless/iwlegacy/common.h @@ -1829,14 +1829,6 @@ int il_enqueue_hcmd(struct il_priv *il, struct il_host_cmd *cmd); * PCI * *****************************************************/ -static inline u16 -il_pcie_link_ctl(struct il_priv *il) -{ - u16 pci_lnk_ctl; - pcie_capability_read_word(il->pci_dev, PCI_EXP_LNKCTL, &pci_lnk_ctl); - return pci_lnk_ctl; -} - void il_bg_watchdog(unsigned long data); u32 il_usecs_to_beacons(struct il_priv *il, u32 usec, u32 beacon_interval); __le32 il_add_beacon_time(struct il_priv *il, u32 base, u32 addon,