net: phy: adin1100: Simplify register value passing
authorAlexander Dahl <ada@thorsis.com>
Wed, 19 Nov 2025 12:47:37 +0000 (13:47 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 21 Nov 2025 02:03:59 +0000 (18:03 -0800)
The additional use case for that variable is gone,
the expression is simple enough to pass it inline now.

Signed-off-by: Alexander Dahl <ada@thorsis.com>
Reviewed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Acked-by: Nuno Sá <nuno.sa@analog.com>
Link: https://patch.msgid.link/20251119124737.280939-3-ada@thorsis.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/adin1100.c

index 10b796c..8f9753d 100644 (file)
@@ -192,11 +192,10 @@ static irqreturn_t adin_phy_handle_interrupt(struct phy_device *phydev)
 static int adin_set_powerdown_mode(struct phy_device *phydev, bool en)
 {
        int ret;
-       int val;
 
-       val = en ? ADIN_CRSM_SFT_PD_CNTRL_EN : 0;
        ret = phy_write_mmd(phydev, MDIO_MMD_VEND1,
-                           ADIN_CRSM_SFT_PD_CNTRL, val);
+                           ADIN_CRSM_SFT_PD_CNTRL,
+                           en ? ADIN_CRSM_SFT_PD_CNTRL_EN : 0);
        if (ret < 0)
                return ret;