| Message ID | 20210805103826.GB26417@kili (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | fb653827c758725b149b5c924a5eb50ab4812750 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net] bnx2x: fix an error code in bnx2x_nic_load() | expand |
| Context | Check | Description |
|---|---|---|
| netdev/apply | success | Patch already applied to net |
| netdev/tree_selection | success | Clearly marked for net |
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Thu, 5 Aug 2021 13:38:26 +0300 you wrote: > Set the error code if bnx2x_alloc_fw_stats_mem() fails. The current > code returns success. > > Fixes: ad5afc89365e ("bnx2x: Separate VF and PF logic") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Here is the summary with links: - [net] bnx2x: fix an error code in bnx2x_nic_load() https://git.kernel.org/netdev/net/c/fb653827c758 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c index 1a6ec1a12d53..b5d954cb409a 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c @@ -2669,7 +2669,8 @@ int bnx2x_nic_load(struct bnx2x *bp, int load_mode) } /* Allocated memory for FW statistics */ - if (bnx2x_alloc_fw_stats_mem(bp)) + rc = bnx2x_alloc_fw_stats_mem(bp); + if (rc) LOAD_ERROR_EXIT(bp, load_error0); /* request pf to initialize status blocks */
Set the error code if bnx2x_alloc_fw_stats_mem() fails. The current code returns success. Fixes: ad5afc89365e ("bnx2x: Separate VF and PF logic") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)