bnx2x: downgrade Max BW error message to debug

There are valid configurations where Max BW is configured to zero for
some VNs.
Print the message only if debugging is enabled and do not call the
configuration "illegal".

[v2: use DP(), not BNX2X_DBG_ERR(); recommended by Eilon Greenstein.]

(backported from upstream commit 96b0accb8867627250e911f8929e6c01da1ffd40)
Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
Acked-by: Eilon Greenstein <eilong@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Joe Jin <joe.jin@oracle.com>

diff --git a/drivers/net/bnx2x/bnx2x_cmn.h b/drivers/net/bnx2x/bnx2x_cmn.h
index 8ef9fd1bcc29914d9742e38357cc2017e8a47b8a..4cc912e16b65e83ad760ad07322e7fa7f38f12e6 100644 (file)
--- a/drivers/net/bnx2x/bnx2x_cmn.h
+++ b/drivers/net/bnx2x/bnx2x_cmn.h
@@ -1485,8 +1485,8 @@ static inline u16 bnx2x_extract_max_cfg(struct bnx2x *bp, int vn)
                                           FUNC_MF_CFG_MAX_BW_SHIFT;
 
        if (!max_cfg && !bp->prev_max_cfg_invalid[vn])
-               BNX2X_ERR("Illegal configuration detected for Max BW "
-                         "on vn %d - using 100 instead\n", vn);
+               DP(NETIF_MSG_LINK,
+                  "Max BW configured to 0 - using 100 instead\n");
        bp->prev_max_cfg_invalid[vn] = !max_cfg;
 
        return max_cfg ?: 100;