openthread: platform: trel: Handle enable/disable from CLI

OpenThread interface is initialized beform Wi-Fi interface.
`otPlatTrelEnable` is called by OpenThread stack when it's interface is
being initialized. Given this scenario, socket operation, like `bind`,
will fail. There is also no mean to get a valid pointer to backbone
interface. This is why, `trel_plat_init` was declared and called when
backbone interface reported connectivity.

This commit handles the `ot trel disable/ot trel enable` scenario that
can be initialized from CLI. `otPlatTrelEnable` will be called, but
`trel_plat_init` will not be called anymore, leaving trel socket without
any options set, and `net_socket_service_register` is not called
anymore, to handle incoming traffic.
Now, when `otPlatTrelEnable` is called, it will verify if Wi-Fi
interface is connected and will call `trel_plat_init` if needed.

Signed-off-by: Cristian Bulacu <cristian.bulacu@nxp.com>

Author: Cristib05

modules/openthread/platform/platform-zephyr.h

@@ -129,7 +129,7 @@ otError mdns_plat_socket_init(otInstance *ot_instance, uint32_t ail_iface_idx);
 void mdns_plat_monitor_interface(struct net_if *ail_iface);
 otError border_agent_init(otInstance *instance);
 void border_agent_deinit(void);
-otError trel_plat_init(otInstance *instance, struct net_if *ail_iface_ptr);
+otError trel_plat_init(otInstance *instance, struct net_if *ail_iface);
 otError dhcpv6_pd_client_init(otInstance *ot_instance, uint32_t ail_iface_index);
 otError dns_upstream_resolver_init(otInstance *ot_instance);
 #endif /* CONFIG_OPENTHREAD_ZEPHYR_BORDER_ROUTER */

modules/openthread/platform/trel.c

@@ -14,12 +14,14 @@
 #include <zephyr/net/net_if.h>
 #include <zephyr/net/net_ip.h>
 #include "sockets_internal.h"
+#include <platform-zephyr.h>
 
 #define MAX_SERVICES   1
 
 static struct zsock_pollfd sockfd_udp[MAX_SERVICES];
 static int trel_sock = -1;
 static struct otInstance *ot_instance_ptr;
+static struct net_if *ail_iface_ptr;
 static otPlatTrelCounters trel_counters;
 static bool trel_is_enabled;
 static void trel_receive_handler(struct net_socket_service_event *evt);
@@ -47,6 +49,11 @@ void otPlatTrelEnable(otInstance *aInstance, uint16_t *aUdpPort)
 
 	trel_is_enabled = true;
 
+	if (ail_iface_ptr != NULL && net_if_is_up(ail_iface_ptr)) {
+		(void)trel_plat_init(ot_instance_ptr, ail_iface_ptr);
+	}
+
+
 exit:
 	return;
 
@@ -141,13 +148,14 @@ static void process_trel_message(struct otbr_msg_ctx *msg_ctx_ptr)
 				 &msg_ctx_ptr->sock_addr);
 }
 
-otError trel_plat_init(otInstance *instance, struct net_if *ail_iface_ptr)
+otError trel_plat_init(otInstance *instance, struct net_if *ail_iface)
 {
 	otError error = OT_ERROR_NONE;
 	struct ifreq if_req = {0};
 	char name[CONFIG_NET_INTERFACE_NAME_LEN + 1] = {0};
 
 	ot_instance_ptr = instance;
+	ail_iface_ptr = ail_iface;
 
 	VerifyOrExit(net_if_get_name(ail_iface_ptr, name,
 				     CONFIG_NET_INTERFACE_NAME_LEN) > 0,