[PATCH] net: Add SYS_FAULT_MII_ADDR to Kconfig

[Jerome Forissier]

Hi Tom,

On 8/12/25 19:56, Tom Rini wrote:
> The support found under SYS_FAULT_ECHO_LINK_DOWN requires that the
> SYS_FAULT_MII_ADDR symbol also be set. This wasn't previously found in
> Kconfig, so add it now.
> 
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> This was found as part of testing "allyesconfig". I'm not sure if
> there's anything useful that could be put in the "help" text here.
> ---
>  README      | 3 ---
>  net/Kconfig | 4 ++++
>  2 files changed, 4 insertions(+), 3 deletions(-)>
> diff --git a/README b/README
> index 65305e6a0371..7808af528625 100644
> --- a/README
> +++ b/README
> @@ -1249,9 +1249,6 @@ Note: once the monitor has been relocated, then it will complain if
>  the default environment is used; a new CRC is computed as soon as you
>  use the "saveenv" command to store a valid environment.
>  
> -- CONFIG_SYS_FAULT_MII_ADDR:
> -		MII address of the PHY to check for the Ethernet link state.
> -
>  - CONFIG_DISPLAY_BOARDINFO
>  		Display information about the board that U-Boot is running on
>  		when U-Boot starts up. The board function checkboard() is called
> diff --git a/net/Kconfig b/net/Kconfig
> index 24508026200d..6d5de9a675e4 100644
> --- a/net/Kconfig
> +++ b/net/Kconfig
> @@ -60,6 +60,10 @@ config SYS_FAULT_ECHO_LINK_DOWN
>  	  this option is active, then CONFIG_SYS_FAULT_MII_ADDR also needs to
>  	  be configured.
>  
> +config SYS_FAULT_MII_ADDR
> +	hex "MII address of the PHY to check for the Ethernet link state"
> +	depends on SYS_FAULT_ECHO_LINK_DOWN
> +
>  config TFTP_PORT
>  	bool "Set TFTP UDP source/destination ports via the environment"
>  	help

This breaks qemu_m68k in CI [1].

[1] https://source.denx.de/u-boot/custodians/u-boot-net/-/jobs/1224987

More specifically:

$ tools/buildman/buildman -o /tmp/build -w -E -W -e --board M5208EVBE
Building current source for 1 boards (1 thread, 24 jobs per thread)
      m68k:  +   M5208EVBE
+.config:598:warning: symbol value '' invalid for SYS_FAULT_MII_ADDR
+
+Error in reading or end of file.
+make[3]: *** [scripts/kconfig/Makefile:75: syncconfig] Terminated
+make[2]: *** [Makefile:647: syncconfig] Terminated
+make: *** [Makefile:198: sub-make] Terminated
+(** did you define an int/hex Kconfig with no default? **)
    0    0    1 /1              M5208EVBE
Completed: 1 total built, 1 newly), duration 0:00:04, rate 0.25

Cheers,
-- 
Jerome