Move the implementation of the dns command under cmd/.

Signed-off-by: Jerome Forissier <[email protected]>
---

Changes in v2:
- New patch

 cmd/lwip/dns.c    | 123 ++++++++++++++++++++++++++++++++++++++++++++
 net/lwip/Makefile |   1 -
 net/lwip/dns.c    | 128 ----------------------------------------------
 3 files changed, 123 insertions(+), 129 deletions(-)
 delete mode 100644 net/lwip/dns.c

diff --git a/cmd/lwip/dns.c b/cmd/lwip/dns.c
index 31642b864af..be57a4d1b88 100644
--- a/cmd/lwip/dns.c
+++ b/cmd/lwip/dns.c
@@ -2,7 +2,130 @@
 /* Copyright (C) 2024-2025 Linaro Ltd. */
 
 #include <command.h>
+#include <console.h>
+#include <lwip/dns.h>
+#include <lwip/timeouts.h>
 #include <net.h>
+#include <time.h>
 
 U_BOOT_CMD(dns, 3, 1, do_dns, "lookup the IP of a hostname",
           "hostname [envvar]");
+
+#define DNS_RESEND_MS 1000
+#define DNS_TIMEOUT_MS 10000
+
+struct dns_cb_arg {
+       ip_addr_t host_ipaddr;
+       const char *var;
+       bool done;
+};
+
+static void do_dns_tmr(void *arg)
+{
+       dns_tmr();
+}
+
+static void dns_cb(const char *name, const ip_addr_t *ipaddr, void *arg)
+{
+       struct dns_cb_arg *dns_cb_arg = arg;
+       char *ipstr = ip4addr_ntoa(ipaddr);
+
+       dns_cb_arg->done = true;
+
+       if (!ipaddr) {
+               printf("DNS: host not found\n");
+               dns_cb_arg->host_ipaddr.addr = 0;
+               return;
+       }
+
+       if (dns_cb_arg->var)
+               env_set(dns_cb_arg->var, ipstr);
+
+       printf("%s\n", ipstr);
+}
+
+static int dns_loop(struct udevice *udev, const char *name, const char *var)
+{
+       struct dns_cb_arg dns_cb_arg = { };
+       bool has_server = false;
+       struct netif *netif;
+       ip_addr_t ipaddr;
+       ip_addr_t ns;
+       ulong start;
+       char *nsenv;
+       int ret;
+
+       dns_cb_arg.var = var;
+
+       netif = net_lwip_new_netif(udev);
+       if (!netif)
+               return CMD_RET_FAILURE;
+
+       dns_init();
+
+       nsenv = env_get("dnsip");
+       if (nsenv && ipaddr_aton(nsenv, &ns)) {
+               dns_setserver(0, &ns);
+               has_server = true;
+       }
+
+       nsenv = env_get("dnsip2");
+       if (nsenv && ipaddr_aton(nsenv, &ns)) {
+               dns_setserver(1, &ns);
+               has_server = true;
+       }
+
+       if (!has_server) {
+               log_err("No valid name server (dnsip/dnsip2)\n");
+               net_lwip_remove_netif(netif);
+               return CMD_RET_FAILURE;
+       }
+
+       dns_cb_arg.done = false;
+
+       ret = dns_gethostbyname(name, &ipaddr, dns_cb, &dns_cb_arg);
+
+       if (ret == ERR_OK) {
+               dns_cb(name, &ipaddr, &dns_cb_arg);
+       } else if (ret == ERR_INPROGRESS) {
+               start = get_timer(0);
+               sys_timeout(DNS_RESEND_MS, do_dns_tmr, NULL);
+               do {
+                       net_lwip_rx(udev, netif);
+                       if (dns_cb_arg.done)
+                               break;
+                       sys_check_timeouts();
+                       if (ctrlc()) {
+                               printf("\nAbort\n");
+                               break;
+                       }
+               } while (get_timer(start) < DNS_TIMEOUT_MS);
+               sys_untimeout(do_dns_tmr, NULL);
+       }
+
+       net_lwip_remove_netif(netif);
+
+       if (dns_cb_arg.done && dns_cb_arg.host_ipaddr.addr != 0)
+               return CMD_RET_SUCCESS;
+
+       return CMD_RET_FAILURE;
+}
+
+int do_dns(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
+{
+       char *name;
+       char *var = NULL;
+
+       if (argc == 1 || argc > 3)
+               return CMD_RET_USAGE;
+
+       name = argv[1];
+
+       if (argc == 3)
+               var = argv[2];
+
+       if (net_lwip_eth_start() < 0)
+               return CMD_RET_FAILURE;
+
+       return dns_loop(eth_get_dev(), name, var);
+}
diff --git a/net/lwip/Makefile b/net/lwip/Makefile
index 5df222589b8..255c7d018b0 100644
--- a/net/lwip/Makefile
+++ b/net/lwip/Makefile
@@ -2,7 +2,6 @@ ccflags-y += -I$(srctree)/lib/lwip/lwip/src/include 
-I$(srctree)/lib/lwip/u-boot
 
 obj-$(CONFIG_$(PHASE_)DM_ETH) += net-lwip.o
 obj-$(CONFIG_CMD_DHCP) += dhcp.o
-obj-$(CONFIG_CMD_DNS) += dns.o
 obj-$(CONFIG_CMD_PING) += ping.o
 obj-$(CONFIG_CMD_TFTPBOOT) += tftp.o
 obj-$(CONFIG_WGET) += wget.o
diff --git a/net/lwip/dns.c b/net/lwip/dns.c
deleted file mode 100644
index 19172ac959a..00000000000
--- a/net/lwip/dns.c
+++ /dev/null
@@ -1,128 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/* Copyright (C) 2024 Linaro Ltd. */
-
-#include <command.h>
-#include <console.h>
-#include <lwip/dns.h>
-#include <lwip/timeouts.h>
-#include <net.h>
-#include <time.h>
-
-#define DNS_RESEND_MS 1000
-#define DNS_TIMEOUT_MS 10000
-
-struct dns_cb_arg {
-       ip_addr_t host_ipaddr;
-       const char *var;
-       bool done;
-};
-
-static void do_dns_tmr(void *arg)
-{
-       dns_tmr();
-}
-
-static void dns_cb(const char *name, const ip_addr_t *ipaddr, void *arg)
-{
-       struct dns_cb_arg *dns_cb_arg = arg;
-       char *ipstr = ip4addr_ntoa(ipaddr);
-
-       dns_cb_arg->done = true;
-
-       if (!ipaddr) {
-               printf("DNS: host not found\n");
-               dns_cb_arg->host_ipaddr.addr = 0;
-               return;
-       }
-
-       if (dns_cb_arg->var)
-               env_set(dns_cb_arg->var, ipstr);
-
-       printf("%s\n", ipstr);
-}
-
-static int dns_loop(struct udevice *udev, const char *name, const char *var)
-{
-       struct dns_cb_arg dns_cb_arg = { };
-       bool has_server = false;
-       struct netif *netif;
-       ip_addr_t ipaddr;
-       ip_addr_t ns;
-       ulong start;
-       char *nsenv;
-       int ret;
-
-       dns_cb_arg.var = var;
-
-       netif = net_lwip_new_netif(udev);
-       if (!netif)
-               return CMD_RET_FAILURE;
-
-       dns_init();
-
-       nsenv = env_get("dnsip");
-       if (nsenv && ipaddr_aton(nsenv, &ns)) {
-               dns_setserver(0, &ns);
-               has_server = true;
-       }
-
-       nsenv = env_get("dnsip2");
-       if (nsenv && ipaddr_aton(nsenv, &ns)) {
-               dns_setserver(1, &ns);
-               has_server = true;
-       }
-
-       if (!has_server) {
-               log_err("No valid name server (dnsip/dnsip2)\n");
-               net_lwip_remove_netif(netif);
-               return CMD_RET_FAILURE;
-       }
-
-       dns_cb_arg.done = false;
-
-       ret = dns_gethostbyname(name, &ipaddr, dns_cb, &dns_cb_arg);
-
-       if (ret == ERR_OK) {
-               dns_cb(name, &ipaddr, &dns_cb_arg);
-       } else if (ret == ERR_INPROGRESS) {
-               start = get_timer(0);
-               sys_timeout(DNS_RESEND_MS, do_dns_tmr, NULL);
-               do {
-                       net_lwip_rx(udev, netif);
-                       if (dns_cb_arg.done)
-                               break;
-                       sys_check_timeouts();
-                       if (ctrlc()) {
-                               printf("\nAbort\n");
-                               break;
-                       }
-               } while (get_timer(start) < DNS_TIMEOUT_MS);
-               sys_untimeout(do_dns_tmr, NULL);
-       }
-
-       net_lwip_remove_netif(netif);
-
-       if (dns_cb_arg.done && dns_cb_arg.host_ipaddr.addr != 0)
-               return CMD_RET_SUCCESS;
-
-       return CMD_RET_FAILURE;
-}
-
-int do_dns(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
-{
-       char *name;
-       char *var = NULL;
-
-       if (argc == 1 || argc > 3)
-               return CMD_RET_USAGE;
-
-       name = argv[1];
-
-       if (argc == 3)
-               var = argv[2];
-
-       if (net_lwip_eth_start() < 0)
-               return CMD_RET_FAILURE;
-
-       return dns_loop(eth_get_dev(), name, var);
-}
-- 
2.43.0

Reply via email to