[riot-notifications] [RIOT-OS/RIOT] sys/net/dhcpv6: include RIO after configuring downstream subnets (#16590)

Martine Lenders notifications at github.com
Thu Jul 8 15:56:35 CEST 2021

@miri64 commented on this pull request.

> +void dhcpv6_client_conf_done(unsigned iface)
+    gnrc_netif_t *netif = gnrc_netif_get_by_pid(iface);
+    /* inform upstream network about the subnet(s) now managed by this router */
+    gnrc_ipv6_nib_change_rtr_adv_rio_iface(netif, true);

I would just move the implementation of `dhcpv6_client_conf_done()` to `client_simple_pd.c`. The name already suggests as much. Maybe add a weak implementation here so `client.c` works without.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210708/82ae3340/attachment.htm>

More information about the notifications mailing list