Revert "system-linux: re-apply ethtool on phy attachment"
authorDaniel Golle <daniel@makrotopia.org>
Thu, 12 Sep 2024 11:43:52 +0000 (13:43 +0200)
committerDaniel Golle <daniel@makrotopia.org>
Thu, 12 Sep 2024 13:33:26 +0000 (15:33 +0200)
This reverts commit 68c8a4f94cd3cfd654a52cbc8b57c5c9d99640dd.

Matching on flags IFF_UP AND NOT IFF_LOWER_UP and then reapply ethtool
settings freaks out some PHY drivers. While a better method is being
worked on, revert the problematic solution.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>
system-linux.c

index 58c737337be3e38001018aa3301cd87750fa2011..4463a2a8282a1151cb99734d65dd57689a7fad52 100644 (file)
@@ -701,9 +701,6 @@ static int system_get_arp_accept(struct device *dev, char *buf, const size_t buf
 #define IFF_LOWER_UP   0x10000
 #endif
 
-static void
-system_set_ethtool_settings(struct device *dev, struct device_settings *s);
-
 static void
 system_device_update_state(struct device *dev, unsigned int flags, unsigned int ifindex)
 {
@@ -714,9 +711,6 @@ system_device_update_state(struct device *dev, unsigned int flags, unsigned int
                device_set_present(dev, ifindex > 0);
        }
        device_set_link(dev, flags & IFF_LOWER_UP ? true : false);
-
-       if ((flags & IFF_UP) && !(flags & IFF_LOWER_UP))
-               system_set_ethtool_settings(dev, &dev->settings);
 }
 
 /* Evaluate netlink messages */