From: Scott Branden <scott.bran...@broadcom.com>

Check return value of call to bnxt_hwrm_func_resc_qcaps in
bnxt_hwrm_if_change and return failure on error.

Reviewed-by: Edwin Peer <edwin.p...@broadcom.com>
Signed-off-by: Scott Branden <scott.bran...@broadcom.com>
Signed-off-by: Michael Chan <michael.c...@broadcom.com>
---
 drivers/net/ethernet/broadcom/bnxt/bnxt.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c 
b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
index 3624e79667b6..7f40dd7d847d 100644
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
@@ -9634,6 +9634,9 @@ static int bnxt_hwrm_if_change(struct bnxt *bp, bool up)
                        struct bnxt_hw_resc *hw_resc = &bp->hw_resc;
 
                        rc = bnxt_hwrm_func_resc_qcaps(bp, true);
+                       if (rc)
+                               netdev_err(bp->dev, "resc_qcaps failed\n");
+
                        hw_resc->resv_cp_rings = 0;
                        hw_resc->resv_stat_ctxs = 0;
                        hw_resc->resv_irqs = 0;
@@ -9647,7 +9650,7 @@ static int bnxt_hwrm_if_change(struct bnxt *bp, bool up)
                        }
                }
        }
-       return 0;
+       return rc;
 }
 
 static int bnxt_hwrm_port_led_qcaps(struct bnxt *bp)
-- 
2.18.1

Reply via email to