realtek: fix use of uninitialized sds_mode
authorINAGAKI Hiroshi <musashino.open@gmail.com>
Wed, 30 Mar 2022 03:00:42 +0000 (12:00 +0900)
committerSander Vanheule <sander@svanheule.net>
Sat, 8 Oct 2022 09:05:02 +0000 (11:05 +0200)
The initial state of sds_mode in rtl9300_force_sds_mode() is null and it
will be configured in switch-case. So print message after it.

Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
[amend commit message]
Signed-off-by: Sander Vanheule <sander@svanheule.net>
target/linux/realtek/files-5.10/drivers/net/phy/rtl83xx-phy.c

index 4175b950435ebd86315589f20ac11e9df2b62246..259458b1beef1c71b26dd2853980bd55dc1a6a52 100644 (file)
@@ -1697,7 +1697,6 @@ void rtl9300_force_sds_mode(int sds, phy_interface_t phy_if)
        u32 v, cr_0, cr_1, cr_2;
        u32 m_bit, l_bit;
 
-       pr_info("%s --------------------- serdes %d forcing to %x ...\n", __func__, sds, sds_mode);
        pr_info("%s: SDS: %d, mode %d\n", __func__, sds, phy_if);
        switch (phy_if) {
        case PHY_INTERFACE_MODE_SGMII:
@@ -1740,7 +1739,7 @@ void rtl9300_force_sds_mode(int sds, phy_interface_t phy_if)
                return;
        }
 
-       pr_info("%s: SDS mode %x\n", __func__, sds_mode);
+       pr_info("%s --------------------- serdes %d forcing to %x ...\n", __func__, sds, sds_mode);
        // Power down SerDes
        rtl9300_sds_field_w(sds, 0x20, 0, 7, 6, 0x3);
        if (sds == 5) pr_info("%s after %x\n", __func__, rtl930x_read_sds_phy(sds, 0x20, 0));