mvneta: add FIXED_PHY dependency
authorArnd Bergmann <arnd@arndb.de>
Mon, 9 Nov 2015 14:08:57 +0000 (15:08 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 9 Nov 2015 16:35:03 +0000 (11:35 -0500)
The fixed_phy infrastructure is done in a way that is optional,
by providing 'static inline' helper functions doing nothing in
include/linux/phy_fixed.h for all its APIs. However, three out
of the four users (DSA, BCMGENET, and SYSTEMPORT) always
'select FIXED_PHY', presumably because they need that.
MVNETA is the fourth one, and if that is built-in but FIXED_PHY
is configured as a loadable module, we get a link error:

drivers/built-in.o: In function `mvneta_fixed_link_update':
fpga-mgr.c:(.text+0x33ed80): undefined reference to `fixed_phy_update_state'

Presumably this driver has the same dependency as the others,
so this patch also uses 'select' to ensure that the fixed-phy
support is built-in.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 898b2970e2c9 ("mvneta: implement SGMII-based in-band link state signaling")
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/Kconfig

index 80af9ffce5ead1bd62e61499ff2e0ff9f6e04732..a1c862b4664de02667f7d4c751f1d9a7e8686ee6 100644 (file)
@@ -44,6 +44,7 @@ config MVNETA
        tristate "Marvell Armada 370/38x/XP network interface support"
        depends on PLAT_ORION
        select MVMDIO
+       select FIXED_PHY
        ---help---
          This driver supports the network interface units in the
          Marvell ARMADA XP, ARMADA 370 and ARMADA 38x SoC family.