From patchwork Thu Dec 3 22:37:21 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 336956 Delivered-To: patch@linaro.org Received: by 2002:a92:5e16:0:0:0:0:0 with SMTP id s22csp677895ilb; Thu, 3 Dec 2020 14:38:59 -0800 (PST) X-Google-Smtp-Source: ABdhPJx4kME1eExwrdpOPCsSp8bVHcIZEFf2/WXwmdhA0KGaYbWri/B9C0/L8qHymIP8Qi7odx/6 X-Received: by 2002:aa7:c248:: with SMTP id y8mr5009155edo.344.1607035139658; Thu, 03 Dec 2020 14:38:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607035139; cv=none; d=google.com; s=arc-20160816; b=f8h5EfGEF+xbYQ+rlftlQujMfghrDa6kOYEjJTaq4HSVHvtnMtbvqvbsPEWiTWeucm D+cRFyR7ZE+TLyzKg0ft43OKz8qJe16azOCTHnX4WTia2+QLM3z1oX8YM+bAtskznJUE GZmL5aKvkiPMEihLABNpUE4j7XnFZKPpcf452PpZUcG242L8pQEz/wo+3L8hLIjXul9x x5vHfxsTsOulYo2+Xigc4pK8OuJ6Q5/rcscirB1sXXUKZeH1jloh2wpOgv8RVyulVnh1 jh8iKoiQxYaiq0BPzJdQ/NaOojhh5MBJ4vKC3V9Fd+S1WdcqfY7lZCraCYfG/Srn43MR IS1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=e71YtnRxKqBfQFXzBbr0mV7lS2XHLt/Wz57npNjtddY=; b=yLyJm+SdY9/ssNIW3LnA+8E2PLH0xnutkUk4pHBaHzRDs7GZ1KCMgEbpW79Zspzlej 5tmubk2MQP1qSKQEIDKFZRoGCVyv+A5syaFm5W+z08ygiyHWNfQX5o/pTRy6hCZ8JmxU HeZLUhPgcuhE34jnws+4/HKBDtDxn0m1Z14HiEiBNSJgHMSkoWbl+Sdht40VAUAHGHKr Yz8WCJ9vZr3Oy1jOemJte4/6A455l7zO0wL4xLeMMBgRrvimQ9E79wwC3384h0GSFnWv CT8vLbXD3rvit2TjYrahSa4h8UflsigARHAjgoLpnEBQLbG9zJax9xyr1UtqwkKq+0yp q/VQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id bm11si234758ejb.442.2020.12.03.14.38.59; Thu, 03 Dec 2020 14:38:59 -0800 (PST) Received-SPF: pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731867AbgLCWid (ORCPT + 8 others); Thu, 3 Dec 2020 17:38:33 -0500 Received: from mail.kernel.org ([198.145.29.99]:57304 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728295AbgLCWid (ORCPT ); Thu, 3 Dec 2020 17:38:33 -0500 From: Arnd Bergmann Authentication-Results: mail.kernel.org; dkim=permerror (bad message/signature format) To: Claudiu Manoil , "David S. Miller" , Jakub Kicinski , Russell King , Ioana Ciornei Cc: Arnd Bergmann , Michael Walle , Po Liu , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] enetc: fix build warning Date: Thu, 3 Dec 2020 23:37:21 +0100 Message-Id: <20201203223747.1337389-1-arnd@kernel.org> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Arnd Bergmann When CONFIG_OF is disabled, there is a harmless warning about an unused variable: enetc_pf.c: In function 'enetc_phylink_create': enetc_pf.c:981:17: error: unused variable 'dev' [-Werror=unused-variable] Slightly rearrange the code to pass around the of_node as a function argument, which avoids the problem without hurting readability. Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") Signed-off-by: Arnd Bergmann --- .../net/ethernet/freescale/enetc/enetc_pf.c | 21 +++++++++---------- 1 file changed, 10 insertions(+), 11 deletions(-) -- 2.27.0 diff --git a/drivers/net/ethernet/freescale/enetc/enetc_pf.c b/drivers/net/ethernet/freescale/enetc/enetc_pf.c index ecdc2af8c292..e1269ab4f2e4 100644 --- a/drivers/net/ethernet/freescale/enetc/enetc_pf.c +++ b/drivers/net/ethernet/freescale/enetc/enetc_pf.c @@ -851,13 +851,12 @@ static bool enetc_port_has_pcs(struct enetc_pf *pf) pf->if_mode == PHY_INTERFACE_MODE_USXGMII); } -static int enetc_mdiobus_create(struct enetc_pf *pf) +static int enetc_mdiobus_create(struct enetc_pf *pf, struct device_node *node) { - struct device *dev = &pf->si->pdev->dev; struct device_node *mdio_np; int err; - mdio_np = of_get_child_by_name(dev->of_node, "mdio"); + mdio_np = of_get_child_by_name(node, "mdio"); if (mdio_np) { err = enetc_mdio_probe(pf, mdio_np); @@ -969,18 +968,17 @@ static const struct phylink_mac_ops enetc_mac_phylink_ops = { .mac_link_down = enetc_pl_mac_link_down, }; -static int enetc_phylink_create(struct enetc_ndev_priv *priv) +static int enetc_phylink_create(struct enetc_ndev_priv *priv, + struct device_node *node) { struct enetc_pf *pf = enetc_si_priv(priv->si); - struct device *dev = &pf->si->pdev->dev; struct phylink *phylink; int err; pf->phylink_config.dev = &priv->ndev->dev; pf->phylink_config.type = PHYLINK_NETDEV; - phylink = phylink_create(&pf->phylink_config, - of_fwnode_handle(dev->of_node), + phylink = phylink_create(&pf->phylink_config, of_fwnode_handle(node), pf->if_mode, &enetc_mac_phylink_ops); if (IS_ERR(phylink)) { err = PTR_ERR(phylink); @@ -1005,9 +1003,10 @@ static int enetc_pf_probe(struct pci_dev *pdev, struct net_device *ndev; struct enetc_si *si; struct enetc_pf *pf; + struct device_node *node = pdev->dev.of_node; int err; - if (pdev->dev.of_node && !of_device_is_available(pdev->dev.of_node)) { + if (node && !of_device_is_available(node)) { dev_info(&pdev->dev, "device is disabled, skipping\n"); return -ENODEV; } @@ -1058,12 +1057,12 @@ static int enetc_pf_probe(struct pci_dev *pdev, goto err_alloc_msix; } - if (!of_get_phy_mode(pdev->dev.of_node, &pf->if_mode)) { - err = enetc_mdiobus_create(pf); + if (!of_get_phy_mode(node, &pf->if_mode)) { + err = enetc_mdiobus_create(pf, node); if (err) goto err_mdiobus_create; - err = enetc_phylink_create(priv); + err = enetc_phylink_create(priv, node); if (err) goto err_phylink_create; }