Commit 2013d038 authored by Vasundhara Volam's avatar Vasundhara Volam Committed by David S. Miller

bnxt_en: Fix "fw.mgmt" and "fw.nsci" info via devlink info_get cb

Fix macro names to report fw.mgmt and fw.ncsi versions to match the
devlink documentation.

Example display after fixes:

$ devlink dev info pci/0000:af:00.0
pci/0000:af:00.0:
  driver bnxt_en
  serial_number B0-26-28-FF-FE-25-84-20
  versions:
      fixed:
        board.id BCM957454A4540
        asic.id C454
        asic.rev 1
      running:
        fw 216.1.154.0
        fw.psid 0.0.0
        fw.mgmt 216.1.146.0
        fw.mgmt.api 1.10.1
        fw.ncsi 864.0.44.0
        fw.roce 216.1.16.0

Fixes: 9599e036 ("bnxt_en: Add support for devlink info command")
Signed-off-by: default avatarVasundhara Volam <vasundhara-v.volam@broadcom.com>
Signed-off-by: default avatarMichael Chan <michael.chan@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 56d69c78
...@@ -66,15 +66,15 @@ The ``bnxt_en`` driver reports the following versions ...@@ -66,15 +66,15 @@ The ``bnxt_en`` driver reports the following versions
* - ``fw`` * - ``fw``
- stored, running - stored, running
- Overall board firmware version - Overall board firmware version
* - ``fw.app`` * - ``fw.mgmt``
- stored, running - stored, running
- Data path firmware version - NIC hardware resource management firmware version
* - ``fw.mgmt.api`` * - ``fw.mgmt.api``
- running - running
- Minimum firmware interface spec version supported between driver and firmware - Minimum firmware interface spec version supported between driver and firmware
* - ``fw.mgmt`` * - ``fw.nsci``
- stored, running - stored, running
- Management firmware version - General platform management firmware version
* - ``fw.roce`` * - ``fw.roce``
- stored, running - stored, running
- RoCE management firmware version - RoCE management firmware version
...@@ -479,7 +479,7 @@ static int bnxt_dl_info_get(struct devlink *dl, struct devlink_info_req *req, ...@@ -479,7 +479,7 @@ static int bnxt_dl_info_get(struct devlink *dl, struct devlink_info_req *req,
ver_resp->roce_fw_bld_8b, ver_resp->roce_fw_rsvd_8b); ver_resp->roce_fw_bld_8b, ver_resp->roce_fw_rsvd_8b);
} }
rc = devlink_info_version_running_put(req, rc = devlink_info_version_running_put(req,
DEVLINK_INFO_VERSION_GENERIC_FW_APP, fw_ver); DEVLINK_INFO_VERSION_GENERIC_FW_MGMT, fw_ver);
if (rc) if (rc)
return rc; return rc;
...@@ -491,7 +491,7 @@ static int bnxt_dl_info_get(struct devlink *dl, struct devlink_info_req *req, ...@@ -491,7 +491,7 @@ static int bnxt_dl_info_get(struct devlink *dl, struct devlink_info_req *req,
if (!(bp->flags & BNXT_FLAG_CHIP_P5)) { if (!(bp->flags & BNXT_FLAG_CHIP_P5)) {
rc = devlink_info_version_running_put(req, rc = devlink_info_version_running_put(req,
DEVLINK_INFO_VERSION_GENERIC_FW_MGMT, mgmt_ver); DEVLINK_INFO_VERSION_GENERIC_FW_NCSI, mgmt_ver);
if (rc) if (rc)
return rc; return rc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment