Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES
@ 2024-08-02 10:44 Robert Marko
  2024-08-02 16:35 ` Thomas Petazzoni via buildroot
  2024-09-03 16:18 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Robert Marko @ 2024-08-02 10:44 UTC (permalink / raw)
  To: buildroot; +Cc: Robert Marko

mdio-tools depends on CONFIG_MDIO_DEVICE in order for mdiobus driver to
be built, but CONFIG_MDIO_DEVICE depends on CONFIG_NETDEVICES which we
are not enabling so on platforms without it enabled in kernel config
building mdio-tools will fail with:

ERROR: modpost: "mdio_find_bus" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
ERROR: modpost: "__mdiobus_c45_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
ERROR: modpost: "__mdiobus_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
ERROR: modpost: "__mdiobus_c45_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
ERROR: modpost: "__mdiobus_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!

So enable CONFIG_NETDEVICES as well to make sure CONFIG_MDIO_DEVICE can be enabled.

Fixes: http://autobuild.buildroot.net/results/edf47df96cde6094c890c0b74034cced90335a39/

Signed-off-by: Robert Marko <robimarko@gmail.com>
---
 package/mdio-tools/mdio-tools.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/mdio-tools/mdio-tools.mk b/package/mdio-tools/mdio-tools.mk
index 16eba74734..b42ea41c79 100644
--- a/package/mdio-tools/mdio-tools.mk
+++ b/package/mdio-tools/mdio-tools.mk
@@ -14,6 +14,7 @@ MDIO_TOOLS_MODULE_SUBDIRS = kernel
 
 define MDIO_TOOLS_LINUX_CONFIG_FIXUPS
 	$(call KCONFIG_ENABLE_OPT,CONFIG_NET)
+	$(call KCONFIG_ENABLE_OPT,CONFIG_NETDEVICES)
 	$(call KCONFIG_ENABLE_OPT,CONFIG_MDIO_DEVICE)
 endef
 
-- 
2.45.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES
  2024-08-02 10:44 [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES Robert Marko
@ 2024-08-02 16:35 ` Thomas Petazzoni via buildroot
  2024-09-03 16:18 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2024-08-02 16:35 UTC (permalink / raw)
  To: Robert Marko; +Cc: buildroot

On Fri,  2 Aug 2024 12:44:37 +0200
Robert Marko <robimarko@gmail.com> wrote:

> mdio-tools depends on CONFIG_MDIO_DEVICE in order for mdiobus driver to
> be built, but CONFIG_MDIO_DEVICE depends on CONFIG_NETDEVICES which we
> are not enabling so on platforms without it enabled in kernel config
> building mdio-tools will fail with:
> 
> ERROR: modpost: "mdio_find_bus" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
> ERROR: modpost: "__mdiobus_c45_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
> ERROR: modpost: "__mdiobus_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
> ERROR: modpost: "__mdiobus_c45_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
> ERROR: modpost: "__mdiobus_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
> 
> So enable CONFIG_NETDEVICES as well to make sure CONFIG_MDIO_DEVICE can be enabled.
> 
> Fixes: http://autobuild.buildroot.net/results/edf47df96cde6094c890c0b74034cced90335a39/
> 
> Signed-off-by: Robert Marko <robimarko@gmail.com>
> ---
>  package/mdio-tools/mdio-tools.mk | 1 +
>  1 file changed, 1 insertion(+)

Applied to master, thanks a lot for this fix!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES
  2024-08-02 10:44 [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES Robert Marko
  2024-08-02 16:35 ` Thomas Petazzoni via buildroot
@ 2024-09-03 16:18 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2024-09-03 16:18 UTC (permalink / raw)
  To: Robert Marko; +Cc: buildroot

>>>>> "Robert" == Robert Marko <robimarko@gmail.com> writes:

 > mdio-tools depends on CONFIG_MDIO_DEVICE in order for mdiobus driver to
 > be built, but CONFIG_MDIO_DEVICE depends on CONFIG_NETDEVICES which we
 > are not enabling so on platforms without it enabled in kernel config
 > building mdio-tools will fail with:

 > ERROR: modpost: "mdio_find_bus" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
 > ERROR: modpost: "__mdiobus_c45_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
 > ERROR: modpost: "__mdiobus_read" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
 > ERROR: modpost: "__mdiobus_c45_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!
 > ERROR: modpost: "__mdiobus_write" [output-1/build/mdio-tools-1.3.1/kernel/mdio-netlink.ko] undefined!

 > So enable CONFIG_NETDEVICES as well to make sure CONFIG_MDIO_DEVICE can be enabled.

 > Fixes: http://autobuild.buildroot.net/results/edf47df96cde6094c890c0b74034cced90335a39/

 > Signed-off-by: Robert Marko <robimarko@gmail.com>

Committed to 2024.02.x and 2024.05.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-09-03 16:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-02 10:44 [Buildroot] [PATCH 1/1] package/mdio-tools: enable CONFIG_NETDEVICES Robert Marko
2024-08-02 16:35 ` Thomas Petazzoni via buildroot
2024-09-03 16:18 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox