bnxt_en: Don't issue AP reset during ethtool's reset operation
authorSreekanth Reddy <sreekanth.reddy@broadcom.com>
Wed, 7 Jun 2023 07:54:05 +0000 (00:54 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 14 Jun 2023 09:15:22 +0000 (11:15 +0200)
commit53a0c6d5c9294bad7fa8390c73ac96f386b15ce2
tree2d4fe8dc1d656a51fdac92487e479d5f2abd8ad5
parent5df74018d1e6b8fdf4bba9be6e8f9bc8e638e243
bnxt_en: Don't issue AP reset during ethtool's reset operation

[ Upstream commit 1d997801c7cc6a7f542e46d5a6bf16f893ad3fe9 ]

Only older NIC controller's firmware uses the PROC AP reset type.
Firmware on 5731X/5741X and newer chips does not support this reset
type.  When bnxt_reset() issues a series of resets, this PROC AP
reset may actually fail on these newer chips because the firmware
is not ready to accept this unsupported command yet.  Avoid this
unnecessary error by skipping this reset type on chips that don't
support it.

Fixes: 7a13240e3718 ("bnxt_en: fix ethtool_reset_flags ABI violations")
Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Signed-off-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c