Commit 60453008 authored by Vikas Gupta's avatar Vikas Gupta Committed by Jakub Kicinski
Browse files

bnge: fix reserving resources from FW 



HWRM_FUNC_CFG is used to reserve resources, whereas HWRM_FUNC_QCFG is
intended for querying resource information from the firmware.
Since __bnge_hwrm_reserve_pf_rings() reserves resources for a specific
PF, the command type should be HWRM_FUNC_CFG.

Fixes: 627c67f0 ("bng_en: Add resource management support")
Signed-off-by: default avatarVikas Gupta <vikas.gupta@broadcom.com>
Reviewed-by: default avatarBhargava Chenna Marreddy <bhargava.marreddy@broadcom.com>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20260218052755.4097468-1-vikas.gupta@broadcom.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent e977fcb3

drivers/net/ethernet/broadcom/bnge/bnge_hwrm_lib.c

+1 −1
Original line number Diff line number Diff line
@@ -442,7 +442,7 @@ __bnge_hwrm_reserve_pf_rings(struct bnge_dev *bd, struct bnge_hw_rings *hwr) 
	struct hwrm_func_cfg_input *req;

	u32 enables = 0;



	if (bnge_hwrm_req_init(bd, req, HWRM_FUNC_QCFG))

	if (bnge_hwrm_req_init(bd, req, HWRM_FUNC_CFG))

		return NULL;



	req->fid = cpu_to_le16(0xffff);