leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: ag71xx: make use of generic NET_SELFTESTS library

Browse Source 
With this patch the ag71xx on Atheros AR9331 will able to run generic net
selftests.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Oleksij Rempel 2021-04-19 15:01:05 +02:00 committed by David S. Miller
parent 6016ba345f
commit b62a12fc04
2 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
drivers/net/ethernet/atheros/Kconfig
View File

@ -20,6 +20,7 @@ if NET_VENDOR_ATHEROS
config AG71XX config AG71XX
tristate "Atheros AR7XXX/AR9XXX built-in ethernet mac support" tristate "Atheros AR7XXX/AR9XXX built-in ethernet mac support"
depends on ATH79 depends on ATH79
select NET_SELFTESTS
select PHYLINK select PHYLINK
help help
If you wish to compile a kernel for AR7XXX/91XXX and enable If you wish to compile a kernel for AR7XXX/91XXX and enable

16
drivers/net/ethernet/atheros/ag71xx.c
View File

@ -37,6 +37,7 @@
#include <linux/reset.h> #include <linux/reset.h>
#include <linux/clk.h> #include <linux/clk.h>
#include <linux/io.h> #include <linux/io.h>
#include <net/selftests.h>
/* For our NAPI weight bigger does *NOT* mean better - it means more /* For our NAPI weight bigger does *NOT* mean better - it means more
* D-cache misses and lots more wasted cycles than we'll ever * D-cache misses and lots more wasted cycles than we'll ever
@ -497,12 +498,17 @@ static int ag71xx_ethtool_set_pauseparam(struct net_device *ndev,
static void ag71xx_ethtool_get_strings(struct net_device *netdev, u32 sset, static void ag71xx_ethtool_get_strings(struct net_device *netdev, u32 sset,
u8 *data) u8 *data)
{ {
if (sset == ETH_SS_STATS) {
int i; int i;
switch (sset) {
case ETH_SS_STATS:
for (i = 0; i < ARRAY_SIZE(ag71xx_statistics); i++) for (i = 0; i < ARRAY_SIZE(ag71xx_statistics); i++)
memcpy(data + i * ETH_GSTRING_LEN, memcpy(data + i * ETH_GSTRING_LEN,
ag71xx_statistics[i].name, ETH_GSTRING_LEN); ag71xx_statistics[i].name, ETH_GSTRING_LEN);
break;
case ETH_SS_TEST:
net_selftest_get_strings(data);
break;
} }
} }
@ -519,10 +525,15 @@ static void ag71xx_ethtool_get_stats(struct net_device *ndev,
static int ag71xx_ethtool_get_sset_count(struct net_device *ndev, int sset) static int ag71xx_ethtool_get_sset_count(struct net_device *ndev, int sset)
{ {
if (sset == ETH_SS_STATS) switch (sset) {
case ETH_SS_STATS:
return ARRAY_SIZE(ag71xx_statistics); return ARRAY_SIZE(ag71xx_statistics);
case ETH_SS_TEST:
return net_selftest_get_count();
default:
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
}
static const struct ethtool_ops ag71xx_ethtool_ops = { static const struct ethtool_ops ag71xx_ethtool_ops = {
.get_drvinfo = ag71xx_get_drvinfo, .get_drvinfo = ag71xx_get_drvinfo,
@ -536,6 +547,7 @@ static const struct ethtool_ops ag71xx_ethtool_ops = {
.get_strings = ag71xx_ethtool_get_strings, .get_strings = ag71xx_ethtool_get_strings,
.get_ethtool_stats = ag71xx_ethtool_get_stats, .get_ethtool_stats = ag71xx_ethtool_get_stats,
.get_sset_count = ag71xx_ethtool_get_sset_count, .get_sset_count = ag71xx_ethtool_get_sset_count,
.self_test = net_selftest,
}; };
static int ag71xx_mdio_wait_busy(struct ag71xx *ag) static int ag71xx_mdio_wait_busy(struct ag71xx *ag)