bnxt: Avoid logging an unnecessary message when a flow can't be offloaded
authorSomnath Kotur <somnath.kotur@broadcom.com>
Thu, 31 Oct 2019 05:07:46 +0000 (01:07 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 31 Oct 2019 21:48:11 +0000 (14:48 -0700)
commit08f8280e8788202a67a359952cd436707f8789bd
treee9320e1894581db121570d7746663ebadf4b239c
parent90f906243bf633f07757467506dfab3422b43ca2
bnxt: Avoid logging an unnecessary message when a flow can't be offloaded

For every single case where bnxt_tc_can_offload() can fail, we are
logging a user friendly descriptive message anyway, but because of the
path it would take in case of failure, another redundant error message
would get logged. Just freeing the node and returning from the point of
failure should suffice.

Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c