mirror of
https://github.com/torvalds/linux.git
synced 2026-06-05 04:56:13 +02:00
net: mvpp2: remove interface checks in mvpp2_phylink_validate()
As phylink checks the interface mode against the supported_interfaces bitmap, we no longer need to validate the interface mode in the validation function. Remove this to simplify it. Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
8498e17ed4
commit
6c0c4b7ac0
|
|
@ -6261,32 +6261,13 @@ static void mvpp2_phylink_validate(struct phylink_config *config,
|
|||
struct mvpp2_port *port = mvpp2_phylink_to_port(config);
|
||||
__ETHTOOL_DECLARE_LINK_MODE_MASK(mask) = { 0, };
|
||||
|
||||
/* Invalid combinations */
|
||||
switch (state->interface) {
|
||||
case PHY_INTERFACE_MODE_10GBASER:
|
||||
case PHY_INTERFACE_MODE_XAUI:
|
||||
if (!mvpp2_port_supports_xlg(port))
|
||||
goto empty_set;
|
||||
break;
|
||||
case PHY_INTERFACE_MODE_RGMII:
|
||||
case PHY_INTERFACE_MODE_RGMII_ID:
|
||||
case PHY_INTERFACE_MODE_RGMII_RXID:
|
||||
case PHY_INTERFACE_MODE_RGMII_TXID:
|
||||
if (!mvpp2_port_supports_rgmii(port))
|
||||
goto empty_set;
|
||||
break;
|
||||
case PHY_INTERFACE_MODE_1000BASEX:
|
||||
case PHY_INTERFACE_MODE_2500BASEX:
|
||||
/* When in 802.3z mode, we must have AN enabled:
|
||||
* Bit 2 Field InBandAnEn In-band Auto-Negotiation enable. ...
|
||||
* When <PortType> = 1 (1000BASE-X) this field must be set to 1.
|
||||
*/
|
||||
if (!phylink_test(state->advertising, Autoneg))
|
||||
goto empty_set;
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
/* When in 802.3z mode, we must have AN enabled:
|
||||
* Bit 2 Field InBandAnEn In-band Auto-Negotiation enable. ...
|
||||
* When <PortType> = 1 (1000BASE-X) this field must be set to 1.
|
||||
*/
|
||||
if (phy_interface_mode_is_8023z(state->interface) &&
|
||||
!phylink_test(state->advertising, Autoneg))
|
||||
goto empty_set;
|
||||
|
||||
phylink_set(mask, Autoneg);
|
||||
phylink_set_port_modes(mask);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user