Hi,
On Wed, Feb 22, 2017 at 09:58:42PM +0100, Philipp Tomsich wrote:
This changeset moves two configuration options into Kconfig:
- PHY_GIGE, indicates that a controller (with an appropriate PHY) is Gigabit capable and enables extra support in the miiutil for parsing the status of Gigabit PHYs
- individual configuration options for MICREL_KSZ9021 and MICREL_KSZ9031 GbE PHYs, which previously had to enabled through a board-specific config file
Signed-off-by: Philipp Tomsich philipp.tomsich@theobroma-systems.com
drivers/net/Kconfig | 7 +++++++ drivers/net/phy/Kconfig | 10 ++++++++++ 2 files changed, 17 insertions(+)
diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig index 078d5a8..845df09 100644 --- a/drivers/net/Kconfig +++ b/drivers/net/Kconfig @@ -22,6 +22,12 @@ menuconfig NETDEVICES
if NETDEVICES
+config PHY_GIGE
bool
The indentation is wrong here (spaces instead of tabs), and it would be great if you could add a label, so that we can select in in menuconfig.
- help
Enables support for parsing the status output andconfiguring GbE PHYs (e.g. via miiphyutil.c)config AG7XXX bool "Atheros AG7xxx Ethernet MAC support" depends on DM_ETH && ARCH_ATH79 @@ -165,6 +171,7 @@ config SUN8I_EMAC bool "Allwinner Sun8i Ethernet MAC support" depends on DM_ETH select PHYLIB
- select PHY_GIGE help This driver supports the Allwinner based SUN8I/SUN50I Ethernet MAC. It can be found in H3/A64/A83T based SoCs and compatible with both
diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig index 1d514e9..0941383 100644 --- a/drivers/net/phy/Kconfig +++ b/drivers/net/phy/Kconfig @@ -45,6 +45,16 @@ config PHY_MARVELL config PHY_MICREL bool "Micrel Ethernet PHYs support"
+if PHY_MICREL
+config PHY_MICREL_KSZ9021
bool "Micrel KSZ9021 family support"+config PHY_MICREL_KSZ9031
bool "Micrel KSZ9031 family support"
There's the same indentation issues here, and having some help notice would be great.
Thanks! Maxime