Re: [net-next PATCH v18 5/8] net: dsa: Add Airoha AN8855 5-Port Gigabit DSA Switch driver

Vladimir Oltean
2025-09-17T10:43:30Z

On Mon, Sep 15, 2025 at 12:45:41PM +0200, Christian Marangi wrote:
> +static const struct an8855_mib_desc an8855_mib[] = {
> + MIB_DESC(1, AN8855_PORT_MIB_TX_DROP, "TxDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_TX_CRC_ERR, "TxCrcErr"),
> + MIB_DESC(1, AN8855_PORT_MIB_TX_COLLISION, "TxCollision"),
> + MIB_DESC(1, AN8855_PORT_MIB_TX_OVERSIZE_DROP, "TxOversizeDrop"),
> + MIB_DESC(2, AN8855_PORT_MIB_TX_BAD_PKT_BYTES, "TxBadPktBytes"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_DROP, "RxDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_FILTERING, "RxFiltering"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_CRC_ERR, "RxCrcErr"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_CTRL_DROP, "RxCtrlDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_INGRESS_DROP, "RxIngressDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_RX_ARL_DROP, "RxArlDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_FLOW_CONTROL_DROP, "FlowControlDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_WRED_DROP, "WredDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_MIRROR_DROP, "MirrorDrop"),
> + MIB_DESC(2, AN8855_PORT_MIB_RX_BAD_PKT_BYTES, "RxBadPktBytes"),
> + MIB_DESC(1, AN8855_PORT_MIB_RXS_FLOW_SAMPLING_PKT_DROP, "RxsFlowSamplingPktDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_RXS_FLOW_TOTAL_PKT_DROP, "RxsFlowTotalPktDrop"),
> + MIB_DESC(1, AN8855_PORT_MIB_PORT_CONTROL_DROP, "PortControlDrop"),
> +};
> +
> +static int
> +an8855_mib_init(struct an8855_priv *priv)
> +{
> + int ret;
> +
> + ret = regmap_write(priv->regmap, AN8855_MIB_CCR,
> + AN8855_CCR_MIB_ENABLE);
> + if (ret)
> + return ret;
> +
> + return regmap_write(priv->regmap, AN8855_MIB_CCR,
> + AN8855_CCR_MIB_ACTIVATE);
> +}
> +
> +static void an8855_get_strings(struct dsa_switch *ds, int port,
> + u32 stringset, uint8_t *data)
> +{
> + int i;
> +
> + if (stringset != ETH_SS_STATS)
> + return;
> +
> + for (i = 0; i < ARRAY_SIZE(an8855_mib); i++)
> + ethtool_puts(&data, an8855_mib[i].name);

Same feedback as for yt921x. For new drivers we want in unstructured
ethtool -S only those statistics which are not exposed through standard
variants (to force the adoption of the new interfaces).

> +}
> +
> +static void an8855_read_port_stats(struct an8855_priv *priv, int port,
> + u32 offset, u8 size, uint64_t *data)
> +{
> + u32 val, reg = AN8855_PORT_MIB_COUNTER(port) + offset;
> +
> + regmap_read(priv->regmap, reg, &val);
> + *data = val;
> +
> + if (size == 2) {
> + regmap_read(priv->regmap, reg + 4, &val);
> + *data |= (u64)val << 32;
> + }
> +}
> +
> +static void an8855_get_ethtool_stats(struct dsa_switch *ds, int port,
> + uint64_t *data)
> +{
> + struct an8855_priv *priv = ds->priv;
> + const struct an8855_mib_desc *mib;
> + int i;
> +
> + for (i = 0; i < ARRAY_SIZE(an8855_mib); i++) {
> + mib = &an8855_mib[i];
> +
> + an8855_read_port_stats(priv, port, mib->offset, mib->size,
> + data + i);
> + }
> +}
> +
> +static int an8855_get_sset_count(struct dsa_switch *ds, int port,
> + int sset)
> +{
> + if (sset != ETH_SS_STATS)
> + return 0;
> +
> + return ARRAY_SIZE(an8855_mib);
> +}
> +
> +static void an8855_get_eth_mac_stats(struct dsa_switch *ds, int port,
> + struct ethtool_eth_mac_stats *mac_stats)
> +{
> + struct an8855_priv *priv = ds->priv;
> +
> + /* MIB counter doesn't provide a FramesTransmittedOK but instead
> + * provide stats for Unicast, Broadcast and Multicast frames separately.
> + * To simulate a global frame counter, read Unicast and addition Multicast
> + * and Broadcast later
> + */
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_UNICAST, 1,
> + &mac_stats->FramesTransmittedOK);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_SINGLE_COLLISION, 1,
> + &mac_stats->SingleCollisionFrames);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_MULTIPLE_COLLISION, 1,
> + &mac_stats->MultipleCollisionFrames);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_UNICAST, 1,
> + &mac_stats->FramesReceivedOK);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_BYTES, 2,
> + &mac_stats->OctetsTransmittedOK);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_ALIGN_ERR, 1,
> + &mac_stats->AlignmentErrors);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_DEFERRED, 1,
> + &mac_stats->FramesWithDeferredXmissions);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_LATE_COLLISION, 1,
> + &mac_stats->LateCollisions);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_EXCESSIVE_COLLISION, 1,
> + &mac_stats->FramesAbortedDueToXSColls);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_BYTES, 2,
> + &mac_stats->OctetsReceivedOK);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_MULTICAST, 1,
> + &mac_stats->MulticastFramesXmittedOK);
> + mac_stats->FramesTransmittedOK += mac_stats->MulticastFramesXmittedOK;
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_BROADCAST, 1,
> + &mac_stats->BroadcastFramesXmittedOK);
> + mac_stats->FramesTransmittedOK += mac_stats->BroadcastFramesXmittedOK;
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_MULTICAST, 1,
> + &mac_stats->MulticastFramesReceivedOK);
> + mac_stats->FramesReceivedOK += mac_stats->MulticastFramesReceivedOK;
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_BROADCAST, 1,
> + &mac_stats->BroadcastFramesReceivedOK);
> + mac_stats->FramesReceivedOK += mac_stats->BroadcastFramesReceivedOK;
> +}
> +
> +static const struct ethtool_rmon_hist_range an8855_rmon_ranges[] = {
> + { 0, 64 },
> + { 65, 127 },
> + { 128, 255 },
> + { 256, 511 },
> + { 512, 1023 },
> + { 1024, 1518 },
> + { 1519, AN8855_MAX_MTU },
> + {}
> +};
> +
> +static void an8855_get_rmon_stats(struct dsa_switch *ds, int port,
> + struct ethtool_rmon_stats *rmon_stats,
> + const struct ethtool_rmon_hist_range **ranges)
> +{
> + struct an8855_priv *priv = ds->priv;
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_UNDER_SIZE_ERR, 1,
> + &rmon_stats->undersize_pkts);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_OVER_SZ_ERR, 1,
> + &rmon_stats->oversize_pkts);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_FRAG_ERR, 1,
> + &rmon_stats->fragments);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_JABBER_ERR, 1,
> + &rmon_stats->jabbers);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_64, 1,
> + &rmon_stats->hist[0]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_65_TO_127, 1,
> + &rmon_stats->hist[1]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_128_TO_255, 1,
> + &rmon_stats->hist[2]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_256_TO_511, 1,
> + &rmon_stats->hist[3]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_512_TO_1023, 1,
> + &rmon_stats->hist[4]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_1024_TO_1518, 1,
> + &rmon_stats->hist[5]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PKT_SZ_1519_TO_MAX, 1,
> + &rmon_stats->hist[6]);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_64, 1,
> + &rmon_stats->hist_tx[0]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_65_TO_127, 1,
> + &rmon_stats->hist_tx[1]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_128_TO_255, 1,
> + &rmon_stats->hist_tx[2]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_256_TO_511, 1,
> + &rmon_stats->hist_tx[3]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_512_TO_1023, 1,
> + &rmon_stats->hist_tx[4]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_1024_TO_1518, 1,
> + &rmon_stats->hist_tx[5]);
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PKT_SZ_1519_TO_MAX, 1,
> + &rmon_stats->hist_tx[6]);
> +
> + *ranges = an8855_rmon_ranges;
> +}
> +
> +static void an8855_get_eth_ctrl_stats(struct dsa_switch *ds, int port,
> + struct ethtool_eth_ctrl_stats *ctrl_stats)
> +{
> + struct an8855_priv *priv = ds->priv;
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_TX_PAUSE, 1,
> + &ctrl_stats->MACControlFramesTransmitted);
> +
> + an8855_read_port_stats(priv, port, AN8855_PORT_MIB_RX_PAUSE, 1,
> + &ctrl_stats->MACControlFramesReceived);
> +}