summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorManish Chopra <manishc@marvell.com>2019-12-11 09:59:56 -0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-01-12 12:11:57 +0100
commit9c914b0d4beb0af271049453d1e05ac278ebfabc (patch)
tree9441a505d76131d1bd126985de6da24fac7b2a01
parent58ee0e0b74e7e4d8dadc3597d3d721a9cd252a88 (diff)
downloadlinux-9c914b0d4beb0af271049453d1e05ac278ebfabc.tar.gz
linux-9c914b0d4beb0af271049453d1e05ac278ebfabc.tar.bz2
linux-9c914b0d4beb0af271049453d1e05ac278ebfabc.zip
bnx2x: Fix logic to get total no. of PFs per engine
[ Upstream commit ee699f89bdbaa19c399804504241b5c531b48888 ] Driver doesn't calculate total number of PFs configured on a given engine correctly which messed up resources in the PFs loaded on that engine, leading driver to exceed configuration of resources (like vlan filters etc.) beyond the limit per engine, which ended up with asserts from the firmware. Signed-off-by: Manish Chopra <manishc@marvell.com> Signed-off-by: Ariel Elior <aelior@marvell.com> Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h
index 4e091a11daaf..52bce009d096 100644
--- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h
+++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h
@@ -1112,7 +1112,7 @@ static inline u8 bnx2x_get_path_func_num(struct bnx2x *bp)
for (i = 0; i < E1H_FUNC_MAX / 2; i++) {
u32 func_config =
MF_CFG_RD(bp,
- func_mf_config[BP_PORT(bp) + 2 * i].
+ func_mf_config[BP_PATH(bp) + 2 * i].
config);
func_num +=
((func_config & FUNC_MF_CFG_FUNC_HIDE) ? 0 : 1);