Firmware does not allow the operation and would return failure, causing
a warning in dmesg.  So check for VF and disallow it in the driver.

Signed-off-by: Michael Chan <michael.c...@broadcom.com>
---
 drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c 
b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
index 0ea8466..a699ca54 100644
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
@@ -1818,6 +1818,11 @@ static int nvm_get_dir_info(struct net_device *dev, u32 
*entries, u32 *length)
 
 static int bnxt_get_eeprom_len(struct net_device *dev)
 {
+       struct bnxt *bp = netdev_priv(dev);
+
+       if (BNXT_VF(bp))
+               return 0;
+
        /* The -1 return value allows the entire 32-bit range of offsets to be
         * passed via the ethtool command-line utility.
         */
-- 
1.8.3.1

Reply via email to