[dpdk-stable] patch 'net/bnxt: remove excess log messages' has been queued to stable release 18.08.1

[Kevin Traynor]

Hi,

FYI, your patch has been queued to stable release 18.08.1

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/27/18. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the patch applied
to the branch. If the code is different (ie: not only metadata diffs), due for example to
a change in context or macro names, please double check it.

Thanks.

Kevin Traynor

---
>From e7a4eaeaa21af8d0631eb3d2627ff0cb79dd1dcd Mon Sep 17 00:00:00 2001
From: Ajit Khaparde <ajit.khaparde at broadcom.com>
Date: Fri, 28 Sep 2018 19:00:05 -0700
Subject: [PATCH] net/bnxt: remove excess log messages

[ upstream commit 6f93b2aa2da8a1b0d1c5149e88b6428ac00296a5 ]

When the firmware version and the driver HWRM version do not match,
we are logging some messages. These messages unnecessarily clutter
the logs and can add to the noise. We are logging the HWRM version
and the firmware version anyway. The difference in version numbers
can be gleaned from that. Removing the remaining log messages.

Fixes: 804e746c7b73 ("net/bnxt: add hardware resource manager init code")

Signed-off-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
---
 drivers/net/bnxt/bnxt_hwrm.c | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index 05217ccdb..982d2c2bb 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -785,5 +785,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
 	struct hwrm_ver_get_input req = {.req_type = 0 };
 	struct hwrm_ver_get_output *resp = bp->hwrm_cmd_resp_addr;
-	uint32_t my_version;
 	uint32_t fw_version;
 	uint16_t max_resp_len;
@@ -813,8 +812,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
 		HWRM_VERSION_MAJOR, HWRM_VERSION_MINOR, HWRM_VERSION_UPDATE);
 
-	my_version = HWRM_VERSION_MAJOR << 16;
-	my_version |= HWRM_VERSION_MINOR << 8;
-	my_version |= HWRM_VERSION_UPDATE;
-
 	fw_version = resp->hwrm_intf_maj_8b << 16;
 	fw_version |= resp->hwrm_intf_min_8b << 8;
@@ -828,19 +823,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
 	}
 
-	if (my_version != fw_version) {
-		PMD_DRV_LOG(INFO, "BNXT Driver/HWRM API mismatch.\n");
-		if (my_version < fw_version) {
-			PMD_DRV_LOG(INFO,
-				"Firmware API version is newer than driver.\n");
-			PMD_DRV_LOG(INFO,
-				"The driver may be missing features.\n");
-		} else {
-			PMD_DRV_LOG(INFO,
-				"Firmware API version is older than driver.\n");
-			PMD_DRV_LOG(INFO,
-				"Not all driver features may be functional.\n");
-		}
-	}
-
 	if (bp->max_req_len > resp->max_req_win_len) {
 		PMD_DRV_LOG(ERR, "Unsupported request length\n");
-- 
2.19.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2018-11-21 16:44:32.048668057 +0000
+++ 0040-net-bnxt-remove-excess-log-messages.patch	2018-11-21 16:44:30.000000000 +0000
@@ -1,8 +1,10 @@
-From 6f93b2aa2da8a1b0d1c5149e88b6428ac00296a5 Mon Sep 17 00:00:00 2001
+From e7a4eaeaa21af8d0631eb3d2627ff0cb79dd1dcd Mon Sep 17 00:00:00 2001
 From: Ajit Khaparde <ajit.khaparde at broadcom.com>
 Date: Fri, 28 Sep 2018 19:00:05 -0700
 Subject: [PATCH] net/bnxt: remove excess log messages
 
+[ upstream commit 6f93b2aa2da8a1b0d1c5149e88b6428ac00296a5 ]
+
 When the firmware version and the driver HWRM version do not match,
 we are logging some messages. These messages unnecessarily clutter
 the logs and can add to the noise. We are logging the HWRM version
@@ -10,7 +12,6 @@
 can be gleaned from that. Removing the remaining log messages.
 
 Fixes: 804e746c7b73 ("net/bnxt: add hardware resource manager init code")
-Cc: stable at dpdk.org
 
 Signed-off-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
 ---
@@ -18,16 +19,16 @@
  1 file changed, 20 deletions(-)
 
 diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
-index 38698e214..76e443ec1 100644
+index 05217ccdb..982d2c2bb 100644
 --- a/drivers/net/bnxt/bnxt_hwrm.c
 +++ b/drivers/net/bnxt/bnxt_hwrm.c
-@@ -790,5 +790,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
+@@ -785,5 +785,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
  	struct hwrm_ver_get_input req = {.req_type = 0 };
  	struct hwrm_ver_get_output *resp = bp->hwrm_cmd_resp_addr;
 -	uint32_t my_version;
  	uint32_t fw_version;
  	uint16_t max_resp_len;
-@@ -818,8 +817,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
+@@ -813,8 +812,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
  		HWRM_VERSION_MAJOR, HWRM_VERSION_MINOR, HWRM_VERSION_UPDATE);
  
 -	my_version = HWRM_VERSION_MAJOR << 16;
@@ -36,7 +37,7 @@
 -
  	fw_version = resp->hwrm_intf_maj_8b << 16;
  	fw_version |= resp->hwrm_intf_min_8b << 8;
-@@ -833,19 +828,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
+@@ -828,19 +823,4 @@ int bnxt_hwrm_ver_get(struct bnxt *bp)
  	}
  
 -	if (my_version != fw_version) {