]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
net: usb: lan78xx: rename phy_mutex to mdiobus_mutex
authorOleksij Rempel <o.rempel@pengutronix.de>
Mon, 16 Dec 2024 12:09:39 +0000 (13:09 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 18 Dec 2024 03:51:55 +0000 (19:51 -0800)
Rename `phy_mutex` to `mdiobus_mutex` for clarity, as the mutex protects
MDIO bus access rather than PHY-specific operations. Update all
references to ensure consistency.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Link: https://patch.msgid.link/20241216120941.1690908-5-o.rempel@pengutronix.de
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/usb/lan78xx.c

index 30301af29ab2b1bd9d374f80b2b8ed7b735d3186..78c75599b8f17730266ee2a2f3f9b4d078a9f59b 100644 (file)
@@ -439,7 +439,7 @@ struct lan78xx_net {
        struct usb_anchor       deferred;
 
        struct mutex            dev_mutex; /* serialise open/stop wrt suspend/resume */
-       struct mutex            phy_mutex; /* for phy access */
+       struct mutex            mdiobus_mutex; /* for MDIO bus access */
        unsigned int            pipe_in, pipe_out, pipe_intr;
 
        unsigned int            bulk_in_delay;
@@ -952,7 +952,7 @@ static int lan78xx_flush_rx_fifo(struct lan78xx_net *dev)
        return lan78xx_flush_fifo(dev, FCT_RX_CTL, FCT_RX_CTL_RST_);
 }
 
-/* Loop until the read is completed with timeout called with phy_mutex held */
+/* Loop until the read is completed with timeout called with mdiobus_mutex held */
 static int lan78xx_mdiobus_wait_not_busy(struct lan78xx_net *dev)
 {
        unsigned long start_time = jiffies;
@@ -1596,7 +1596,7 @@ static int lan78xx_mac_reset(struct lan78xx_net *dev)
        u32 val;
        int ret;
 
-       mutex_lock(&dev->phy_mutex);
+       mutex_lock(&dev->mdiobus_mutex);
 
        /* Resetting the device while there is activity on the MDIO
         * bus can result in the MAC interface locking up and not
@@ -1631,7 +1631,7 @@ static int lan78xx_mac_reset(struct lan78xx_net *dev)
 
        ret = -ETIMEDOUT;
 exit_unlock:
-       mutex_unlock(&dev->phy_mutex);
+       mutex_unlock(&dev->mdiobus_mutex);
 
        return ret;
 }
@@ -2249,7 +2249,7 @@ static int lan78xx_mdiobus_read(struct mii_bus *bus, int phy_id, int idx)
        if (ret < 0)
                return ret;
 
-       mutex_lock(&dev->phy_mutex);
+       mutex_lock(&dev->mdiobus_mutex);
 
        /* confirm MII not busy */
        ret = lan78xx_mdiobus_wait_not_busy(dev);
@@ -2273,7 +2273,7 @@ static int lan78xx_mdiobus_read(struct mii_bus *bus, int phy_id, int idx)
        ret = (int)(val & 0xFFFF);
 
 done:
-       mutex_unlock(&dev->phy_mutex);
+       mutex_unlock(&dev->mdiobus_mutex);
        usb_autopm_put_interface(dev->intf);
 
        return ret;
@@ -2290,7 +2290,7 @@ static int lan78xx_mdiobus_write(struct mii_bus *bus, int phy_id, int idx,
        if (ret < 0)
                return ret;
 
-       mutex_lock(&dev->phy_mutex);
+       mutex_lock(&dev->mdiobus_mutex);
 
        /* confirm MII not busy */
        ret = lan78xx_mdiobus_wait_not_busy(dev);
@@ -2313,7 +2313,7 @@ static int lan78xx_mdiobus_write(struct mii_bus *bus, int phy_id, int idx,
                goto done;
 
 done:
-       mutex_unlock(&dev->phy_mutex);
+       mutex_unlock(&dev->mdiobus_mutex);
        usb_autopm_put_interface(dev->intf);
        return ret;
 }
@@ -4476,7 +4476,7 @@ static int lan78xx_probe(struct usb_interface *intf,
        skb_queue_head_init(&dev->rxq_done);
        skb_queue_head_init(&dev->txq_pend);
        skb_queue_head_init(&dev->rxq_overflow);
-       mutex_init(&dev->phy_mutex);
+       mutex_init(&dev->mdiobus_mutex);
        mutex_init(&dev->dev_mutex);
 
        ret = lan78xx_urb_config_init(dev);