Please pull net-next-20250708

[E Shattow]

On 7/11/25 20:12, E Shattow wrote:
> 
> 
> On 7/8/25 02:36, Jerome Forissier wrote:
>> Hi Tom,
>>
>> The following changes since commit b379335f141e3ddca0cdbb5d9d39af0444530542:
>>
>>   configs: Resync with savedefconfig (2025-07-07 14:15:18 -0600)
>>
>> are available in the Git repository at:
>>
>>   https://source.denx.de/u-boot/custodians/u-boot-net.git/ tags/net-next-20250708
>>
>> for you to fetch changes up to 203be3197de1016236f55652ce8ee4cbfe3035ad:
>>
>>   configs: qemu_arm64_lwip_defconfig: enable CMD_SNTP (2025-07-08 11:07:37 +0200)
>>
>> ----------------------------------------------------------------
>> Pull request net-next-20250708
>>
>> lwip:
>> - Call sys_check_timeouts() and schedule() on RX to fix an issue on
>>   boards with a watchdog and simplify the code
>> - Enable IP_FRAG and IP_REASSEMBLY
>> - Add support for setting the TFTP blocksize at runtime
>> - Fix DNS initialization in wget
>> - Add the sntp command
>> - Move code from net/lwip/${cmd}.c to cmd/lwip/${cmd}.c
>>
>> ----------------------------------------------------------------
>> Jerome Forissier (13):
>>       lwip: remove net/lwip/eth_internal.h
>>       lwip: move net/lwip/dns.c to cmd/lwip
>>       lwip: move net/lwip/ping.c to cmd/lwip
>>       lwip: split cmd/net-lwip.c into one file per command
>>       lwip: split net/lwip/wget.c
>>       lwip: dns: do not return CMD_RET_FAILURE on successful resolution
>>       lwip: dns: do not print IP address when a variable is specified
>>       lwip: add net_lwip_dns_resolve()
>>       lwip: ping: accept host name when CONFIG_CMD_DNS=y
>>       net: extract function net_sntp_set_rtc() from sntp_handler()
>>       lwip: add sntp command
>>       doc: add doc/cmd/sntp.rst and remove doc/README.SNTP
>>       configs: qemu_arm64_lwip_defconfig: enable CMD_SNTP
>>
>> Tim Harvey (5):
>>       net: lwip: call sys_check_timeouts and schedule on rx
>>       net: lwip: enable IP_FRAG and IP_REASSEMBLY
>>       net: lwip: tftp: add support for setting blocksize at runtime
>>       net: lwip: move dns init to common function
>>       net: lwip: wget: initialize dns if a hostname is used in a URL
>>
>>  MAINTAINERS                       |   1 +
>>  cmd/Kconfig                       |  13 +++++----
>>  cmd/Makefile                      |   3 +-
>>  cmd/lwip/Makefile                 |   6 ++++
>>  cmd/lwip/dhcp.c                   |   9 ++++++
>>  {net => cmd}/lwip/dns.c           |  33 +++++++--------------
>>  {net => cmd}/lwip/ping.c          |   6 ++--
>>  cmd/lwip/sntp.c                   | 134 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  cmd/lwip/tftp.c                   |   9 ++++++
>>  cmd/lwip/wget.c                   | 222 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  cmd/net-lwip.c                    |  50 --------------------------------
>>  configs/qemu_arm64_lwip_defconfig |   1 +
>>  doc/README.SNTP                   |  17 -----------
>>  doc/usage/cmd/sntp.rst            |  72 ++++++++++++++++++++++++++++++++++++++++++++++
>>  doc/usage/index.rst               |   1 +
>>  include/net-common.h              |  13 +++++++++
>>  include/net-lwip.h                |  16 +++++++++++
>>  lib/lwip/Makefile                 |   1 +
>>  lib/lwip/u-boot/arch/cc.h         |   4 +++
>>  lib/lwip/u-boot/lwipopts.h        |   8 ++++--
>>  net/lwip/Makefile                 |   2 --
>>  net/lwip/dhcp.c                   |   1 -
>>  net/lwip/eth_internal.h           |  35 -----------------------
>>  net/lwip/net-lwip.c               |  81 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  net/lwip/tftp.c                   |   8 ++++--
>>  net/lwip/wget.c                   | 207 ++++-------------------------------------------------------------------------------------------------------------------------------
>>  net/net-common.c                  |  23 +++++++++++++++
>>  net/sntp.c                        |  23 ++-------------
>>  28 files changed, 637 insertions(+), 362 deletions(-)
>>  create mode 100644 cmd/lwip/Makefile
>>  create mode 100644 cmd/lwip/dhcp.c
>>  rename {net => cmd}/lwip/dns.c (80%)
>>  rename {net => cmd}/lwip/ping.c (95%)
>>  create mode 100644 cmd/lwip/sntp.c
>>  create mode 100644 cmd/lwip/tftp.c
>>  create mode 100644 cmd/lwip/wget.c
>>  delete mode 100644 cmd/net-lwip.c
>>  delete mode 100644 doc/README.SNTP
>>  create mode 100644 doc/usage/cmd/sntp.rst
>>  delete mode 100644 net/lwip/eth_internal.h
>>
>> CI link: https://source.denx.de/u-boot/custodians/u-boot-net/-/pipelines/26989
>>
>> Thanks,
> 
> I think this broke something with DNS? Build from 6bb0679377ab with
> starfive_visionfive2_defconfig menuconfig says PROT_DNS_LWIP [=y] but on
> following use of dhcp command then wget use says that DNS is disabled (!?)
> 
> Adding CONFIG_CMD_DNS to the config and all works as expected again. Is
> support for DNS now tied to having the dns command or is that a bug?
> 
> Best regards,
> 
> - E Shattow

P.S. What I describe actually goes back to commit
a383869d6b58b3d66a91460a8b1a6fdcc09a1fe9 ; I'll send a patch.