mirror of
https://github.com/lkl/linux.git
synced 2025-12-19 16:13:19 +09:00
net: pcs: xpcs: export xpcs_config_eee
There is no good reason why we need to go through:
stmmac_xpcs_config_eee
-> stmmac_do_callback
-> mdio_xpcs_ops->config_eee
-> xpcs_config_eee
when we can simply call xpcs_config_eee.
priv->hw->xpcs is of the type "const struct mdio_xpcs_ops *" and is used
as a placeholder/synonym for priv->plat->mdio_bus_data->has_xpcs. It is
done that way because the mdio_bus_data pointer might or might not be
populated in all stmmac instantiations.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
committed by
David S. Miller
parent
a1a753ed1d
commit
14b517cb62
@@ -622,8 +622,6 @@ struct stmmac_mmc_ops {
|
|||||||
stmmac_do_callback(__priv, xpcs, link_up, __args)
|
stmmac_do_callback(__priv, xpcs, link_up, __args)
|
||||||
#define stmmac_xpcs_probe(__priv, __args...) \
|
#define stmmac_xpcs_probe(__priv, __args...) \
|
||||||
stmmac_do_callback(__priv, xpcs, probe, __args)
|
stmmac_do_callback(__priv, xpcs, probe, __args)
|
||||||
#define stmmac_xpcs_config_eee(__priv, __args...) \
|
|
||||||
stmmac_do_callback(__priv, xpcs, config_eee, __args)
|
|
||||||
|
|
||||||
struct stmmac_regs_off {
|
struct stmmac_regs_off {
|
||||||
u32 ptp_off;
|
u32 ptp_off;
|
||||||
|
|||||||
@@ -720,11 +720,13 @@ static int stmmac_ethtool_op_set_eee(struct net_device *dev,
|
|||||||
netdev_warn(priv->dev,
|
netdev_warn(priv->dev,
|
||||||
"Setting EEE tx-lpi is not supported\n");
|
"Setting EEE tx-lpi is not supported\n");
|
||||||
|
|
||||||
ret = stmmac_xpcs_config_eee(priv, &priv->hw->xpcs_args,
|
if (priv->hw->xpcs) {
|
||||||
priv->plat->mult_fact_100ns,
|
ret = xpcs_config_eee(&priv->hw->xpcs_args,
|
||||||
edata->eee_enabled);
|
priv->plat->mult_fact_100ns,
|
||||||
if (ret)
|
edata->eee_enabled);
|
||||||
return ret;
|
if (ret)
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
if (!edata->eee_enabled)
|
if (!edata->eee_enabled)
|
||||||
stmmac_disable_eee_mode(priv);
|
stmmac_disable_eee_mode(priv);
|
||||||
|
|||||||
@@ -715,8 +715,8 @@ void xpcs_validate(struct mdio_xpcs_args *xpcs, unsigned long *supported,
|
|||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(xpcs_validate);
|
EXPORT_SYMBOL_GPL(xpcs_validate);
|
||||||
|
|
||||||
static int xpcs_config_eee(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
int xpcs_config_eee(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
||||||
int enable)
|
int enable)
|
||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
@@ -747,6 +747,7 @@ static int xpcs_config_eee(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
|||||||
ret |= DW_VR_MII_EEE_TRN_LPI;
|
ret |= DW_VR_MII_EEE_TRN_LPI;
|
||||||
return xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL1, ret);
|
return xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL1, ret);
|
||||||
}
|
}
|
||||||
|
EXPORT_SYMBOL_GPL(xpcs_config_eee);
|
||||||
|
|
||||||
static int xpcs_config_aneg_c37_sgmii(struct mdio_xpcs_args *xpcs)
|
static int xpcs_config_aneg_c37_sgmii(struct mdio_xpcs_args *xpcs)
|
||||||
{
|
{
|
||||||
@@ -1033,7 +1034,6 @@ static struct mdio_xpcs_ops xpcs_ops = {
|
|||||||
.get_state = xpcs_get_state,
|
.get_state = xpcs_get_state,
|
||||||
.link_up = xpcs_link_up,
|
.link_up = xpcs_link_up,
|
||||||
.probe = xpcs_probe,
|
.probe = xpcs_probe,
|
||||||
.config_eee = xpcs_config_eee,
|
|
||||||
};
|
};
|
||||||
|
|
||||||
struct mdio_xpcs_ops *mdio_xpcs_get_ops(void)
|
struct mdio_xpcs_ops *mdio_xpcs_get_ops(void)
|
||||||
|
|||||||
@@ -30,13 +30,13 @@ struct mdio_xpcs_ops {
|
|||||||
int (*link_up)(struct mdio_xpcs_args *xpcs, int speed,
|
int (*link_up)(struct mdio_xpcs_args *xpcs, int speed,
|
||||||
phy_interface_t interface);
|
phy_interface_t interface);
|
||||||
int (*probe)(struct mdio_xpcs_args *xpcs, phy_interface_t interface);
|
int (*probe)(struct mdio_xpcs_args *xpcs, phy_interface_t interface);
|
||||||
int (*config_eee)(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
|
||||||
int enable);
|
|
||||||
};
|
};
|
||||||
|
|
||||||
int xpcs_get_an_mode(struct mdio_xpcs_args *xpcs, phy_interface_t interface);
|
int xpcs_get_an_mode(struct mdio_xpcs_args *xpcs, phy_interface_t interface);
|
||||||
struct mdio_xpcs_ops *mdio_xpcs_get_ops(void);
|
struct mdio_xpcs_ops *mdio_xpcs_get_ops(void);
|
||||||
void xpcs_validate(struct mdio_xpcs_args *xpcs, unsigned long *supported,
|
void xpcs_validate(struct mdio_xpcs_args *xpcs, unsigned long *supported,
|
||||||
struct phylink_link_state *state);
|
struct phylink_link_state *state);
|
||||||
|
int xpcs_config_eee(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
||||||
|
int enable);
|
||||||
|
|
||||||
#endif /* __LINUX_PCS_XPCS_H */
|
#endif /* __LINUX_PCS_XPCS_H */
|
||||||
|
|||||||
Reference in New Issue
Block a user