diff options
author | Shay Agroskin <shayagr@amazon.com> | 2020-12-08 20:02:03 +0200 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2020-12-09 15:26:40 -0800 |
commit | 1e5847395eebfc98e89558c9f5d0058b949ddc1d (patch) | |
tree | 199aede1c7ae2432af2921a4b4543d0eb8a572f5 /drivers/net/ethernet/amazon/ena/ena_ethtool.c | |
parent | net: ena: store values in their appropriate variables types (diff) | |
download | linux-1e5847395eebfc98e89558c9f5d0058b949ddc1d.tar.xz linux-1e5847395eebfc98e89558c9f5d0058b949ddc1d.zip |
net: ena: fix coding style nits
This commit fixes two nits, but it does not generate any change to binary
because of the optimization of gcc.
- use `count` instead of `channels->combined_count`
- change return type from `int` to `bool`
Also add spaces and change macro order in OR assignment to make the code
easier to read.
Signed-off-by: Sameeh Jubran <sameehj@amazon.com>
Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.co.jp>
Signed-off-by: Shay Agroskin <shayagr@amazon.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/ethernet/amazon/ena/ena_ethtool.c')
-rw-r--r-- | drivers/net/ethernet/amazon/ena/ena_ethtool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/amazon/ena/ena_ethtool.c b/drivers/net/ethernet/amazon/ena/ena_ethtool.c index 6cdd9efe8df3..2ad44ae74cf6 100644 --- a/drivers/net/ethernet/amazon/ena/ena_ethtool.c +++ b/drivers/net/ethernet/amazon/ena/ena_ethtool.c @@ -839,7 +839,7 @@ static int ena_set_channels(struct net_device *netdev, /* The check for max value is already done in ethtool */ if (count < ENA_MIN_NUM_IO_QUEUES || (ena_xdp_present(adapter) && - !ena_xdp_legal_queue_count(adapter, channels->combined_count))) + !ena_xdp_legal_queue_count(adapter, count))) return -EINVAL; return ena_update_queue_count(adapter, count); |