From patchwork Mon Aug 7 10:41:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 109546 Delivered-To: patch@linaro.org Received: by 10.140.95.78 with SMTP id h72csp1434505qge; Mon, 7 Aug 2017 03:42:51 -0700 (PDT) X-Received: by 10.84.232.13 with SMTP id h13mr105810plk.168.1502102571809; Mon, 07 Aug 2017 03:42:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1502102571; cv=none; d=google.com; s=arc-20160816; b=jQRGLBHle+6nPFG1lDHwKqGCyA6q4MUxc+DIJbEd42JC+LFZUJxw3qlbHiUCw9ds85 4qxhmDyCS4uOTm4H94TiX5rip8nhIHlZIwLKI+rUKKPo+CwOb/sxtW+zlhy0VK2uYr9W cUK/Mpda99vAdn7QhmbMi0D6u7IBN3KqMgAO365PYt3HVANRBI1QUM7uAHtxWDa9zb8G +utGCxGoyNKr/CQEz8YYIDcz847yML2wUX/AV1HmBfe0O8L+Wmu4IaL0Syh5eCDcfL3R EQ5hRCrMHSPS3pQW8G/ADmRP4nYhud2APovqWtK5Lpc3XjxAZTeW+BCG53Rtstl/f/ZA HL4Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=D5ntkeoBudD+nl6Dc83P/hExbTDJwCRrDvRT4z6j2Cg=; b=PqoZ1U2Kg1EeaXuP4Vm+vhUBgfRdzuTbzwtVBm6kMs2HPXmJP+Dj8Wux+vpqfJ/oMs yUaTE1Ip8VZi5S39kzP58hNL6NlF21W2MRDKKrr7PA3/pjnGTi/ylB70BeYuoDghbxq1 bjXAkMbWg3JOgwYQKbH0x1IfzAj51ubzx4+DaP+kf4TNhg/X0DtpQIRv9ANUbjed4ISf Ce4LniZthjNRoKRL4sODPrAqUfHtyxLM7QRCAsYFhqbAOam8G66f70fBAx45igp8Z3N0 tATFLJUM0pxxNG3E1C7eAYMajqxHUcoDuQsN4hJq70dfLYajfYh1cEBnKzKHh3zV6tsF 0ysA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 1si4464082pgs.57.2017.08.07.03.42.51; Mon, 07 Aug 2017 03:42:51 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753045AbdHGKmt (ORCPT + 25 others); Mon, 7 Aug 2017 06:42:49 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:58066 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752795AbdHGKmr (ORCPT ); Mon, 7 Aug 2017 06:42:47 -0400 Received: from wuerfel.lan ([78.43.238.10]) by mrelayeu.kundenserver.de (mreue102 [212.227.15.145]) with ESMTPA (Nemesis) id 0MCILJ-1dnmJP29jd-009Bjr; Mon, 07 Aug 2017 12:42:34 +0200 From: Arnd Bergmann To: Yisen Zhuang , Salil Mehta , "David S. Miller" Cc: Arnd Bergmann , "Wei Hu (Xavier)" , lipeng , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] hns3: fix unused function warning Date: Mon, 7 Aug 2017 12:41:53 +0200 Message-Id: <20170807104230.3564976-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:g2SCUroU3tOvc6Ruh/Z3myUzMs8Yq+3TRC8DHydN/+LOg/0uzyG BSaP4zRvzXC6JoF3Xdv4TuQgHPMyTINk5EVgdhMK5jjA96Qa9WogYfNKvfKuZMXxupO1K8k sB3yQaNWOfJLR5vNXH2arO4GPkoO/LBN5PTSXK/ZIFqBj6nKjjon8LHWnl334hz2E0o+Owy BhCzzumKPbda/IFsnVFYg== X-UI-Out-Filterresults: notjunk:1; V01:K0:J0kGxPzlPhU=:kgckmUE9fbuli5NpBEL6P8 +j18y6PbkxaPfNgGsXECQzdj/yD+U2u8ZWvclFVMPerr9ugccSD0IoBsGmtxbHSQ3Z6L0gJ3K KvJtCiW9YlTPMuDxbX/AkhZstdrXapoQOGaCV+81mkBQGwSW0gmtFxAjR9/Um5CjDYCsKKA6j NFgJ5ka1VABtGQLnxneaLUuMQ+GCZj3iljxwNaLRu5zcETsbN3g/4dTmUEr0XQfGY4MWoBkFC XDXQxTKJNVmm+wjZ8lkS8QcLxPVvsg5xyO4x2dflw2e7eHXXiaRiKVXbfHTdUIUKlx0WF7Ixg 3lseJsFuGS1yYPDulGK9XD4c0gOVuh9U7VEwqzT5oe8vBSQ5S9r98t6j95xo6ukzh2xz3jGyR y2PTRjxOzfvQUGKo/fPIB2wnuav83hAMUf1n+C1p4DGsUUKEmtKBCE7j0k6bS3YBMrmpoTBga HOLtrQTJu9aITvUqrPBZPeZEbEjMdMZDxHXC+T86DenypOw6EIJi3x3FV6EbVuvPkwtsBBjZC xoQ8WBjUfVhFDrU9ghyoMU3kO+HJOKHwDX0pm1WvUef0mWx9t/Jq/Qq/Asx8dnie5wYnDocC7 RLutDYjPQgkIvJCq5EZb5fabbhyc8xofu8lSpx4eiSVLWRyf0FCkNCv+aVFY02dQCGzyRtRlU ij4UQC1Ahr+r842sLmNTIKd6GY+LvIDt6hmX+RXbiEvYyKmUPHitgnKllEyoJJzXk9RjkWbE+ kVEfus67ydl0MN/yadyCPTBqmLRC2aCSvdr4ng== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Without CONFIG_PCI_IOV, we get a harmless warning about an unused function: drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c:2273:13: error: 'hclge_disable_sriov' defined but not used [-Werror=unused-function] The #ifdefs in this driver are obviously wrong, so this just removes them and uses an IS_ENABLED() check that does the same thing correctly in a more readable way. Fixes: 46a3df9f9718 ("net: hns3: Add HNS3 Acceleration Engine & Compatibility Layer Support") Signed-off-by: Arnd Bergmann --- .../ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 27 ++++++++++------------ 1 file changed, 12 insertions(+), 15 deletions(-) -- 2.9.0 diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c index 3611991689bc..7440e85b607c 100644 --- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c +++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c @@ -2272,19 +2272,17 @@ static void hclge_service_task(struct work_struct *work) static void hclge_disable_sriov(struct hclge_dev *hdev) { -#ifdef CONFIG_PCI_IOV - /* If our VFs are assigned we cannot shut down SR-IOV - * without causing issues, so just leave the hardware - * available but disabled - */ - if (pci_vfs_assigned(hdev->pdev)) { - dev_warn(&hdev->pdev->dev, - "disabling driver while VFs are assigned\n"); - return; - } + /* If our VFs are assigned we cannot shut down SR-IOV + * without causing issues, so just leave the hardware + * available but disabled + */ + if (pci_vfs_assigned(hdev->pdev)) { + dev_warn(&hdev->pdev->dev, + "disabling driver while VFs are assigned\n"); + return; + } - pci_disable_sriov(hdev->pdev); -#endif + pci_disable_sriov(hdev->pdev); } struct hclge_vport *hclge_get_vport(struct hnae3_handle *handle) @@ -4182,9 +4180,8 @@ static void hclge_uninit_ae_dev(struct hnae3_ae_dev *ae_dev) set_bit(HCLGE_STATE_DOWN, &hdev->state); -#ifdef CONFIG_PCI_IOV - hclge_disable_sriov(hdev); -#endif + if (IS_ENABLED(CONFIG_PCI_IOV)) + hclge_disable_sriov(hdev); if (hdev->service_timer.data) del_timer_sync(&hdev->service_timer);