ixgbe: cleanup X540 PHY reset function pointer
authorDon Skidmore <donald.c.skidmore@intel.com>
Fri, 18 Feb 2011 19:29:46 +0000 (19:29 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Thu, 3 Mar 2011 11:21:15 +0000 (03:21 -0800)
The X540 PHY reset pointer isn't currently used which is a good thing as it
wouldn't work as implemented.  On top of that the X540 firmware is written
with the assumption that is does not need to be reset for proper
initialization so it's not needed.  I'm just assigning the pointer at NULL
as the current implementation is rather misleading.

Signed-off-by: Don Skidmore <donald.c.skidmore@intel.com>
Tested-by: Stephen Ko <stephen.s.ko@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ixgbe/ixgbe_x540.c

index a6f06d59a64a2dbec689ba4a5587c7b7205d3bf9..3229398630f6a34fc231d4c3b877b81daf6d9f3a 100644 (file)
@@ -712,7 +712,7 @@ static struct ixgbe_phy_operations phy_ops_X540 = {
        .identify               = &ixgbe_identify_phy_generic,
        .identify_sfp           = &ixgbe_identify_sfp_module_generic,
        .init                   = NULL,
-       .reset                  = &ixgbe_reset_phy_generic,
+       .reset                  = NULL,
        .read_reg               = &ixgbe_read_phy_reg_generic,
        .write_reg              = &ixgbe_write_phy_reg_generic,
        .setup_link             = &ixgbe_setup_phy_link_generic,