bnxt_en: Fix asic.rev in devlink dev info command
authorMichael Chan <michael.chan@broadcom.com>
Sun, 5 Sep 2021 18:10:57 +0000 (14:10 -0400)
committerDavid S. Miller <davem@davemloft.net>
Sun, 5 Sep 2021 19:43:04 +0000 (20:43 +0100)
The current asic.rev is incomplete and does not include the metal
revision.  Add the metal revision and decode the complete asic
revision into the more common and readable form (A0, B0, etc).

Fixes: 7154917a12b2 ("bnxt_en: Refactor bnxt_dl_info_get().")
Reviewed-by: Edwin Peer <edwin.peer@broadcom.com>
Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c

index cb20e627282a62347249daef527a2acb4778d55f..9576547df4aba20c2d2d7459af6de3d8c557551b 100644 (file)
@@ -477,7 +477,7 @@ static int bnxt_dl_info_get(struct devlink *dl, struct devlink_info_req *req,
                return rc;
 
        ver_resp = &bp->ver_resp;
-       sprintf(buf, "%X", ver_resp->chip_rev);
+       sprintf(buf, "%c%d", 'A' + ver_resp->chip_rev, ver_resp->chip_metal);
        rc = bnxt_dl_info_put(bp, req, BNXT_VERSION_FIXED,
                              DEVLINK_INFO_VERSION_GENERIC_ASIC_REV, buf);
        if (rc)