summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAjit Khaparde <ajit.khaparde@broadcom.com>2019-11-18 14:23:45 -0800
committerAjit Khaparde <ajit.khaparde@broadcom.com>2019-11-19 10:32:39 -0800
commitc5be81ef7c54cdc1e619ea5b1e75e7f400d5d7fe (patch)
tree43dfbf6f3d7e63d6590d88a1144f9e8774d42320
parent3093713cb76aca4154f06e9afb197e2af1cb5403 (diff)
downloaddpdk-next-net-brcm-c5be81ef7c54cdc1e619ea5b1e75e7f400d5d7fe.zip
dpdk-next-net-brcm-c5be81ef7c54cdc1e619ea5b1e75e7f400d5d7fe.tar.gz
dpdk-next-net-brcm-c5be81ef7c54cdc1e619ea5b1e75e7f400d5d7fe.tar.xz
net/bnxt: fix to ignore resource qcaps error with older FW
On some old versions of FW, bnxt_hwrm_func_resc_qcaps can return an error. This is because the command was not implemented completely in FW till the subsequent version. Ignore the error and continue with the driver initialization. Fixes: edafb57ba4a1 ("net/bnxt: fix VF resource allocation") Cc: stable@dpdk.org Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com> Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
-rw-r--r--drivers/net/bnxt/bnxt_hwrm.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index 2cba007..1d3732e 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -692,7 +692,12 @@ int bnxt_hwrm_func_qcaps(struct bnxt *bp)
bp->flags |= BNXT_FLAG_NEW_RM;
}
- return rc;
+ /* On older FW,
+ * bnxt_hwrm_func_resc_qcaps can fail and cause init failure.
+ * But the error can be ignored. Return success.
+ */
+
+ return 0;
}
/* VNIC cap covers capability of all VNICs. So no need to pass vnic_id */
@@ -901,7 +906,7 @@ int bnxt_hwrm_func_resc_qcaps(struct bnxt *bp)
rc = bnxt_hwrm_send_message(bp, &req, sizeof(req), BNXT_USE_CHIMP_MB);
- HWRM_CHECK_RESULT();
+ HWRM_CHECK_RESULT_SILENT();
if (BNXT_VF(bp)) {
bp->max_rsscos_ctx = rte_le_to_cpu_16(resp->max_rsscos_ctx);