2020-03-09 16:36:26 +08:00
|
|
|
// SPDX-License-Identifier: GPL-2.0
|
|
|
|
/*
|
|
|
|
* Copyright (c) 2020 Synopsys, Inc. and/or its affiliates.
|
|
|
|
* Synopsys DesignWare XPCS helpers
|
|
|
|
*
|
|
|
|
* Author: Jose Abreu <Jose.Abreu@synopsys.com>
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include <linux/delay.h>
|
2020-08-27 10:00:28 +08:00
|
|
|
#include <linux/pcs/pcs-xpcs.h>
|
2020-03-09 16:36:26 +08:00
|
|
|
#include <linux/mdio.h>
|
|
|
|
#include <linux/phylink.h>
|
|
|
|
#include <linux/workqueue.h>
|
|
|
|
|
2021-06-03 00:20:12 +08:00
|
|
|
#define SYNOPSYS_XPCS_ID 0x7996ced0
|
2020-03-09 16:36:26 +08:00
|
|
|
#define SYNOPSYS_XPCS_MASK 0xffffffff
|
|
|
|
|
|
|
|
/* Vendor regs access */
|
|
|
|
#define DW_VENDOR BIT(15)
|
|
|
|
|
|
|
|
/* VR_XS_PCS */
|
|
|
|
#define DW_USXGMII_RST BIT(10)
|
|
|
|
#define DW_USXGMII_EN BIT(9)
|
|
|
|
#define DW_VR_XS_PCS_DIG_STS 0x0010
|
|
|
|
#define DW_RXFIFO_ERR GENMASK(6, 5)
|
|
|
|
|
|
|
|
/* SR_MII */
|
|
|
|
#define DW_USXGMII_FULL BIT(8)
|
|
|
|
#define DW_USXGMII_SS_MASK (BIT(13) | BIT(6) | BIT(5))
|
|
|
|
#define DW_USXGMII_10000 (BIT(13) | BIT(6))
|
|
|
|
#define DW_USXGMII_5000 (BIT(13) | BIT(5))
|
|
|
|
#define DW_USXGMII_2500 (BIT(5))
|
|
|
|
#define DW_USXGMII_1000 (BIT(6))
|
|
|
|
#define DW_USXGMII_100 (BIT(13))
|
|
|
|
#define DW_USXGMII_10 (0)
|
|
|
|
|
|
|
|
/* SR_AN */
|
|
|
|
#define DW_SR_AN_ADV1 0x10
|
|
|
|
#define DW_SR_AN_ADV2 0x11
|
|
|
|
#define DW_SR_AN_ADV3 0x12
|
|
|
|
#define DW_SR_AN_LP_ABL1 0x13
|
|
|
|
#define DW_SR_AN_LP_ABL2 0x14
|
|
|
|
#define DW_SR_AN_LP_ABL3 0x15
|
|
|
|
|
|
|
|
/* Clause 73 Defines */
|
|
|
|
/* AN_LP_ABL1 */
|
|
|
|
#define DW_C73_PAUSE BIT(10)
|
|
|
|
#define DW_C73_ASYM_PAUSE BIT(11)
|
|
|
|
#define DW_C73_AN_ADV_SF 0x1
|
|
|
|
/* AN_LP_ABL2 */
|
|
|
|
#define DW_C73_1000KX BIT(5)
|
|
|
|
#define DW_C73_10000KX4 BIT(6)
|
|
|
|
#define DW_C73_10000KR BIT(7)
|
|
|
|
/* AN_LP_ABL3 */
|
|
|
|
#define DW_C73_2500KX BIT(0)
|
|
|
|
#define DW_C73_5000KR BIT(1)
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
/* Clause 37 Defines */
|
|
|
|
/* VR MII MMD registers offsets */
|
|
|
|
#define DW_VR_MII_DIG_CTRL1 0x8000
|
|
|
|
#define DW_VR_MII_AN_CTRL 0x8001
|
|
|
|
#define DW_VR_MII_AN_INTR_STS 0x8002
|
2021-05-17 17:43:31 +08:00
|
|
|
/* EEE Mode Control Register */
|
|
|
|
#define DW_VR_MII_EEE_MCTRL0 0x8006
|
|
|
|
#define DW_VR_MII_EEE_MCTRL1 0x800b
|
2021-03-15 13:27:07 +08:00
|
|
|
|
|
|
|
/* VR_MII_DIG_CTRL1 */
|
|
|
|
#define DW_VR_MII_DIG_CTRL1_MAC_AUTO_SW BIT(9)
|
|
|
|
|
|
|
|
/* VR_MII_AN_CTRL */
|
|
|
|
#define DW_VR_MII_AN_CTRL_TX_CONFIG_SHIFT 3
|
|
|
|
#define DW_VR_MII_TX_CONFIG_MASK BIT(3)
|
|
|
|
#define DW_VR_MII_TX_CONFIG_PHY_SIDE_SGMII 0x1
|
|
|
|
#define DW_VR_MII_TX_CONFIG_MAC_SIDE_SGMII 0x0
|
|
|
|
#define DW_VR_MII_AN_CTRL_PCS_MODE_SHIFT 1
|
|
|
|
#define DW_VR_MII_PCS_MODE_MASK GENMASK(2, 1)
|
|
|
|
#define DW_VR_MII_PCS_MODE_C37_1000BASEX 0x0
|
|
|
|
#define DW_VR_MII_PCS_MODE_C37_SGMII 0x2
|
|
|
|
|
|
|
|
/* VR_MII_AN_INTR_STS */
|
|
|
|
#define DW_VR_MII_AN_STS_C37_ANSGM_FD BIT(1)
|
|
|
|
#define DW_VR_MII_AN_STS_C37_ANSGM_SP_SHIFT 2
|
|
|
|
#define DW_VR_MII_AN_STS_C37_ANSGM_SP GENMASK(3, 2)
|
|
|
|
#define DW_VR_MII_C37_ANSGM_SP_10 0x0
|
|
|
|
#define DW_VR_MII_C37_ANSGM_SP_100 0x1
|
|
|
|
#define DW_VR_MII_C37_ANSGM_SP_1000 0x2
|
|
|
|
#define DW_VR_MII_C37_ANSGM_SP_LNKSTS BIT(4)
|
|
|
|
|
2021-05-17 17:43:31 +08:00
|
|
|
/* VR MII EEE Control 0 defines */
|
|
|
|
#define DW_VR_MII_EEE_LTX_EN BIT(0) /* LPI Tx Enable */
|
|
|
|
#define DW_VR_MII_EEE_LRX_EN BIT(1) /* LPI Rx Enable */
|
|
|
|
#define DW_VR_MII_EEE_TX_QUIET_EN BIT(2) /* Tx Quiet Enable */
|
|
|
|
#define DW_VR_MII_EEE_RX_QUIET_EN BIT(3) /* Rx Quiet Enable */
|
|
|
|
#define DW_VR_MII_EEE_TX_EN_CTRL BIT(4) /* Tx Control Enable */
|
|
|
|
#define DW_VR_MII_EEE_RX_EN_CTRL BIT(7) /* Rx Control Enable */
|
|
|
|
|
|
|
|
#define DW_VR_MII_EEE_MULT_FACT_100NS_SHIFT 8
|
|
|
|
#define DW_VR_MII_EEE_MULT_FACT_100NS GENMASK(11, 8)
|
|
|
|
|
|
|
|
/* VR MII EEE Control 1 defines */
|
|
|
|
#define DW_VR_MII_EEE_TRN_LPI BIT(0) /* Transparent Mode Enable */
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
#define phylink_pcs_to_xpcs(pl_pcs) \
|
|
|
|
container_of((pl_pcs), struct mdio_xpcs_args, pcs)
|
|
|
|
|
2020-03-09 16:36:26 +08:00
|
|
|
static const int xpcs_usxgmii_features[] = {
|
|
|
|
ETHTOOL_LINK_MODE_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_Asym_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_Autoneg_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_1000baseKX_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_10000baseKX4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_10000baseKR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_2500baseX_Full_BIT,
|
|
|
|
__ETHTOOL_LINK_MODE_MASK_NBITS,
|
|
|
|
};
|
|
|
|
|
|
|
|
static const int xpcs_10gkr_features[] = {
|
|
|
|
ETHTOOL_LINK_MODE_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_Asym_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_10000baseKR_Full_BIT,
|
|
|
|
__ETHTOOL_LINK_MODE_MASK_NBITS,
|
|
|
|
};
|
|
|
|
|
2020-03-17 17:18:50 +08:00
|
|
|
static const int xpcs_xlgmii_features[] = {
|
|
|
|
ETHTOOL_LINK_MODE_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_Asym_Pause_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_25000baseCR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_25000baseKR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_25000baseSR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_40000baseKR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_40000baseCR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_40000baseSR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_40000baseLR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseKR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseSR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseCR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseLR_ER_FR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_50000baseDR_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseKR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseSR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseCR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseLR2_ER2_FR2_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100000baseDR2_Full_BIT,
|
|
|
|
__ETHTOOL_LINK_MODE_MASK_NBITS,
|
|
|
|
};
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
static const int xpcs_sgmii_features[] = {
|
|
|
|
ETHTOOL_LINK_MODE_10baseT_Half_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_10baseT_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100baseT_Half_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_100baseT_Full_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_1000baseT_Half_BIT,
|
|
|
|
ETHTOOL_LINK_MODE_1000baseT_Full_BIT,
|
|
|
|
__ETHTOOL_LINK_MODE_MASK_NBITS,
|
|
|
|
};
|
|
|
|
|
2020-03-09 16:36:26 +08:00
|
|
|
static const phy_interface_t xpcs_usxgmii_interfaces[] = {
|
|
|
|
PHY_INTERFACE_MODE_USXGMII,
|
|
|
|
};
|
|
|
|
|
|
|
|
static const phy_interface_t xpcs_10gkr_interfaces[] = {
|
|
|
|
PHY_INTERFACE_MODE_10GKR,
|
|
|
|
};
|
|
|
|
|
2020-03-17 17:18:50 +08:00
|
|
|
static const phy_interface_t xpcs_xlgmii_interfaces[] = {
|
|
|
|
PHY_INTERFACE_MODE_XLGMII,
|
|
|
|
};
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
static const phy_interface_t xpcs_sgmii_interfaces[] = {
|
|
|
|
PHY_INTERFACE_MODE_SGMII,
|
|
|
|
};
|
|
|
|
|
2021-06-03 00:20:12 +08:00
|
|
|
enum {
|
|
|
|
DW_XPCS_USXGMII,
|
|
|
|
DW_XPCS_10GKR,
|
|
|
|
DW_XPCS_XLGMII,
|
|
|
|
DW_XPCS_SGMII,
|
|
|
|
DW_XPCS_INTERFACE_MAX,
|
|
|
|
};
|
|
|
|
|
|
|
|
struct xpcs_compat {
|
2020-03-09 16:36:26 +08:00
|
|
|
const int *supported;
|
|
|
|
const phy_interface_t *interface;
|
2021-06-03 00:20:12 +08:00
|
|
|
int num_interfaces;
|
2021-03-15 13:27:06 +08:00
|
|
|
int an_mode;
|
2021-06-03 00:20:12 +08:00
|
|
|
};
|
|
|
|
|
|
|
|
struct xpcs_id {
|
|
|
|
u32 id;
|
|
|
|
u32 mask;
|
|
|
|
const struct xpcs_compat *compat;
|
2020-03-09 16:36:26 +08:00
|
|
|
};
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
static const struct xpcs_compat *xpcs_find_compat(const struct xpcs_id *id,
|
|
|
|
phy_interface_t interface)
|
|
|
|
{
|
|
|
|
int i, j;
|
|
|
|
|
|
|
|
for (i = 0; i < DW_XPCS_INTERFACE_MAX; i++) {
|
|
|
|
const struct xpcs_compat *compat = &id->compat[i];
|
|
|
|
|
|
|
|
for (j = 0; j < compat->num_interfaces; j++)
|
|
|
|
if (compat->interface[j] == interface)
|
|
|
|
return compat;
|
|
|
|
}
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
int xpcs_get_an_mode(struct mdio_xpcs_args *xpcs, phy_interface_t interface)
|
|
|
|
{
|
|
|
|
const struct xpcs_compat *compat;
|
|
|
|
|
|
|
|
compat = xpcs_find_compat(xpcs->id, interface);
|
|
|
|
if (!compat)
|
|
|
|
return -ENODEV;
|
|
|
|
|
|
|
|
return compat->an_mode;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(xpcs_get_an_mode);
|
|
|
|
|
|
|
|
static bool __xpcs_linkmode_supported(const struct xpcs_compat *compat,
|
|
|
|
enum ethtool_link_mode_bit_indices linkmode)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; compat->supported[i] != __ETHTOOL_LINK_MODE_MASK_NBITS; i++)
|
|
|
|
if (compat->supported[i] == linkmode)
|
|
|
|
return true;
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
#define xpcs_linkmode_supported(compat, mode) \
|
|
|
|
__xpcs_linkmode_supported(compat, ETHTOOL_LINK_MODE_ ## mode ## _BIT)
|
|
|
|
|
2020-03-09 16:36:26 +08:00
|
|
|
static int xpcs_read(struct mdio_xpcs_args *xpcs, int dev, u32 reg)
|
|
|
|
{
|
2021-06-03 00:20:17 +08:00
|
|
|
u32 reg_addr = mdiobus_c45_addr(dev, reg);
|
2021-06-03 00:20:18 +08:00
|
|
|
struct mii_bus *bus = xpcs->mdiodev->bus;
|
|
|
|
int addr = xpcs->mdiodev->addr;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
2021-06-03 00:20:18 +08:00
|
|
|
return mdiobus_read(bus, addr, reg_addr);
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_write(struct mdio_xpcs_args *xpcs, int dev, u32 reg, u16 val)
|
|
|
|
{
|
2021-06-03 00:20:17 +08:00
|
|
|
u32 reg_addr = mdiobus_c45_addr(dev, reg);
|
2021-06-03 00:20:18 +08:00
|
|
|
struct mii_bus *bus = xpcs->mdiodev->bus;
|
|
|
|
int addr = xpcs->mdiodev->addr;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
2021-06-03 00:20:18 +08:00
|
|
|
return mdiobus_write(bus, addr, reg_addr, val);
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_read_vendor(struct mdio_xpcs_args *xpcs, int dev, u32 reg)
|
|
|
|
{
|
|
|
|
return xpcs_read(xpcs, dev, DW_VENDOR | reg);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_write_vendor(struct mdio_xpcs_args *xpcs, int dev, int reg,
|
|
|
|
u16 val)
|
|
|
|
{
|
|
|
|
return xpcs_write(xpcs, dev, DW_VENDOR | reg, val);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_read_vpcs(struct mdio_xpcs_args *xpcs, int reg)
|
|
|
|
{
|
|
|
|
return xpcs_read_vendor(xpcs, MDIO_MMD_PCS, reg);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_write_vpcs(struct mdio_xpcs_args *xpcs, int reg, u16 val)
|
|
|
|
{
|
|
|
|
return xpcs_write_vendor(xpcs, MDIO_MMD_PCS, reg, val);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_poll_reset(struct mdio_xpcs_args *xpcs, int dev)
|
|
|
|
{
|
|
|
|
/* Poll until the reset bit clears (50ms per retry == 0.6 sec) */
|
|
|
|
unsigned int retries = 12;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
do {
|
|
|
|
msleep(50);
|
|
|
|
ret = xpcs_read(xpcs, dev, MDIO_CTRL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
} while (ret & MDIO_CTRL1_RESET && --retries);
|
|
|
|
|
|
|
|
return (ret & MDIO_CTRL1_RESET) ? -ETIMEDOUT : 0;
|
|
|
|
}
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
static int xpcs_soft_reset(struct mdio_xpcs_args *xpcs,
|
|
|
|
const struct xpcs_compat *compat)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
2021-03-15 13:27:06 +08:00
|
|
|
int ret, dev;
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
switch (compat->an_mode) {
|
2021-03-15 13:27:06 +08:00
|
|
|
case DW_AN_C73:
|
|
|
|
dev = MDIO_MMD_PCS;
|
|
|
|
break;
|
2021-03-15 13:27:07 +08:00
|
|
|
case DW_AN_C37_SGMII:
|
|
|
|
dev = MDIO_MMD_VEND2;
|
|
|
|
break;
|
2021-03-15 13:27:06 +08:00
|
|
|
default:
|
|
|
|
return -1;
|
|
|
|
}
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
ret = xpcs_write(xpcs, dev, MDIO_CTRL1, MDIO_CTRL1_RESET);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
return xpcs_poll_reset(xpcs, dev);
|
|
|
|
}
|
|
|
|
|
|
|
|
#define xpcs_warn(__xpcs, __state, __args...) \
|
|
|
|
({ \
|
|
|
|
if ((__state)->link) \
|
2021-06-03 00:20:18 +08:00
|
|
|
dev_warn(&(__xpcs)->mdiodev->dev, ##__args); \
|
2020-03-09 16:36:26 +08:00
|
|
|
})
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static int xpcs_read_fault_c73(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MDIO_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & MDIO_STAT1_FAULT) {
|
|
|
|
xpcs_warn(xpcs, state, "Link fault condition detected!\n");
|
|
|
|
return -EFAULT;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MDIO_STAT2);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & MDIO_STAT2_RXFAULT)
|
|
|
|
xpcs_warn(xpcs, state, "Receiver fault detected!\n");
|
|
|
|
if (ret & MDIO_STAT2_TXFAULT)
|
|
|
|
xpcs_warn(xpcs, state, "Transmitter fault detected!\n");
|
|
|
|
|
|
|
|
ret = xpcs_read_vendor(xpcs, MDIO_MMD_PCS, DW_VR_XS_PCS_DIG_STS);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & DW_RXFIFO_ERR) {
|
|
|
|
xpcs_warn(xpcs, state, "FIFO fault condition detected!\n");
|
|
|
|
return -EFAULT;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MDIO_PCS_10GBRT_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (!(ret & MDIO_PCS_10GBRT_STAT1_BLKLK))
|
|
|
|
xpcs_warn(xpcs, state, "Link is not locked!\n");
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MDIO_PCS_10GBRT_STAT2);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
2020-03-20 17:53:34 +08:00
|
|
|
if (ret & MDIO_PCS_10GBRT_STAT2_ERR) {
|
2020-03-09 16:36:26 +08:00
|
|
|
xpcs_warn(xpcs, state, "Link has errors!\n");
|
2020-03-20 17:53:34 +08:00
|
|
|
return -EFAULT;
|
|
|
|
}
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static int xpcs_read_link_c73(struct mdio_xpcs_args *xpcs, bool an)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
bool link = true;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MDIO_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (!(ret & MDIO_STAT1_LSTATUS))
|
|
|
|
link = false;
|
|
|
|
|
|
|
|
if (an) {
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, MDIO_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (!(ret & MDIO_STAT1_LSTATUS))
|
|
|
|
link = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
return link;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int xpcs_get_max_usxgmii_speed(const unsigned long *supported)
|
|
|
|
{
|
|
|
|
int max = SPEED_UNKNOWN;
|
|
|
|
|
|
|
|
if (phylink_test(supported, 1000baseKX_Full))
|
|
|
|
max = SPEED_1000;
|
|
|
|
if (phylink_test(supported, 2500baseX_Full))
|
|
|
|
max = SPEED_2500;
|
|
|
|
if (phylink_test(supported, 10000baseKX4_Full))
|
|
|
|
max = SPEED_10000;
|
|
|
|
if (phylink_test(supported, 10000baseKR_Full))
|
|
|
|
max = SPEED_10000;
|
|
|
|
|
|
|
|
return max;
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static void xpcs_config_usxgmii(struct mdio_xpcs_args *xpcs, int speed)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret, speed_sel;
|
|
|
|
|
|
|
|
switch (speed) {
|
|
|
|
case SPEED_10:
|
|
|
|
speed_sel = DW_USXGMII_10;
|
|
|
|
break;
|
|
|
|
case SPEED_100:
|
|
|
|
speed_sel = DW_USXGMII_100;
|
|
|
|
break;
|
|
|
|
case SPEED_1000:
|
|
|
|
speed_sel = DW_USXGMII_1000;
|
|
|
|
break;
|
|
|
|
case SPEED_2500:
|
|
|
|
speed_sel = DW_USXGMII_2500;
|
|
|
|
break;
|
|
|
|
case SPEED_5000:
|
|
|
|
speed_sel = DW_USXGMII_5000;
|
|
|
|
break;
|
|
|
|
case SPEED_10000:
|
|
|
|
speed_sel = DW_USXGMII_10000;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
/* Nothing to do here */
|
2021-06-03 00:20:19 +08:00
|
|
|
return;
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
ret = xpcs_read_vpcs(xpcs, MDIO_CTRL1);
|
|
|
|
if (ret < 0)
|
2021-06-03 00:20:19 +08:00
|
|
|
goto out;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
ret = xpcs_write_vpcs(xpcs, MDIO_CTRL1, ret | DW_USXGMII_EN);
|
|
|
|
if (ret < 0)
|
2021-06-03 00:20:19 +08:00
|
|
|
goto out;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, MDIO_CTRL1);
|
|
|
|
if (ret < 0)
|
2021-06-03 00:20:19 +08:00
|
|
|
goto out;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
ret &= ~DW_USXGMII_SS_MASK;
|
|
|
|
ret |= speed_sel | DW_USXGMII_FULL;
|
|
|
|
|
|
|
|
ret = xpcs_write(xpcs, MDIO_MMD_VEND2, MDIO_CTRL1, ret);
|
|
|
|
if (ret < 0)
|
2021-06-03 00:20:19 +08:00
|
|
|
goto out;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
ret = xpcs_read_vpcs(xpcs, MDIO_CTRL1);
|
|
|
|
if (ret < 0)
|
2021-06-03 00:20:19 +08:00
|
|
|
goto out;
|
|
|
|
|
|
|
|
ret = xpcs_write_vpcs(xpcs, MDIO_CTRL1, ret | DW_USXGMII_RST);
|
|
|
|
if (ret < 0)
|
|
|
|
goto out;
|
|
|
|
|
|
|
|
return;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
out:
|
|
|
|
pr_err("%s: XPCS access returned %pe\n", __func__, ERR_PTR(ret));
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
static int _xpcs_config_aneg_c73(struct mdio_xpcs_args *xpcs,
|
|
|
|
const struct xpcs_compat *compat)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret, adv;
|
|
|
|
|
|
|
|
/* By default, in USXGMII mode XPCS operates at 10G baud and
|
|
|
|
* replicates data to achieve lower speeds. Hereby, in this
|
|
|
|
* default configuration we need to advertise all supported
|
|
|
|
* modes and not only the ones we want to use.
|
|
|
|
*/
|
|
|
|
|
|
|
|
/* SR_AN_ADV3 */
|
|
|
|
adv = 0;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, 2500baseX_Full))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_2500KX;
|
|
|
|
|
|
|
|
/* TODO: 5000baseKR */
|
|
|
|
|
|
|
|
ret = xpcs_write(xpcs, MDIO_MMD_AN, DW_SR_AN_ADV3, adv);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
/* SR_AN_ADV2 */
|
|
|
|
adv = 0;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, 1000baseKX_Full))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_1000KX;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, 10000baseKX4_Full))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_10000KX4;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, 10000baseKR_Full))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_10000KR;
|
|
|
|
|
|
|
|
ret = xpcs_write(xpcs, MDIO_MMD_AN, DW_SR_AN_ADV2, adv);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
/* SR_AN_ADV1 */
|
|
|
|
adv = DW_C73_AN_ADV_SF;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, Pause))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_PAUSE;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (xpcs_linkmode_supported(compat, Asym_Pause))
|
2020-03-09 16:36:26 +08:00
|
|
|
adv |= DW_C73_ASYM_PAUSE;
|
|
|
|
|
|
|
|
return xpcs_write(xpcs, MDIO_MMD_AN, DW_SR_AN_ADV1, adv);
|
|
|
|
}
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
static int xpcs_config_aneg_c73(struct mdio_xpcs_args *xpcs,
|
|
|
|
const struct xpcs_compat *compat)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
ret = _xpcs_config_aneg_c73(xpcs, compat);
|
2020-03-09 16:36:26 +08:00
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, MDIO_CTRL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret |= MDIO_AN_CTRL1_ENABLE | MDIO_AN_CTRL1_RESTART;
|
|
|
|
|
|
|
|
return xpcs_write(xpcs, MDIO_MMD_AN, MDIO_CTRL1, ret);
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static int xpcs_aneg_done_c73(struct mdio_xpcs_args *xpcs,
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
struct phylink_link_state *state,
|
|
|
|
const struct xpcs_compat *compat)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, MDIO_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & MDIO_AN_STAT1_COMPLETE) {
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, DW_SR_AN_LP_ABL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
/* Check if Aneg outcome is valid */
|
2020-03-20 17:53:37 +08:00
|
|
|
if (!(ret & DW_C73_AN_ADV_SF)) {
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
xpcs_config_aneg_c73(xpcs, compat);
|
2020-03-09 16:36:26 +08:00
|
|
|
return 0;
|
2020-03-20 17:53:37 +08:00
|
|
|
}
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static int xpcs_read_lpa_c73(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, MDIO_STAT1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (!(ret & MDIO_AN_STAT1_LPABLE)) {
|
|
|
|
phylink_clear(state->lp_advertising, Autoneg);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
phylink_set(state->lp_advertising, Autoneg);
|
|
|
|
|
|
|
|
/* Clause 73 outcome */
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, DW_SR_AN_LP_ABL3);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & DW_C73_2500KX)
|
|
|
|
phylink_set(state->lp_advertising, 2500baseX_Full);
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, DW_SR_AN_LP_ABL2);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & DW_C73_1000KX)
|
|
|
|
phylink_set(state->lp_advertising, 1000baseKX_Full);
|
|
|
|
if (ret & DW_C73_10000KX4)
|
|
|
|
phylink_set(state->lp_advertising, 10000baseKX4_Full);
|
|
|
|
if (ret & DW_C73_10000KR)
|
|
|
|
phylink_set(state->lp_advertising, 10000baseKR_Full);
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_AN, DW_SR_AN_LP_ABL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (ret & DW_C73_PAUSE)
|
|
|
|
phylink_set(state->lp_advertising, Pause);
|
|
|
|
if (ret & DW_C73_ASYM_PAUSE)
|
|
|
|
phylink_set(state->lp_advertising, Asym_Pause);
|
|
|
|
|
|
|
|
linkmode_and(state->lp_advertising, state->lp_advertising,
|
|
|
|
state->advertising);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static void xpcs_resolve_lpa_c73(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int max_speed = xpcs_get_max_usxgmii_speed(state->lp_advertising);
|
|
|
|
|
|
|
|
state->pause = MLO_PAUSE_TX | MLO_PAUSE_RX;
|
|
|
|
state->speed = max_speed;
|
|
|
|
state->duplex = DUPLEX_FULL;
|
|
|
|
}
|
|
|
|
|
2020-03-17 17:18:50 +08:00
|
|
|
static int xpcs_get_max_xlgmii_speed(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
|
|
|
{
|
|
|
|
unsigned long *adv = state->advertising;
|
|
|
|
int speed = SPEED_UNKNOWN;
|
|
|
|
int bit;
|
|
|
|
|
|
|
|
for_each_set_bit(bit, adv, __ETHTOOL_LINK_MODE_MASK_NBITS) {
|
|
|
|
int new_speed = SPEED_UNKNOWN;
|
|
|
|
|
|
|
|
switch (bit) {
|
|
|
|
case ETHTOOL_LINK_MODE_25000baseCR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_25000baseKR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_25000baseSR_Full_BIT:
|
|
|
|
new_speed = SPEED_25000;
|
|
|
|
break;
|
|
|
|
case ETHTOOL_LINK_MODE_40000baseKR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_40000baseCR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_40000baseSR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_40000baseLR4_Full_BIT:
|
|
|
|
new_speed = SPEED_40000;
|
|
|
|
break;
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseKR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseSR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseCR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseLR_ER_FR_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_50000baseDR_Full_BIT:
|
|
|
|
new_speed = SPEED_50000;
|
|
|
|
break;
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseKR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseSR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseCR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseLR2_ER2_FR2_Full_BIT:
|
|
|
|
case ETHTOOL_LINK_MODE_100000baseDR2_Full_BIT:
|
|
|
|
new_speed = SPEED_100000;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (new_speed > speed)
|
|
|
|
speed = new_speed;
|
|
|
|
}
|
|
|
|
|
|
|
|
return speed;
|
|
|
|
}
|
|
|
|
|
2020-03-09 16:36:26 +08:00
|
|
|
static void xpcs_resolve_pma(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
|
|
|
{
|
|
|
|
state->pause = MLO_PAUSE_TX | MLO_PAUSE_RX;
|
|
|
|
state->duplex = DUPLEX_FULL;
|
|
|
|
|
|
|
|
switch (state->interface) {
|
|
|
|
case PHY_INTERFACE_MODE_10GKR:
|
|
|
|
state->speed = SPEED_10000;
|
|
|
|
break;
|
2020-03-17 17:18:50 +08:00
|
|
|
case PHY_INTERFACE_MODE_XLGMII:
|
|
|
|
state->speed = xpcs_get_max_xlgmii_speed(xpcs, state);
|
|
|
|
break;
|
2020-03-09 16:36:26 +08:00
|
|
|
default:
|
|
|
|
state->speed = SPEED_UNKNOWN;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:14 +08:00
|
|
|
void xpcs_validate(struct mdio_xpcs_args *xpcs, unsigned long *supported,
|
|
|
|
struct phylink_link_state *state)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
__ETHTOOL_DECLARE_LINK_MODE_MASK(xpcs_supported);
|
|
|
|
const struct xpcs_compat *compat;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
/* phylink expects us to report all supported modes with
|
|
|
|
* PHY_INTERFACE_MODE_NA, just don't limit the supported and
|
|
|
|
* advertising masks and exit.
|
|
|
|
*/
|
|
|
|
if (state->interface == PHY_INTERFACE_MODE_NA)
|
2021-06-03 00:20:14 +08:00
|
|
|
return;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
|
|
|
|
bitmap_zero(xpcs_supported, __ETHTOOL_LINK_MODE_MASK_NBITS);
|
|
|
|
|
|
|
|
compat = xpcs_find_compat(xpcs->id, state->interface);
|
|
|
|
|
|
|
|
/* Populate the supported link modes for this
|
|
|
|
* PHY interface type
|
|
|
|
*/
|
|
|
|
if (compat)
|
|
|
|
for (i = 0; compat->supported[i] != __ETHTOOL_LINK_MODE_MASK_NBITS; i++)
|
|
|
|
set_bit(compat->supported[i], xpcs_supported);
|
|
|
|
|
|
|
|
linkmode_and(supported, supported, xpcs_supported);
|
|
|
|
linkmode_and(state->advertising, state->advertising, xpcs_supported);
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
2021-06-03 00:20:14 +08:00
|
|
|
EXPORT_SYMBOL_GPL(xpcs_validate);
|
2020-03-09 16:36:26 +08:00
|
|
|
|
2021-06-03 00:20:15 +08:00
|
|
|
int xpcs_config_eee(struct mdio_xpcs_args *xpcs, int mult_fact_100ns,
|
|
|
|
int enable)
|
2021-05-17 17:43:31 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (enable) {
|
|
|
|
/* Enable EEE */
|
|
|
|
ret = DW_VR_MII_EEE_LTX_EN | DW_VR_MII_EEE_LRX_EN |
|
|
|
|
DW_VR_MII_EEE_TX_QUIET_EN | DW_VR_MII_EEE_RX_QUIET_EN |
|
|
|
|
DW_VR_MII_EEE_TX_EN_CTRL | DW_VR_MII_EEE_RX_EN_CTRL |
|
|
|
|
mult_fact_100ns << DW_VR_MII_EEE_MULT_FACT_100NS_SHIFT;
|
|
|
|
} else {
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL0);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
ret &= ~(DW_VR_MII_EEE_LTX_EN | DW_VR_MII_EEE_LRX_EN |
|
|
|
|
DW_VR_MII_EEE_TX_QUIET_EN | DW_VR_MII_EEE_RX_QUIET_EN |
|
|
|
|
DW_VR_MII_EEE_TX_EN_CTRL | DW_VR_MII_EEE_RX_EN_CTRL |
|
|
|
|
DW_VR_MII_EEE_MULT_FACT_100NS);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL0, ret);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret |= DW_VR_MII_EEE_TRN_LPI;
|
|
|
|
return xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_EEE_MCTRL1, ret);
|
|
|
|
}
|
2021-06-03 00:20:15 +08:00
|
|
|
EXPORT_SYMBOL_GPL(xpcs_config_eee);
|
2021-05-17 17:43:31 +08:00
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
static int xpcs_config_aneg_c37_sgmii(struct mdio_xpcs_args *xpcs)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
/* For AN for C37 SGMII mode, the settings are :-
|
|
|
|
* 1) VR_MII_AN_CTRL Bit(2:1)[PCS_MODE] = 10b (SGMII AN)
|
|
|
|
* 2) VR_MII_AN_CTRL Bit(3) [TX_CONFIG] = 0b (MAC side SGMII)
|
|
|
|
* DW xPCS used with DW EQoS MAC is always MAC side SGMII.
|
|
|
|
* 3) VR_MII_DIG_CTRL1 Bit(9) [MAC_AUTO_SW] = 1b (Automatic
|
|
|
|
* speed/duplex mode change by HW after SGMII AN complete)
|
|
|
|
*
|
|
|
|
* Note: Since it is MAC side SGMII, there is no need to set
|
|
|
|
* SR_MII_AN_ADV. MAC side SGMII receives AN Tx Config from
|
|
|
|
* PHY about the link state change after C28 AN is completed
|
|
|
|
* between PHY and Link Partner. There is also no need to
|
|
|
|
* trigger AN restart for MAC-side SGMII.
|
|
|
|
*/
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, DW_VR_MII_AN_CTRL);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret &= ~(DW_VR_MII_PCS_MODE_MASK | DW_VR_MII_TX_CONFIG_MASK);
|
|
|
|
ret |= (DW_VR_MII_PCS_MODE_C37_SGMII <<
|
|
|
|
DW_VR_MII_AN_CTRL_PCS_MODE_SHIFT &
|
|
|
|
DW_VR_MII_PCS_MODE_MASK);
|
|
|
|
ret |= (DW_VR_MII_TX_CONFIG_MAC_SIDE_SGMII <<
|
|
|
|
DW_VR_MII_AN_CTRL_TX_CONFIG_SHIFT &
|
|
|
|
DW_VR_MII_TX_CONFIG_MASK);
|
|
|
|
ret = xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_AN_CTRL, ret);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, DW_VR_MII_DIG_CTRL1);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret |= DW_VR_MII_DIG_CTRL1_MAC_AUTO_SW;
|
|
|
|
|
|
|
|
return xpcs_write(xpcs, MDIO_MMD_VEND2, DW_VR_MII_DIG_CTRL1, ret);
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static int xpcs_do_config(struct mdio_xpcs_args *xpcs,
|
|
|
|
phy_interface_t interface, unsigned int mode)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
const struct xpcs_compat *compat;
|
2020-03-09 16:36:26 +08:00
|
|
|
int ret;
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
compat = xpcs_find_compat(xpcs->id, interface);
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (!compat)
|
|
|
|
return -ENODEV;
|
|
|
|
|
|
|
|
switch (compat->an_mode) {
|
2021-03-15 13:27:06 +08:00
|
|
|
case DW_AN_C73:
|
2021-06-03 00:20:19 +08:00
|
|
|
if (phylink_autoneg_inband(mode)) {
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
ret = xpcs_config_aneg_c73(xpcs, compat);
|
2021-03-15 13:27:06 +08:00
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
break;
|
2021-03-15 13:27:07 +08:00
|
|
|
case DW_AN_C37_SGMII:
|
|
|
|
ret = xpcs_config_aneg_c37_sgmii(xpcs);
|
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
break;
|
2021-03-15 13:27:06 +08:00
|
|
|
default:
|
|
|
|
return -1;
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static int xpcs_config(struct phylink_pcs *pcs, unsigned int mode,
|
|
|
|
phy_interface_t interface,
|
|
|
|
const unsigned long *advertising,
|
|
|
|
bool permit_pause_to_mac)
|
|
|
|
{
|
|
|
|
struct mdio_xpcs_args *xpcs = phylink_pcs_to_xpcs(pcs);
|
|
|
|
|
|
|
|
return xpcs_do_config(xpcs, interface, mode);
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:06 +08:00
|
|
|
static int xpcs_get_state_c73(struct mdio_xpcs_args *xpcs,
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
struct phylink_link_state *state,
|
|
|
|
const struct xpcs_compat *compat)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
/* Link needs to be read first ... */
|
2021-03-15 13:27:06 +08:00
|
|
|
state->link = xpcs_read_link_c73(xpcs, state->an_enabled) > 0 ? 1 : 0;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
/* ... and then we check the faults. */
|
2021-03-15 13:27:06 +08:00
|
|
|
ret = xpcs_read_fault_c73(xpcs, state);
|
2020-03-09 16:36:26 +08:00
|
|
|
if (ret) {
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
ret = xpcs_soft_reset(xpcs, compat);
|
2020-03-09 16:36:26 +08:00
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
state->link = 0;
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
return xpcs_do_config(xpcs, state->interface, MLO_AN_INBAND);
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if (state->an_enabled && xpcs_aneg_done_c73(xpcs, state, compat)) {
|
2020-03-09 16:36:26 +08:00
|
|
|
state->an_complete = true;
|
2021-03-15 13:27:06 +08:00
|
|
|
xpcs_read_lpa_c73(xpcs, state);
|
|
|
|
xpcs_resolve_lpa_c73(xpcs, state);
|
2020-03-20 17:53:36 +08:00
|
|
|
} else if (state->an_enabled) {
|
|
|
|
state->link = 0;
|
2020-03-09 16:36:26 +08:00
|
|
|
} else if (state->link) {
|
|
|
|
xpcs_resolve_pma(xpcs, state);
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
static int xpcs_get_state_c37_sgmii(struct mdio_xpcs_args *xpcs,
|
|
|
|
struct phylink_link_state *state)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
/* Reset link_state */
|
|
|
|
state->link = false;
|
|
|
|
state->speed = SPEED_UNKNOWN;
|
|
|
|
state->duplex = DUPLEX_UNKNOWN;
|
|
|
|
state->pause = 0;
|
|
|
|
|
|
|
|
/* For C37 SGMII mode, we check DW_VR_MII_AN_INTR_STS for link
|
|
|
|
* status, speed and duplex.
|
|
|
|
*/
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, DW_VR_MII_AN_INTR_STS);
|
|
|
|
if (ret < 0)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
if (ret & DW_VR_MII_C37_ANSGM_SP_LNKSTS) {
|
|
|
|
int speed_value;
|
|
|
|
|
|
|
|
state->link = true;
|
|
|
|
|
|
|
|
speed_value = (ret & DW_VR_MII_AN_STS_C37_ANSGM_SP) >>
|
|
|
|
DW_VR_MII_AN_STS_C37_ANSGM_SP_SHIFT;
|
|
|
|
if (speed_value == DW_VR_MII_C37_ANSGM_SP_1000)
|
|
|
|
state->speed = SPEED_1000;
|
|
|
|
else if (speed_value == DW_VR_MII_C37_ANSGM_SP_100)
|
|
|
|
state->speed = SPEED_100;
|
|
|
|
else
|
|
|
|
state->speed = SPEED_10;
|
|
|
|
|
|
|
|
if (ret & DW_VR_MII_AN_STS_C37_ANSGM_FD)
|
|
|
|
state->duplex = DUPLEX_FULL;
|
|
|
|
else
|
|
|
|
state->duplex = DUPLEX_HALF;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static void xpcs_get_state(struct phylink_pcs *pcs,
|
|
|
|
struct phylink_link_state *state)
|
2021-03-15 13:27:06 +08:00
|
|
|
{
|
2021-06-03 00:20:19 +08:00
|
|
|
struct mdio_xpcs_args *xpcs = phylink_pcs_to_xpcs(pcs);
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
const struct xpcs_compat *compat;
|
2021-03-15 13:27:06 +08:00
|
|
|
int ret;
|
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
compat = xpcs_find_compat(xpcs->id, state->interface);
|
|
|
|
if (!compat)
|
2021-06-03 00:20:19 +08:00
|
|
|
return;
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
|
|
|
|
switch (compat->an_mode) {
|
2021-03-15 13:27:06 +08:00
|
|
|
case DW_AN_C73:
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
ret = xpcs_get_state_c73(xpcs, state, compat);
|
2021-06-03 00:20:19 +08:00
|
|
|
if (ret) {
|
|
|
|
pr_err("xpcs_get_state_c73 returned %pe\n",
|
|
|
|
ERR_PTR(ret));
|
|
|
|
return;
|
|
|
|
}
|
2021-03-15 13:27:06 +08:00
|
|
|
break;
|
2021-03-15 13:27:07 +08:00
|
|
|
case DW_AN_C37_SGMII:
|
|
|
|
ret = xpcs_get_state_c37_sgmii(xpcs, state);
|
2021-06-03 00:20:19 +08:00
|
|
|
if (ret) {
|
|
|
|
pr_err("xpcs_get_state_c37_sgmii returned %pe\n",
|
|
|
|
ERR_PTR(ret));
|
|
|
|
}
|
2021-03-15 13:27:07 +08:00
|
|
|
break;
|
2021-03-15 13:27:06 +08:00
|
|
|
default:
|
2021-06-03 00:20:19 +08:00
|
|
|
return;
|
2021-03-15 13:27:06 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static void xpcs_link_up(struct phylink_pcs *pcs, unsigned int mode,
|
|
|
|
phy_interface_t interface, int speed, int duplex)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
2021-06-03 00:20:19 +08:00
|
|
|
struct mdio_xpcs_args *xpcs = phylink_pcs_to_xpcs(pcs);
|
|
|
|
|
2020-03-09 16:36:26 +08:00
|
|
|
if (interface == PHY_INTERFACE_MODE_USXGMII)
|
|
|
|
return xpcs_config_usxgmii(xpcs, speed);
|
|
|
|
}
|
|
|
|
|
|
|
|
static u32 xpcs_get_id(struct mdio_xpcs_args *xpcs)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
u32 id;
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
/* First, search C73 PCS using PCS MMD */
|
2020-03-09 16:36:26 +08:00
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MII_PHYSID1);
|
|
|
|
if (ret < 0)
|
|
|
|
return 0xffffffff;
|
|
|
|
|
|
|
|
id = ret << 16;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_PCS, MII_PHYSID2);
|
|
|
|
if (ret < 0)
|
|
|
|
return 0xffffffff;
|
|
|
|
|
2021-03-15 13:27:07 +08:00
|
|
|
/* If Device IDs are not all zeros, we found C73 AN-type device */
|
|
|
|
if (id | ret)
|
|
|
|
return id | ret;
|
|
|
|
|
|
|
|
/* Next, search C37 PCS using Vendor-Specific MII MMD */
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, MII_PHYSID1);
|
|
|
|
if (ret < 0)
|
|
|
|
return 0xffffffff;
|
|
|
|
|
|
|
|
id = ret << 16;
|
|
|
|
|
|
|
|
ret = xpcs_read(xpcs, MDIO_MMD_VEND2, MII_PHYSID2);
|
|
|
|
if (ret < 0)
|
|
|
|
return 0xffffffff;
|
|
|
|
|
|
|
|
/* If Device IDs are not all zeros, we found C37 AN-type device */
|
|
|
|
if (id | ret)
|
|
|
|
return id | ret;
|
|
|
|
|
|
|
|
return 0xffffffff;
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:12 +08:00
|
|
|
static const struct xpcs_compat synopsys_xpcs_compat[DW_XPCS_INTERFACE_MAX] = {
|
|
|
|
[DW_XPCS_USXGMII] = {
|
|
|
|
.supported = xpcs_usxgmii_features,
|
|
|
|
.interface = xpcs_usxgmii_interfaces,
|
|
|
|
.num_interfaces = ARRAY_SIZE(xpcs_usxgmii_interfaces),
|
|
|
|
.an_mode = DW_AN_C73,
|
|
|
|
},
|
|
|
|
[DW_XPCS_10GKR] = {
|
|
|
|
.supported = xpcs_10gkr_features,
|
|
|
|
.interface = xpcs_10gkr_interfaces,
|
|
|
|
.num_interfaces = ARRAY_SIZE(xpcs_10gkr_interfaces),
|
|
|
|
.an_mode = DW_AN_C73,
|
|
|
|
},
|
|
|
|
[DW_XPCS_XLGMII] = {
|
|
|
|
.supported = xpcs_xlgmii_features,
|
|
|
|
.interface = xpcs_xlgmii_interfaces,
|
|
|
|
.num_interfaces = ARRAY_SIZE(xpcs_xlgmii_interfaces),
|
|
|
|
.an_mode = DW_AN_C73,
|
|
|
|
},
|
|
|
|
[DW_XPCS_SGMII] = {
|
|
|
|
.supported = xpcs_sgmii_features,
|
|
|
|
.interface = xpcs_sgmii_interfaces,
|
|
|
|
.num_interfaces = ARRAY_SIZE(xpcs_sgmii_interfaces),
|
|
|
|
.an_mode = DW_AN_C37_SGMII,
|
|
|
|
},
|
|
|
|
};
|
|
|
|
|
|
|
|
static const struct xpcs_id xpcs_id_list[] = {
|
|
|
|
{
|
|
|
|
.id = SYNOPSYS_XPCS_ID,
|
|
|
|
.mask = SYNOPSYS_XPCS_MASK,
|
|
|
|
.compat = synopsys_xpcs_compat,
|
|
|
|
},
|
|
|
|
};
|
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
static const struct phylink_pcs_ops xpcs_phylink_ops = {
|
|
|
|
.pcs_config = xpcs_config,
|
|
|
|
.pcs_get_state = xpcs_get_state,
|
|
|
|
.pcs_link_up = xpcs_link_up,
|
|
|
|
};
|
|
|
|
|
2021-06-03 00:20:18 +08:00
|
|
|
struct mdio_xpcs_args *xpcs_create(struct mdio_device *mdiodev,
|
|
|
|
phy_interface_t interface)
|
2020-03-09 16:36:26 +08:00
|
|
|
{
|
2021-06-03 00:20:18 +08:00
|
|
|
struct mdio_xpcs_args *xpcs;
|
|
|
|
u32 xpcs_id;
|
|
|
|
int i, ret;
|
|
|
|
|
|
|
|
xpcs = kzalloc(sizeof(*xpcs), GFP_KERNEL);
|
|
|
|
if (!xpcs)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
xpcs->mdiodev = mdiodev;
|
|
|
|
|
|
|
|
xpcs_id = xpcs_get_id(xpcs);
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(xpcs_id_list); i++) {
|
2021-06-03 00:20:12 +08:00
|
|
|
const struct xpcs_id *entry = &xpcs_id_list[i];
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
const struct xpcs_compat *compat;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
if ((xpcs_id & entry->mask) != entry->id)
|
|
|
|
continue;
|
2020-03-09 16:36:26 +08:00
|
|
|
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
xpcs->id = entry;
|
|
|
|
|
|
|
|
compat = xpcs_find_compat(entry, interface);
|
2021-06-03 00:20:18 +08:00
|
|
|
if (!compat) {
|
|
|
|
ret = -ENODEV;
|
|
|
|
goto out;
|
|
|
|
}
|
net: pcs: xpcs: make the checks related to the PHY interface mode stateless
The operating mode of the driver is currently to populate its
struct mdio_xpcs_args::supported and struct mdio_xpcs_args::an_mode
statically in xpcs_probe(), based on the passed phy_interface_t,
and work with those.
However this is not the operation that phylink expects from a PCS
driver, because the port might be attached to an SFP cage that triggers
changes of the phy_interface_t dynamically as one SFP module is
unpluggged and another is plugged.
To migrate towards that model, the struct mdio_xpcs_args should not
cache anything related to the phy_interface_t, but just look up the
statically defined, const struct xpcs_compat structure corresponding to
the detected PCS OUI/model number.
So we delete the "supported" and "an_mode" members of struct
mdio_xpcs_args, and add the "id" structure there (since the ID is not
expected to change at runtime).
Since xpcs->supported is used deep in the code in _xpcs_config_aneg_c73(),
we need to modify some function headers to pass the xpcs_compat from all
callers. In turn, the xpcs_compat is always supplied externally to the
xpcs module:
- Most of the time by phylink
- In xpcs_probe() it is needed because xpcs_soft_reset() writes to
MDIO_MMD_PCS or to MDIO_MMD_VEND2 depending on whether an_mode is clause
37 or clause 73. In order to not introduce functional changes related
to when the soft reset is issued, we continue to require the initial
phy_interface_t argument to be passed to xpcs_probe() so we can pass
this on to xpcs_soft_reset().
- stmmac_open() wants to know whether to call stmmac_init_phy() or not,
and for that it looks inside xpcs->an_mode, because the clause 73
(backplane) AN modes supposedly do not have a PHY. Because we moved
an_mode outside of struct mdio_xpcs_args, this is now no longer
directly possible, so we introduce a helper function xpcs_get_an_mode()
which protects the data encapsulation of the xpcs module and requires
a phy_interface_t to be passed as argument. This function can look up
the appropriate compat based on the phy_interface_t.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2021-06-03 00:20:13 +08:00
|
|
|
|
2021-06-03 00:20:19 +08:00
|
|
|
xpcs->pcs.ops = &xpcs_phylink_ops;
|
|
|
|
xpcs->pcs.poll = true;
|
|
|
|
|
2021-06-03 00:20:18 +08:00
|
|
|
ret = xpcs_soft_reset(xpcs, compat);
|
|
|
|
if (ret)
|
|
|
|
goto out;
|
|
|
|
|
|
|
|
return xpcs;
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
|
|
|
|
2021-06-03 00:20:18 +08:00
|
|
|
ret = -ENODEV;
|
|
|
|
|
|
|
|
out:
|
|
|
|
kfree(xpcs);
|
|
|
|
|
|
|
|
return ERR_PTR(ret);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(xpcs_create);
|
|
|
|
|
|
|
|
void xpcs_destroy(struct mdio_xpcs_args *xpcs)
|
|
|
|
{
|
|
|
|
kfree(xpcs);
|
2020-03-09 16:36:26 +08:00
|
|
|
}
|
2021-06-03 00:20:18 +08:00
|
|
|
EXPORT_SYMBOL_GPL(xpcs_destroy);
|
2020-03-09 16:36:26 +08:00
|
|
|
|
|
|
|
MODULE_LICENSE("GPL v2");
|