diff mbox series

[net] qed: Fix a error code in qed_hw_init()

Message ID 20200131050326.n3axoo7axxvzcrv3@kili.mountain
State Accepted
Delegated to: David Miller
Headers show
Series [net] qed: Fix a error code in qed_hw_init() | expand

Commit Message

Dan Carpenter Jan. 31, 2020, 5:03 a.m. UTC
If the qed_fw_overlay_mem_alloc() then we should return -ENOMEM instead
of success.

Fixes: 30d5f85895fa ("qed: FW 8.42.2.0 Add fw overlay feature")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
 drivers/net/ethernet/qlogic/qed/qed_dev.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Jakub Kicinski Feb. 1, 2020, 7:41 p.m. UTC | #1
On Fri, 31 Jan 2020 08:03:26 +0300, Dan Carpenter wrote:
> If the qed_fw_overlay_mem_alloc() then we should return -ENOMEM instead
> of success.
> 
> Fixes: 30d5f85895fa ("qed: FW 8.42.2.0 Add fw overlay feature")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Applied, thanks Dan!
diff mbox series

Patch

diff --git a/drivers/net/ethernet/qlogic/qed/qed_dev.c b/drivers/net/ethernet/qlogic/qed/qed_dev.c
index 7912911337d4..03bdd2e26329 100644
--- a/drivers/net/ethernet/qlogic/qed/qed_dev.c
+++ b/drivers/net/ethernet/qlogic/qed/qed_dev.c
@@ -3114,6 +3114,7 @@  int qed_hw_init(struct qed_dev *cdev, struct qed_hw_init_params *p_params)
 		if (!p_hwfn->fw_overlay_mem) {
 			DP_NOTICE(p_hwfn,
 				  "Failed to allocate fw overlay memory\n");
+			rc = -ENOMEM;
 			goto load_err;
 		}