Commit 1ddee09f authored by Ben Hutchings's avatar Ben Hutchings Committed by David S. Miller

tehuti: Convert ethtool get_stats_count() ops to get_sset_count()

This string query operation was supposed to be replaced by the
generic get_sset_count() starting in 2007.  Convert tehuti's
implementation.

Also remove the dummy self-test name which was not used since tehuti
does not advertise any self-tests.
Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent df8b4ec8
...@@ -2105,12 +2105,6 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -2105,12 +2105,6 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
} }
/****************** Ethtool interface *********************/ /****************** Ethtool interface *********************/
/* get strings for tests */
static const char
bdx_test_names[][ETH_GSTRING_LEN] = {
"No tests defined"
};
/* get strings for statistics counters */ /* get strings for statistics counters */
static const char static const char
bdx_stat_names[][ETH_GSTRING_LEN] = { bdx_stat_names[][ETH_GSTRING_LEN] = {
...@@ -2380,9 +2374,6 @@ bdx_set_ringparam(struct net_device *netdev, struct ethtool_ringparam *ring) ...@@ -2380,9 +2374,6 @@ bdx_set_ringparam(struct net_device *netdev, struct ethtool_ringparam *ring)
static void bdx_get_strings(struct net_device *netdev, u32 stringset, u8 *data) static void bdx_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
{ {
switch (stringset) { switch (stringset) {
case ETH_SS_TEST:
memcpy(data, *bdx_test_names, sizeof(bdx_test_names));
break;
case ETH_SS_STATS: case ETH_SS_STATS:
memcpy(data, *bdx_stat_names, sizeof(bdx_stat_names)); memcpy(data, *bdx_stat_names, sizeof(bdx_stat_names));
break; break;
...@@ -2390,15 +2381,21 @@ static void bdx_get_strings(struct net_device *netdev, u32 stringset, u8 *data) ...@@ -2390,15 +2381,21 @@ static void bdx_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
} }
/* /*
* bdx_get_stats_count - return number of 64bit statistics counters * bdx_get_sset_count - return number of statistics or tests
* @netdev * @netdev
*/ */
static int bdx_get_stats_count(struct net_device *netdev) static int bdx_get_sset_count(struct net_device *netdev, int stringset)
{ {
struct bdx_priv *priv = netdev_priv(netdev); struct bdx_priv *priv = netdev_priv(netdev);
BDX_ASSERT(ARRAY_SIZE(bdx_stat_names)
!= sizeof(struct bdx_stats) / sizeof(u64)); switch (stringset) {
return ((priv->stats_flag) ? ARRAY_SIZE(bdx_stat_names) : 0); case ETH_SS_STATS:
BDX_ASSERT(ARRAY_SIZE(bdx_stat_names)
!= sizeof(struct bdx_stats) / sizeof(u64));
return ((priv->stats_flag) ? ARRAY_SIZE(bdx_stat_names) : 0);
default:
return -EINVAL;
}
} }
/* /*
...@@ -2441,7 +2438,7 @@ static void bdx_ethtool_ops(struct net_device *netdev) ...@@ -2441,7 +2438,7 @@ static void bdx_ethtool_ops(struct net_device *netdev)
.get_sg = ethtool_op_get_sg, .get_sg = ethtool_op_get_sg,
.get_tso = ethtool_op_get_tso, .get_tso = ethtool_op_get_tso,
.get_strings = bdx_get_strings, .get_strings = bdx_get_strings,
.get_stats_count = bdx_get_stats_count, .get_sset_count = bdx_get_sset_count,
.get_ethtool_stats = bdx_get_ethtool_stats, .get_ethtool_stats = bdx_get_ethtool_stats,
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment