bnx2x: Utilize firmware 7.13.21.0
authorManish Chopra <manishc@marvell.com>
Fri, 17 Dec 2021 16:55:51 +0000 (08:55 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 20 Dec 2021 11:29:20 +0000 (11:29 +0000)
This new firmware addresses few important issues and enhancements
as mentioned below -

- Support direct invalidation of FP HSI Ver per function ID, required for
  invalidating FP HSI Ver prior to each VF start, as there is no VF start
- BRB hardware block parity error detection support for the driver
- Fix the FCOE underrun flow
- Fix PSOD during FCoE BFS over the NIC ports after preboot driver
- Maintains backward compatibility

This patch incorporates this new firmware 7.13.21.0 in bnx2x driver.

Signed-off-by: Manish Chopra <manishc@marvell.com>
Signed-off-by: Prabhakar Kushwaha <pkushwaha@marvell.com>
Signed-off-by: Alok Prasad <palok@marvell.com>
Signed-off-by: Ariel Elior <aelior@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x.h
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c
drivers/net/ethernet/broadcom/bnx2x/bnx2x_fw_defs.h
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c

index 2b06d78..a19dd67 100644 (file)
@@ -1850,6 +1850,14 @@ struct bnx2x {
 
        /* Vxlan/Geneve related information */
        u16 udp_tunnel_ports[BNX2X_UDP_PORT_MAX];
+
+#define FW_CAP_INVALIDATE_VF_FP_HSI    BIT(0)
+       u32 fw_cap;
+
+       u32 fw_major;
+       u32 fw_minor;
+       u32 fw_rev;
+       u32 fw_eng;
 };
 
 /* Tx queues may be less or equal to Rx queues */
@@ -2525,5 +2533,6 @@ void bnx2x_register_phc(struct bnx2x *bp);
  * Meant for implicit re-load flows.
  */
 int bnx2x_vlan_reconfigure_vid(struct bnx2x *bp);
-
+int bnx2x_init_firmware(struct bnx2x *bp);
+void bnx2x_release_firmware(struct bnx2x *bp);
 #endif /* bnx2x.h */
index 54a2334..8d36ebb 100644 (file)
@@ -2365,10 +2365,8 @@ int bnx2x_compare_fw_ver(struct bnx2x *bp, u32 load_code, bool print_err)
        if (load_code != FW_MSG_CODE_DRV_LOAD_COMMON_CHIP &&
            load_code != FW_MSG_CODE_DRV_LOAD_COMMON) {
                /* build my FW version dword */
-               u32 my_fw = (BCM_5710_FW_MAJOR_VERSION) +
-                       (BCM_5710_FW_MINOR_VERSION << 8) +
-                       (BCM_5710_FW_REVISION_VERSION << 16) +
-                       (BCM_5710_FW_ENGINEERING_VERSION << 24);
+               u32 my_fw = (bp->fw_major) + (bp->fw_minor << 8) +
+                               (bp->fw_rev << 16) + (bp->fw_eng << 24);
 
                /* read loaded FW from chip */
                u32 loaded_fw = REG_RD(bp, XSEM_REG_PRAM);
index 3f84352..a84d015 100644 (file)
        IRO[221].m2))
 #define XSTORM_VF_TO_PF_OFFSET(funcId) \
        (IRO[48].base + ((funcId) * IRO[48].m1))
+#define XSTORM_ETH_FUNCTION_INFO_FP_HSI_VALID_E2_OFFSET(fid)   \
+       (IRO[386].base + ((fid) * IRO[386].m1))
 #define COMMON_ASM_INVALID_ASSERT_OPCODE 0x0
 
 /* eth hsi version */
index 622fadc..611efee 100644 (file)
@@ -3024,7 +3024,8 @@ struct afex_stats {
 
 #define BCM_5710_FW_MAJOR_VERSION                      7
 #define BCM_5710_FW_MINOR_VERSION                      13
-#define BCM_5710_FW_REVISION_VERSION           15
+#define BCM_5710_FW_REVISION_VERSION           21
+#define BCM_5710_FW_REVISION_VERSION_V15       15
 #define BCM_5710_FW_ENGINEERING_VERSION                0
 #define BCM_5710_FW_COMPILE_FLAGS                      1
 
index 651bc1d..4953f5e 100644 (file)
        __stringify(BCM_5710_FW_MINOR_VERSION) "."      \
        __stringify(BCM_5710_FW_REVISION_VERSION) "."   \
        __stringify(BCM_5710_FW_ENGINEERING_VERSION)
+
+#define FW_FILE_VERSION_V15                            \
+       __stringify(BCM_5710_FW_MAJOR_VERSION) "."      \
+       __stringify(BCM_5710_FW_MINOR_VERSION) "."      \
+       __stringify(BCM_5710_FW_REVISION_VERSION_V15) "."       \
+       __stringify(BCM_5710_FW_ENGINEERING_VERSION)
+
 #define FW_FILE_NAME_E1                "bnx2x/bnx2x-e1-" FW_FILE_VERSION ".fw"
 #define FW_FILE_NAME_E1H       "bnx2x/bnx2x-e1h-" FW_FILE_VERSION ".fw"
 #define FW_FILE_NAME_E2                "bnx2x/bnx2x-e2-" FW_FILE_VERSION ".fw"
+#define FW_FILE_NAME_E1_V15    "bnx2x/bnx2x-e1-" FW_FILE_VERSION_V15 ".fw"
+#define FW_FILE_NAME_E1H_V15   "bnx2x/bnx2x-e1h-" FW_FILE_VERSION_V15 ".fw"
+#define FW_FILE_NAME_E2_V15    "bnx2x/bnx2x-e2-" FW_FILE_VERSION_V15 ".fw"
 
 /* Time in jiffies before concluding the transmitter is hung */
 #define TX_TIMEOUT             (5*HZ)
@@ -747,9 +757,7 @@ static int bnx2x_mc_assert(struct bnx2x *bp)
                  CHIP_IS_E1(bp) ? "everest1" :
                  CHIP_IS_E1H(bp) ? "everest1h" :
                  CHIP_IS_E2(bp) ? "everest2" : "everest3",
-                 BCM_5710_FW_MAJOR_VERSION,
-                 BCM_5710_FW_MINOR_VERSION,
-                 BCM_5710_FW_REVISION_VERSION);
+                 bp->fw_major, bp->fw_minor, bp->fw_rev);
 
        return rc;
 }
@@ -12308,6 +12316,15 @@ static int bnx2x_init_bp(struct bnx2x *bp)
 
        bnx2x_read_fwinfo(bp);
 
+       if (IS_PF(bp)) {
+               rc = bnx2x_init_firmware(bp);
+
+               if (rc) {
+                       bnx2x_free_mem_bp(bp);
+                       return rc;
+               }
+       }
+
        func = BP_FUNC(bp);
 
        /* need to reset chip if undi was active */
@@ -12320,6 +12337,7 @@ static int bnx2x_init_bp(struct bnx2x *bp)
 
                rc = bnx2x_prev_unload(bp);
                if (rc) {
+                       bnx2x_release_firmware(bp);
                        bnx2x_free_mem_bp(bp);
                        return rc;
                }
@@ -13317,16 +13335,11 @@ static int bnx2x_check_firmware(struct bnx2x *bp)
        /* Check FW version */
        offset = be32_to_cpu(fw_hdr->fw_version.offset);
        fw_ver = firmware->data + offset;
