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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Sep 2021 10:28:06 +0000 (12:28 +0200)
[ Upstream commit 6fdab8a3ade2adc123bbf5c4fdec3394560b1fb1 ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c

index 2bd476a..e2fd625 100644 (file)
@@ -452,7 +452,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)