i40e: Fix parameters in aq_get_phy_register()
authorGrzegorz Siwik <grzegorz.siwik@intel.com>
Wed, 24 Mar 2021 08:58:27 +0000 (09:58 +0100)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Thu, 8 Apr 2021 17:21:37 +0000 (10:21 -0700)
Change parameters order in aq_get_phy_register() due to wrong
statistics in PHY reported by ethtool. Previously all PHY statistics were
exactly the same for all interfaces
Now statistics are reported correctly - different for different interfaces

Fixes: 0514db37dd78 ("i40e: Extend PHY access with page change flag")
Signed-off-by: Grzegorz Siwik <grzegorz.siwik@intel.com>
Tested-by: Dave Switzer <david.switzer@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/i40e/i40e_ethtool.c

index 96d5202a73e880c83fa876f35abfbf74e6a69635..0e92668012e36879a3391419699dd8e860d6cd44 100644 (file)
@@ -5480,7 +5480,7 @@ static int i40e_get_module_eeprom(struct net_device *netdev,
 
                status = i40e_aq_get_phy_register(hw,
                                I40E_AQ_PHY_REG_ACCESS_EXTERNAL_MODULE,
-                               true, addr, offset, &value, NULL);
+                               addr, true, offset, &value, NULL);
                if (status)
                        return -EIO;
                data[i] = value;