-       if ((fw_ver[0] != BCM_5710_FW_MAJOR_VERSION) ||
-           (fw_ver[1] != BCM_5710_FW_MINOR_VERSION) ||
-           (fw_ver[2] != BCM_5710_FW_REVISION_VERSION) ||
-           (fw_ver[3] != BCM_5710_FW_ENGINEERING_VERSION)) {
+       if (fw_ver[0] != bp->fw_major || fw_ver[1] != bp->fw_minor ||
+           fw_ver[2] != bp->fw_rev || fw_ver[3] != bp->fw_eng) {
                BNX2X_ERR("Bad FW version:%d.%d.%d.%d. Should be %d.%d.%d.%d\n",
-                      fw_ver[0], fw_ver[1], fw_ver[2], fw_ver[3],
-                      BCM_5710_FW_MAJOR_VERSION,
-                      BCM_5710_FW_MINOR_VERSION,
-                      BCM_5710_FW_REVISION_VERSION,
-                      BCM_5710_FW_ENGINEERING_VERSION);
+                         fw_ver[0], fw_ver[1], fw_ver[2], fw_ver[3],
+                         bp->fw_major, bp->fw_minor, bp->fw_rev, bp->fw_eng);
                return -EINVAL;
        }
 
@@ -13404,34 +13417,51 @@ do {                                                                  \
             (u8 *)bp->arr, len);                                       \
 } while (0)
 
-static int bnx2x_init_firmware(struct bnx2x *bp)
+int bnx2x_init_firmware(struct bnx2x *bp)
 {
-       const char *fw_file_name;
+       const char *fw_file_name, *fw_file_name_v15;
        struct bnx2x_fw_file_hdr *fw_hdr;
        int rc;
 
        if (bp->firmware)
                return 0;
 
-       if (CHIP_IS_E1(bp))
+       if (CHIP_IS_E1(bp)) {
                fw_file_name = FW_FILE_NAME_E1;
-       else if (CHIP_IS_E1H(bp))
+               fw_file_name_v15 = FW_FILE_NAME_E1_V15;
+       } else if (CHIP_IS_E1H(bp)) {
                fw_file_name = FW_FILE_NAME_E1H;
-       else if (!CHIP_IS_E1x(bp))
+               fw_file_name_v15 = FW_FILE_NAME_E1H_V15;
+       } else if (!CHIP_IS_E1x(bp)) {
                fw_file_name = FW_FILE_NAME_E2;
-       else {
+               fw_file_name_v15 = FW_FILE_NAME_E2_V15;
+       } else {
                BNX2X_ERR("Unsupported chip revision\n");
                return -EINVAL;
        }
+
        BNX2X_DEV_INFO("Loading %s\n", fw_file_name);
 
        rc = request_firmware(&bp->firmware, fw_file_name, &bp->pdev->dev);
        if (rc) {
-               BNX2X_ERR("Can't load firmware file %s\n",
-                         fw_file_name);
-               goto request_firmware_exit;
+               BNX2X_DEV_INFO("Trying to load older fw %s\n", fw_file_name_v15);
+
+               /* try to load prev version */
+               rc = request_firmware(&bp->firmware, fw_file_name_v15, &bp->pdev->dev);
+
+               if (rc)
+                       goto request_firmware_exit;
+
+               bp->fw_rev = BCM_5710_FW_REVISION_VERSION_V15;
+       } else {
+               bp->fw_cap |= FW_CAP_INVALIDATE_VF_FP_HSI;
+               bp->fw_rev = BCM_5710_FW_REVISION_VERSION;
        }
 
+       bp->fw_major = BCM_5710_FW_MAJOR_VERSION;
+       bp->fw_minor = BCM_5710_FW_MINOR_VERSION;
+       bp->fw_eng = BCM_5710_FW_ENGINEERING_VERSION;
+
        rc = bnx2x_check_firmware(bp);
        if (rc) {
                BNX2X_ERR("Corrupt firmware file %s\n", fw_file_name);
@@ -13487,7 +13517,7 @@ request_firmware_exit:
        return rc;
 }
 
-static void bnx2x_release_firmware(struct bnx2x *bp)
+void bnx2x_release_firmware(struct bnx2x *bp)
 {
        kfree(bp->init_ops_offsets);
        kfree(bp->init_ops);
@@ -14004,6 +14034,7 @@ static int bnx2x_init_one(struct pci_dev *pdev,
        return 0;
 
 init_one_freemem:
+       bnx2x_release_firmware(bp);
        bnx2x_free_mem_bp(bp);
 
 init_one_exit: