diff options
author | Michael Chan | 2016-09-19 09:58:04 +0200 |
---|---|---|
committer | David S. Miller | 2016-09-20 03:32:25 +0200 |
commit | 47f8e8b9bbbbe00740786bd1da0d5097d45ba46b (patch) | |
tree | 127d358416b8b4b97d007c27b435778ecd269664 /drivers/net/ethernet/broadcom/bnxt/bnxt.c | |
parent | bnxt_en: Added support for Secure Firmware Update (diff) | |
download | kernel-qcow2-linux-47f8e8b9bbbbe00740786bd1da0d5097d45ba46b.tar.gz kernel-qcow2-linux-47f8e8b9bbbbe00740786bd1da0d5097d45ba46b.tar.xz kernel-qcow2-linux-47f8e8b9bbbbe00740786bd1da0d5097d45ba46b.zip |
bnxt_en: Fix ethtool -l|-L inconsistent channel counts.
The existing code is inconsistent in reporting and accepting the combined
channel count. bnxt_get_channels() reports maximum combined as the
maximum rx count. bnxt_set_channels() accepts combined count that
cannot be bigger than max rx or max tx.
For example, if max rx = 2 and max tx = 1, we report max supported
combined to be 2. But if the user tries to set combined to 2, it will
fail because 2 is bigger than max tx which is 1.
Fix the code to be consistent. Max allowed combined = max(max_rx, max_tx).
We will accept a combined channel count <= max(max_rx, max_tx).
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/bnxt/bnxt.c')
0 files changed, 0 insertions, 0 deletions