net: stmmac: mediatek: remove of_get_phy_mode()
authorRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Wed, 12 Mar 2025 09:20:31 +0000 (09:20 +0000)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 18 Mar 2025 14:53:15 +0000 (15:53 +0100)
devm_stmmac_probe_config_dt() already gets the PHY mode from firmware,
which is stored in plat_dat->phy_interface. Therefore, we don't need to
get it in platform code.

Initialise priv_plat->phy_mode from plat->phy_interface
inmediatek_dwmac_common_data().

Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1tsIGN-005uzZ-NG@rmk-PC.armlinux.org.uk
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/stmicro/stmmac/dwmac-mediatek.c

index c9636832a570a211a53f9480b0a8aec56509199f..d178d5ddc7c7a4588a9f7321d43071e20fe896b9 100644 (file)
@@ -456,7 +456,6 @@ static int mediatek_dwmac_config_dt(struct mediatek_dwmac_plat_data *plat)
 {
        struct mac_delay_struct *mac_delay = &plat->mac_delay;
        u32 tx_delay_ps, rx_delay_ps;
-       int err;
 
        plat->peri_regmap = syscon_regmap_lookup_by_phandle(plat->np, "mediatek,pericfg");
        if (IS_ERR(plat->peri_regmap)) {
@@ -464,12 +463,6 @@ static int mediatek_dwmac_config_dt(struct mediatek_dwmac_plat_data *plat)
                return PTR_ERR(plat->peri_regmap);
        }
 
-       err = of_get_phy_mode(plat->np, &plat->phy_mode);
-       if (err) {
-               dev_err(plat->dev, "not find phy-mode\n");
-               return err;
-       }
-
        if (!of_property_read_u32(plat->np, "mediatek,tx-delay-ps", &tx_delay_ps)) {
                if (tx_delay_ps < plat->variant->tx_delay_max) {
                        mac_delay->tx_delay = tx_delay_ps;
@@ -587,6 +580,7 @@ static int mediatek_dwmac_common_data(struct platform_device *pdev,
 {
        int i;
 
+       priv_plat->phy_mode = plat->phy_interface;
        plat->mac_interface = priv_plat->phy_mode;
        if (priv_plat->mac_wol)
                plat->flags &= ~STMMAC_FLAG_USE_PHY_WOL;