Re: linux-next: Tree for Aug 31 (nixge.c and phy)

5 messages, 3 authors, 2018-08-31 · open the first message on its own page

Re: linux-next: Tree for Aug 31 (nixge.c and phy)

From: Randy Dunlap <hidden>
Date: 2018-08-31 15:57:34

On 08/30/2018 08:03 PM, Stephen Rothwell wrote:
Hi all,

Changes since 20180830:
on i386:

WARNING: unmet direct dependencies detected for OF_MDIO
  Depends on [n]: OF [=n] && PHYLIB [=y]
  Selected by [y]:
  - NI_XGE_MANAGEMENT_ENET [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_NI [=y] && HAS_IOMEM [=y] && HAS_DMA [=y]

WARNING: unmet direct dependencies detected for OF_MDIO
  Depends on [n]: OF [=n] && PHYLIB [=y]
  Selected by [y]:
  - NI_XGE_MANAGEMENT_ENET [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_NI [=y] && HAS_IOMEM [=y] && HAS_DMA [=y]

drivers/net/phy/mdio-gpio.o: In function `mdio_gpio_probe':
mdio-gpio.c:(.text+0x294): undefined reference to `of_mdiobus_register'
drivers/net/phy/mdio-hisi-femac.o: In function `hisi_femac_mdio_probe':
mdio-hisi-femac.c:(.text+0x352): undefined reference to `of_mdiobus_register'
drivers/net/ethernet/ni/nixge.o: In function `nixge_probe':
nixge.c:(.text+0x1357): undefined reference to `of_mdiobus_register'
drivers/net/ethernet/ni/nixge.o: In function `nixge_open':
nixge.c:(.text+0x21e9): undefined reference to `of_phy_connect'
drivers/net/usb/lan78xx.o: In function `lan78xx_probe':
lan78xx.c:(.text+0x5e29): undefined reference to `of_mdiobus_register'
/home/rdunlap/lnx/next/linux-next-20180831/Makefile:1035: recipe for target 'vmlinux' failed


Full randconfig file is attached.


-- 
~Randy

Re: linux-next: Tree for Aug 31 (nixge.c and phy)

From: Moritz Fischer <hidden>
Date: 2018-08-31 16:45:23

Randy, Dave,

On Fri, Aug 31, 2018 at 8:57 AM Randy Dunlap [off-list ref] wrote:
On 08/30/2018 08:03 PM, Stephen Rothwell wrote:
quoted
Hi all,

Changes since 20180830:
on i386:

WARNING: unmet direct dependencies detected for OF_MDIO
  Depends on [n]: OF [=n] && PHYLIB [=y]
  Selected by [y]:
  - NI_XGE_MANAGEMENT_ENET [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_NI [=y] && HAS_IOMEM [=y] && HAS_DMA [=y]

WARNING: unmet direct dependencies detected for OF_MDIO
  Depends on [n]: OF [=n] && PHYLIB [=y]
  Selected by [y]:
  - NI_XGE_MANAGEMENT_ENET [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_NI [=y] && HAS_IOMEM [=y] && HAS_DMA [=y]

drivers/net/phy/mdio-gpio.o: In function `mdio_gpio_probe':
mdio-gpio.c:(.text+0x294): undefined reference to `of_mdiobus_register'
drivers/net/phy/mdio-hisi-femac.o: In function `hisi_femac_mdio_probe':
mdio-hisi-femac.c:(.text+0x352): undefined reference to `of_mdiobus_register'
drivers/net/ethernet/ni/nixge.o: In function `nixge_probe':
nixge.c:(.text+0x1357): undefined reference to `of_mdiobus_register'
drivers/net/ethernet/ni/nixge.o: In function `nixge_open':
nixge.c:(.text+0x21e9): undefined reference to `of_phy_connect'
drivers/net/usb/lan78xx.o: In function `lan78xx_probe':
lan78xx.c:(.text+0x5e29): undefined reference to `of_mdiobus_register'
/home/rdunlap/lnx/next/linux-next-20180831/Makefile:1035: recipe for target 'vmlinux' failed


Full randconfig file is attached.


--
~Randy
Would:
diff --git a/drivers/net/ethernet/ni/Kconfig b/drivers/net/ethernet/ni/Kconfig
index 04e315704f71..83b6922767af 100644
--- a/drivers/net/ethernet/ni/Kconfig
+++ b/drivers/net/ethernet/ni/Kconfig
@@ -20,7 +20,6 @@ config NI_XGE_MANAGEMENT_ENET
        tristate "National Instruments XGE management enet support"
        depends on HAS_IOMEM && HAS_DMA
        select PHYLIB
-       select OF_MDIO
        help
          Simple LAN device for debug or management purposes. Can
          support either 10G or 1G PHYs via SFP+ ports.

be the right solution? I did a quick build-test with the configuration
above and it seems to fix the issue.
Do I submit a follow-up patch to Dave?

Thanks,

Moritz

Re: linux-next: Tree for Aug 31 (nixge.c and phy)

From: Andrew Lunn <andrew@lunn.ch>
Date: 2018-08-31 17:54:42

quoted hunk
@@ -20,7 +20,6 @@ config NI_XGE_MANAGEMENT_ENET
        tristate "National Instruments XGE management enet support"
        depends on HAS_IOMEM && HAS_DMA
        select PHYLIB
-       select OF_MDIO
        help
          Simple LAN device for debug or management purposes. Can
          support either 10G or 1G PHYs via SFP+ ports.
Hi Moritz

Since you are targeting both DT and non-DT systems, you cannot
unconditionally select OF_MDIO

What might work is

         select OF_MDIO if OF

	 Andrew

Re: linux-next: Tree for Aug 31 (nixge.c and phy)

From: Moritz Fischer <hidden>
Date: 2018-08-31 18:01:42

Andrew,

On Fri, Aug 31, 2018 at 10:54 AM Andrew Lunn [off-list ref] wrote:
quoted
@@ -20,7 +20,6 @@ config NI_XGE_MANAGEMENT_ENET
        tristate "National Instruments XGE management enet support"
        depends on HAS_IOMEM && HAS_DMA
        select PHYLIB
-       select OF_MDIO
        help
          Simple LAN device for debug or management purposes. Can
          support either 10G or 1G PHYs via SFP+ ports.
Hi Moritz

Since you are targeting both DT and non-DT systems, you cannot
unconditionally select OF_MDIO

What might work is

         select OF_MDIO if OF
Sounds good. What's the process here? Follow-up patch to net-next?

Thanks again,

Moritz

Re: linux-next: Tree for Aug 31 (nixge.c and phy)

From: Andrew Lunn <andrew@lunn.ch>
Date: 2018-08-31 18:19:37

Sounds good. What's the process here? Follow-up patch to net-next?
Hi Moritz

Yes. Please include a Fixes: tag, so we have some traceability.

     Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help