| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <82a754f7-cc38-06f0-348a-972612ce9085@hauke-m.de>
Date: Sun, 28 Nov 2021 19:49:25 +0100
From: Hauke Mehrtens <hauke@...ke-m.de>
To: "Russell King (Oracle)" <rmk+kernel@...linux.org.uk>,
Andrew Lunn <andrew@...n.ch>,
Florian Fainelli <f.fainelli@...il.com>,
Vivien Didelot <vivien.didelot@...il.com>,
Vladimir Oltean <olteanv@...il.com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
Claudiu Manoil <claudiu.manoil@....com>,
George McCollister <george.mccollister@...il.com>,
Kurt Kanzenbach <kurt@...utronix.de>,
Vladimir Oltean <vladimir.oltean@....com>,
Woojung Huh <woojung.huh@...rochip.com>
Cc: "David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>, netdev@...r.kernel.org,
UNGLinuxDriver@...rochip.com
Subject: Re: [PATCH RFC net-next 08/12] net: dsa: lantiq: convert to
phylink_generic_validate()
On 11/24/21 6:53 PM, Russell King (Oracle) wrote:
> Populate the supported interfaces and MAC capabilities for the Lantiq
> DSA switches and remove the old validate implementation to allow DSA to
> use phylink_generic_validate() for this switch driver.
>
> The exclusion of Gigabit linkmodes for MII, Reverse MII and Reduced MII
> links is handled within phylink_generic_validate() in phylink, so there
> is no need to make them conditional on the interface mode in the driver.
>
> Signed-off-by: Russell King (Oracle) <rmk+kernel@...linux.org.uk>
Reviewed-by: Hauke Mehrtens <hauke@...ke-m.de>
> ---
> drivers/net/dsa/lantiq_gswip.c | 120 +++++++++++----------------------
> 1 file changed, 38 insertions(+), 82 deletions(-)
>
> diff --git a/drivers/net/dsa/lantiq_gswip.c b/drivers/net/dsa/lantiq_gswip.c
> index 7056d98d8177..583af774e1bd 100644
> --- a/drivers/net/dsa/lantiq_gswip.c
> +++ b/drivers/net/dsa/lantiq_gswip.c
> @@ -1438,114 +1438,70 @@ static int gswip_port_fdb_dump(struct dsa_switch *ds, int port,
> return 0;
> }
>
> -static void gswip_phylink_set_capab(unsigned long *supported,
> - struct phylink_link_state *state)
> -{
> - __ETHTOOL_DECLARE_LINK_MODE_MASK(mask) = { 0, };
> -
> - /* Allow all the expected bits */
> - phylink_set(mask, Autoneg);
> - phylink_set_port_modes(mask);
> - phylink_set(mask, Pause);
> - phylink_set(mask, Asym_Pause);
> -
> - /* With the exclusion of MII, Reverse MII and Reduced MII, we
> - * support Gigabit, including Half duplex
> - */
> - if (state->interface != PHY_INTERFACE_MODE_MII &&
> - state->interface != PHY_INTERFACE_MODE_REVMII &&
> - state->interface != PHY_INTERFACE_MODE_RMII) {
> - phylink_set(mask, 1000baseT_Full);
> - phylink_set(mask, 1000baseT_Half);
> - }
> -
> - phylink_set(mask, 10baseT_Half);
> - phylink_set(mask, 10baseT_Full);
> - phylink_set(mask, 100baseT_Half);
> - phylink_set(mask, 100baseT_Full);
> -
> - linkmode_and(supported, supported, mask);
> - linkmode_and(state->advertising, state->advertising, mask);
> -}
> -
> -static void gswip_xrx200_phylink_validate(struct dsa_switch *ds, int port,
> - unsigned long *supported,
> - struct phylink_link_state *state)
> +static void gswip_xrx200_phylink_get_caps(struct dsa_switch *ds, int port,
> + struct phylink_config *config)
> {
> switch (port) {
> case 0:
> case 1:
> - if (!phy_interface_mode_is_rgmii(state->interface) &&
> - state->interface != PHY_INTERFACE_MODE_MII &&
> - state->interface != PHY_INTERFACE_MODE_REVMII &&
> - state->interface != PHY_INTERFACE_MODE_RMII)
> - goto unsupported;
> + phy_interface_set_rgmii(config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_MII,
> + config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_REVMII,
> + config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_RMII,
> + config->supported_interfaces);
> break;
> +
> case 2:
> case 3:
> case 4:
> - if (state->interface != PHY_INTERFACE_MODE_INTERNAL)
> - goto unsupported;
> + __set_bit(PHY_INTERFACE_MODE_INTERNAL,
> + config->supported_interfaces);
> break;
> +
> case 5:
> - if (!phy_interface_mode_is_rgmii(state->interface) &&
> - state->interface != PHY_INTERFACE_MODE_INTERNAL)
> - goto unsupported;
> + phy_interface_set_rgmii(config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_INTERNAL,
> + config->supported_interfaces);
> break;
> - default:
> - linkmode_zero(supported);
> - dev_err(ds->dev, "Unsupported port: %i\n", port);
> - return;
> }
>
> - gswip_phylink_set_capab(supported, state);
> -
> - return;
> -
> -unsupported:
> - linkmode_zero(supported);
> - dev_err(ds->dev, "Unsupported interface '%s' for port %d\n",
> - phy_modes(state->interface), port);
> + config->mac_capabilities = MAC_ASYM_PAUSE | MAC_SYM_PAUSE |
> + MAC_10 | MAC_100 | MAC_1000;
> }
>
> -static void gswip_xrx300_phylink_validate(struct dsa_switch *ds, int port,
> - unsigned long *supported,
> - struct phylink_link_state *state)
> +static void gswip_xrx300_phylink_get_caps(struct dsa_switch *ds, int port,
> + struct phylink_config *config)
> {
> switch (port) {
> case 0:
> - if (!phy_interface_mode_is_rgmii(state->interface) &&
> - state->interface != PHY_INTERFACE_MODE_GMII &&
> - state->interface != PHY_INTERFACE_MODE_RMII)
> - goto unsupported;
> + phy_interface_set_rgmii(config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_GMII,
> + config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_RMII,
> + config->supported_interfaces);
> break;
> +
> case 1:
> case 2:
> case 3:
> case 4:
> - if (state->interface != PHY_INTERFACE_MODE_INTERNAL)
> - goto unsupported;
> + __set_bit(PHY_INTERFACE_MODE_INTERNAL,
> + config->supported_interfaces);
> break;
> +
> case 5:
> - if (!phy_interface_mode_is_rgmii(state->interface) &&
> - state->interface != PHY_INTERFACE_MODE_INTERNAL &&
> - state->interface != PHY_INTERFACE_MODE_RMII)
> - goto unsupported;
> + phy_interface_set_rgmii(config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_INTERNAL,
> + config->supported_interfaces);
> + __set_bit(PHY_INTERFACE_MODE_RMII,
> + config->supported_interfaces);
> break;
> - default:
> - linkmode_zero(supported);
> - dev_err(ds->dev, "Unsupported port: %i\n", port);
> - return;
> }
>
> - gswip_phylink_set_capab(supported, state);
> -
> - return;
> -
> -unsupported:
> - linkmode_zero(supported);
> - dev_err(ds->dev, "Unsupported interface '%s' for port %d\n",
> - phy_modes(state->interface), port);
> + config->mac_capabilities = MAC_ASYM_PAUSE | MAC_SYM_PAUSE |
> + MAC_10 | MAC_100 | MAC_1000;
> }
>
> static void gswip_port_set_link(struct gswip_priv *priv, int port, bool link)
> @@ -1827,7 +1783,7 @@ static const struct dsa_switch_ops gswip_xrx200_switch_ops = {
> .port_fdb_add = gswip_port_fdb_add,
> .port_fdb_del = gswip_port_fdb_del,
> .port_fdb_dump = gswip_port_fdb_dump,
> - .phylink_validate = gswip_xrx200_phylink_validate,
> + .phylink_get_caps = gswip_xrx200_phylink_get_caps,
> .phylink_mac_config = gswip_phylink_mac_config,
> .phylink_mac_link_down = gswip_phylink_mac_link_down,
> .phylink_mac_link_up = gswip_phylink_mac_link_up,
> @@ -1851,7 +1807,7 @@ static const struct dsa_switch_ops gswip_xrx300_switch_ops = {
> .port_fdb_add = gswip_port_fdb_add,
> .port_fdb_del = gswip_port_fdb_del,
> .port_fdb_dump = gswip_port_fdb_dump,
> - .phylink_validate = gswip_xrx300_phylink_validate,
> + .phylink_get_caps = gswip_xrx300_phylink_get_caps,
> .phylink_mac_config = gswip_phylink_mac_config,
> .phylink_mac_link_down = gswip_phylink_mac_link_down,
> .phylink_mac_link_up = gswip_phylink_mac_link_up,
>
Powered by blists - more mailing lists