net: ag71xx: move clk_eth out of struct
authorRosen Penev <rosenp@gmail.com>
Thu, 22 Aug 2024 19:27:52 +0000 (12:27 -0700)
committerJakub Kicinski <kuba@kernel.org>
Mon, 26 Aug 2024 16:59:19 +0000 (09:59 -0700)
It's only used in one place. It doesn't need to be in the struct.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
Link: https://patch.msgid.link/20240822192758.141201-1-rosenp@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/atheros/ag71xx.c

index ee5d429abdaa6ac24ef08a734730c3ea5ef04dcb..bf02bea03a7bef188933b722ab1c314e47c81787 100644 (file)
@@ -381,7 +381,6 @@ struct ag71xx {
 
        struct reset_control *mdio_reset;
        struct clk *clk_mdio;
-       struct clk *clk_eth;
 };
 
 static int ag71xx_desc_empty(struct ag71xx_desc *desc)
@@ -1799,6 +1798,7 @@ static int ag71xx_probe(struct platform_device *pdev)
        const struct ag71xx_dcfg *dcfg;
        struct net_device *ndev;
        struct resource *res;
+       struct clk *clk_eth;
        int tx_size, err, i;
        struct ag71xx *ag;
 
@@ -1829,10 +1829,10 @@ static int ag71xx_probe(struct platform_device *pdev)
                return -EINVAL;
        }
 
-       ag->clk_eth = devm_clk_get_enabled(&pdev->dev, "eth");
-       if (IS_ERR(ag->clk_eth)) {
+       clk_eth = devm_clk_get_enabled(&pdev->dev, "eth");
+       if (IS_ERR(clk_eth)) {
                netif_err(ag, probe, ndev, "Failed to get eth clk.\n");
-               return PTR_ERR(ag->clk_eth);
+               return PTR_ERR(clk_eth);
        }
 
        SET_NETDEV_DEV(ndev, &pdev->dev);