diff options
| author |   Mark Starovoytov <mstarovoitov@marvell.com> | 2020-07-20 21:32:34 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-07-20 18:07:38 -0700 |
| commit | b772112c5af0e45ddcacb144545f06bdc594c736 (patch) | |
| tree | 3338d21c2b50f7af0f8d3a7c07b2daa940a82bab /drivers/net/ethernet/aquantia/atlantic/aq_vec.c | |
| parent | net: atlantic: use simple assignment in _get_stats and _get_sw_stats (diff) | |
| download | linux-dev-b772112c5af0e45ddcacb144545f06bdc594c736.tar.xz linux-dev-b772112c5af0e45ddcacb144545f06bdc594c736.zip | |
net: atlantic: make _get_sw_stats return count as return value
This patch changes aq_vec_get_sw_stats() to return count as a return
value (which was unused) instead of an out parameter.
Signed-off-by: Mark Starovoytov <mstarovoitov@marvell.com>
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/aquantia/atlantic/aq_vec.c')
| -rw-r--r-- | drivers/net/ethernet/aquantia/atlantic/aq_vec.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_vec.c b/drivers/net/ethernet/aquantia/atlantic/aq_vec.c index 2acdaee18ba0..8f0a0d18e711 100644 --- a/drivers/net/ethernet/aquantia/atlantic/aq_vec.c +++ b/drivers/net/ethernet/aquantia/atlantic/aq_vec.c @@ -380,8 +380,7 @@ static void aq_vec_get_stats(struct aq_vec_s *self, } } -int aq_vec_get_sw_stats(struct aq_vec_s *self, const unsigned int tc, u64 *data, - unsigned int *p_count) +unsigned int aq_vec_get_sw_stats(struct aq_vec_s *self, const unsigned int tc, u64 *data) { struct aq_ring_stats_rx_s stats_rx; struct aq_ring_stats_tx_s stats_tx; @@ -401,8 +400,5 @@ int aq_vec_get_sw_stats(struct aq_vec_s *self, const unsigned int tc, u64 *data, data[++count] = stats_rx.lro_packets; data[++count] = stats_rx.errors; - if (p_count) - *p_count = ++count; - - return 0; + return ++count; } |