Commit 6fc2786e authored Nov 18, 2020 by Zhang Changzhong Committed by Greg Kroah-Hartman Dec 02, 2020

bnxt_en: fix error return code in bnxt_init_one()



[ Upstream commit b5f796b62c98cd8c219c4b788ecb6e1218e648cb ]

Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.

Fixes: c213eae8 ("bnxt_en: Improve VF/PF link change logic.")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Changzhong <zhangchangzhong@huawei.com>
Reviewed-by: Edwin Peer <edwin.peer@broadcom.com>
Link: https://lore.kernel.org/r/1605701851-20270-1-git-send-email-zhangchangzhong@huawei.com


Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 0b364d26

drivers/net/ethernet/broadcom/bnxt/bnxt.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -9120,6 +9120,7 @@ static int bnxt_init_one(struct pci_dev pdev, const struct pci_device_id ent)
		create_singlethread_workqueue("bnxt_pf_wq");
		if (!bnxt_pf_wq) {
		dev_err(&pdev->dev, "Unable to create workqueue.\n");
		rc = -ENOMEM;
		goto init_err_pci_clean;
		}
		}