net: mvmdio: slight optimisation of orion_mdio_write
authorLeigh Brown <leigh@solinno.co.uk>
Tue, 29 Oct 2013 09:33:33 +0000 (09:33 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Oct 2013 22:53:36 +0000 (18:53 -0400)
Make only a single call to mutex_unlock in orion_mdio_write.

Signed-off-by: Leigh Brown <leigh@solinno.co.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/mvmdio.c

index e3898b3c91adcfecb31715151023b0457a3e0fce..0cfa0c860bc3293f8161284e28033fe7751c8f82 100644 (file)
@@ -150,10 +150,8 @@ static int orion_mdio_write(struct mii_bus *bus, int mii_id,
        mutex_lock(&dev->lock);
 
        ret = orion_mdio_wait_ready(bus);
-       if (ret < 0) {
-               mutex_unlock(&dev->lock);
-               return ret;
-       }
+       if (ret < 0)
+               goto out;
 
        writel(((mii_id << MVMDIO_SMI_PHY_ADDR_SHIFT) |
                (regnum << MVMDIO_SMI_PHY_REG_SHIFT)  |
@@ -161,9 +159,9 @@ static int orion_mdio_write(struct mii_bus *bus, int mii_id,
                (value << MVMDIO_SMI_DATA_SHIFT)),
               dev->regs);
 
+out:
        mutex_unlock(&dev->lock);
-
-       return 0;
+       return ret;
 }
 
 static int orion_mdio_reset(struct mii_bus *bus)