]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
net: phy: mediatek: init val in .phy_led_polarity_set for AN7581
authorChristian Marangi <ansuelsmth@gmail.com>
Tue, 15 Apr 2025 10:53:05 +0000 (12:53 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 17 Apr 2025 01:26:42 +0000 (18:26 -0700)
Fix smatch warning for uninitialised val in .phy_led_polarity_set for
AN7581 driver.

Correctly init to 0 to set polarity high by default.

Reported-by: Simon Horman <horms@kernel.org>
Fixes: 6a325aed130b ("net: phy: mediatek: add Airoha PHY ID to SoC driver")
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Link: https://patch.msgid.link/20250415105313.3409-1-ansuelsmth@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/mediatek/mtk-ge-soc.c

index fd0e447ffce79dd95578e0f13a2d73997e8e9392..cd09684780a491d4eabb36195874b6784a0001bb 100644 (file)
@@ -1432,8 +1432,8 @@ static int an7581_phy_probe(struct phy_device *phydev)
 static int an7581_phy_led_polarity_set(struct phy_device *phydev, int index,
                                       unsigned long modes)
 {
+       u16 val = 0;
        u32 mode;
-       u16 val;
 
        if (index >= MTK_PHY_MAX_LEDS)
                return -EINVAL;
@@ -1444,7 +1444,6 @@ static int an7581_phy_led_polarity_set(struct phy_device *phydev, int index,
                        val = MTK_PHY_LED_ON_POLARITY;
                        break;
                case PHY_LED_ACTIVE_HIGH:
-                       val = 0;
                        break;
                default:
                        return -EINVAL;