Thread (18 messages) 18 messages, 4 authors, 2020-07-02

Re: [PATCH net-next v4 03/10] net: ethtool: netlink: Add support for triggering a cable test

From: Jakub Kicinski <kuba@kernel.org>
Date: 2020-07-01 22:56:24

On Sun, 10 May 2020 21:12:33 +0200 Andrew Lunn wrote:
quoted hunk ↗ jump to hunk
diff --git a/net/Kconfig b/net/Kconfig
index c5ba2d180c43..5c524c6ee75d 100644
--- a/net/Kconfig
+++ b/net/Kconfig
@@ -455,6 +455,7 @@ config FAILOVER
 config ETHTOOL_NETLINK
 	bool "Netlink interface for ethtool"
 	default y
+	depends on PHYLIB=y || PHYLIB=n
 	help
 	  An alternative userspace interface for ethtool based on generic
 	  netlink. It provides better extensibility and some new features,
Since ETHTOOL_NETLINK is a bool we end up not enabling it on
allmodconfig builds, (PHYLIB=m so ETHTOOL_NETLINK dependency 
can't be met) - which is v scary for build testing.

Is there a way we can change this dependency? Some REACHABLE shenanigans?

Or since there are just two callbacks maybe phylib can "tell" ethtool
core the pointers to call when it loads?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help