mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-29 22:02:02 +00:00
net: dsa: mt7530: only indicate linkmodes that can be supported
Now that mt7530 is not using the basex helper, it becomes unnecessary to indicate support for both 1000baseX and 2500baseX when one of the 803.3z PHY interface modes is being selected. Ensure that the driver indicates only those linkmodes that can actually be supported by the PHY interface mode. Tested-by: Marek Behún <kabel@kernel.org> Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
parent
fd301137e6
commit
7c04c84891
2 changed files with 9 additions and 4 deletions
|
@ -2522,12 +2522,13 @@ static int mt7531_rgmii_setup(struct mt7530_priv *priv, u32 port,
|
||||||
}
|
}
|
||||||
|
|
||||||
static void mt7531_sgmii_validate(struct mt7530_priv *priv, int port,
|
static void mt7531_sgmii_validate(struct mt7530_priv *priv, int port,
|
||||||
|
phy_interface_t interface,
|
||||||
unsigned long *supported)
|
unsigned long *supported)
|
||||||
{
|
{
|
||||||
/* Port5 supports ethier RGMII or SGMII.
|
/* Port5 supports ethier RGMII or SGMII.
|
||||||
* Port6 supports SGMII only.
|
* Port6 supports SGMII only.
|
||||||
*/
|
*/
|
||||||
if (port == 6) {
|
if (port == 6 && interface == PHY_INTERFACE_MODE_2500BASEX) {
|
||||||
phylink_set(supported, 2500baseX_Full);
|
phylink_set(supported, 2500baseX_Full);
|
||||||
phylink_set(supported, 2500baseT_Full);
|
phylink_set(supported, 2500baseT_Full);
|
||||||
}
|
}
|
||||||
|
@ -2902,16 +2903,18 @@ static void mt753x_phylink_get_caps(struct dsa_switch *ds, int port,
|
||||||
|
|
||||||
static void
|
static void
|
||||||
mt7530_mac_port_validate(struct dsa_switch *ds, int port,
|
mt7530_mac_port_validate(struct dsa_switch *ds, int port,
|
||||||
|
phy_interface_t interface,
|
||||||
unsigned long *supported)
|
unsigned long *supported)
|
||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
static void mt7531_mac_port_validate(struct dsa_switch *ds, int port,
|
static void mt7531_mac_port_validate(struct dsa_switch *ds, int port,
|
||||||
|
phy_interface_t interface,
|
||||||
unsigned long *supported)
|
unsigned long *supported)
|
||||||
{
|
{
|
||||||
struct mt7530_priv *priv = ds->priv;
|
struct mt7530_priv *priv = ds->priv;
|
||||||
|
|
||||||
mt7531_sgmii_validate(priv, port, supported);
|
mt7531_sgmii_validate(priv, port, interface, supported);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void
|
static void
|
||||||
|
@ -2934,12 +2937,13 @@ mt753x_phylink_validate(struct dsa_switch *ds, int port,
|
||||||
}
|
}
|
||||||
|
|
||||||
/* This switch only supports 1G full-duplex. */
|
/* This switch only supports 1G full-duplex. */
|
||||||
if (state->interface != PHY_INTERFACE_MODE_MII) {
|
if (state->interface != PHY_INTERFACE_MODE_MII &&
|
||||||
|
state->interface != PHY_INTERFACE_MODE_2500BASEX) {
|
||||||
phylink_set(mask, 1000baseT_Full);
|
phylink_set(mask, 1000baseT_Full);
|
||||||
phylink_set(mask, 1000baseX_Full);
|
phylink_set(mask, 1000baseX_Full);
|
||||||
}
|
}
|
||||||
|
|
||||||
priv->info->mac_port_validate(ds, port, mask);
|
priv->info->mac_port_validate(ds, port, state->interface, mask);
|
||||||
|
|
||||||
phylink_set(mask, Pause);
|
phylink_set(mask, Pause);
|
||||||
phylink_set(mask, Asym_Pause);
|
phylink_set(mask, Asym_Pause);
|
||||||
|
|
|
@ -772,6 +772,7 @@ struct mt753x_info {
|
||||||
void (*mac_port_get_caps)(struct dsa_switch *ds, int port,
|
void (*mac_port_get_caps)(struct dsa_switch *ds, int port,
|
||||||
struct phylink_config *config);
|
struct phylink_config *config);
|
||||||
void (*mac_port_validate)(struct dsa_switch *ds, int port,
|
void (*mac_port_validate)(struct dsa_switch *ds, int port,
|
||||||
|
phy_interface_t interface,
|
||||||
unsigned long *supported);
|
unsigned long *supported);
|
||||||
int (*mac_port_get_state)(struct dsa_switch *ds, int port,
|
int (*mac_port_get_state)(struct dsa_switch *ds, int port,
|
||||||
struct phylink_link_state *state);
|
struct phylink_link_state *state);
|
||||||
|
|
Loading…
Reference in a new issue