Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1734099 > unrolled thread

[PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig

Started byJerome Brunet <jbrunet@baylibre.com>
First post2017-09-18 15:00 +0200
Last post2017-09-19 23:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig Jerome Brunet <jbrunet@baylibre.com> - 2017-09-18 15:00 +0200
    Re: [PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in  menuconfig David Miller <davem@davemloft.net> - 2017-09-19 23:00 +0200

#1734099 — [PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig

FromJerome Brunet <jbrunet@baylibre.com>
Date2017-09-18 15:00 +0200
Subject[PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig
Message-ID<ur3Oy-6q3-15@gated-at.bofh.it>
Since the integration of PHYLINK, the configuration option which
used to be under the PHY infrastructure menu in menuconfig ended
up one level up (the network device driver section)

By placing PHYLINK option right after PHYLIB entry, it broke the
way Kconfig used to build the menu. See kconfig-language.txt, section
"Menu structure", 2nd method.

This is fixed by placing the PHYLINK option just before PHYLIB.

Fixes: 9525ae83959b ("phylink: add phylink infrastructure")
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
---
 drivers/net/phy/Kconfig | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig
index a9d16a3af514..cd931cf9dcc2 100644
--- a/drivers/net/phy/Kconfig
+++ b/drivers/net/phy/Kconfig
@@ -160,15 +160,6 @@ config MDIO_XGENE
 
 endif
 
-menuconfig PHYLIB
-	tristate "PHY Device support and infrastructure"
-	depends on NETDEVICES
-	select MDIO_DEVICE
-	help
-	  Ethernet controllers are usually attached to PHY
-	  devices.  This option provides infrastructure for
-	  managing PHY devices.
-
 config PHYLINK
 	tristate
 	depends on NETDEVICES
@@ -179,6 +170,15 @@ config PHYLINK
 	  configuration links, PHYs, and Serdes links with MAC level
 	  autonegotiation modes.
 
+menuconfig PHYLIB
+	tristate "PHY Device support and infrastructure"
+	depends on NETDEVICES
+	select MDIO_DEVICE
+	help
+	  Ethernet controllers are usually attached to PHY
+	  devices.  This option provides infrastructure for
+	  managing PHY devices.
+
 if PHYLIB
 
 config SWPHY
-- 
2.13.5

[toc] | [next] | [standalone]


#1735261 — Re: [PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig

FromDavid Miller <davem@davemloft.net>
Date2017-09-19 23:00 +0200
SubjectRe: [PATCH net] net: phy: Kconfig: Fix PHY infrastructure menu in menuconfig
Message-ID<urxMC-1ME-23@gated-at.bofh.it>
In reply to#1734099
From: Jerome Brunet <jbrunet@baylibre.com>
Date: Mon, 18 Sep 2017 14:59:20 +0200

> Since the integration of PHYLINK, the configuration option which
> used to be under the PHY infrastructure menu in menuconfig ended
> up one level up (the network device driver section)
> 
> By placing PHYLINK option right after PHYLIB entry, it broke the
> way Kconfig used to build the menu. See kconfig-language.txt, section
> "Menu structure", 2nd method.
> 
> This is fixed by placing the PHYLINK option just before PHYLIB.
> 
> Fixes: 9525ae83959b ("phylink: add phylink infrastructure")
> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>

Applied, thank you.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web