Add snippets for setting up IPv6 and dual IPv4/6 for Wi-Fi

[jukkar]

There is wifi-ipv4 snippet already, so add IPv6 only snippet and also dual IPv4/IPv6 stack snippet.

[tejlmand]

Couple of small observations, rest is looking fine.

Small thought, would it be better to have two snippets specifically for ipv4 and ipv6, and then users can use them like this ?
west build -S wifi-ipv4
west build -S wifi-ipv6
west build -S wifi-ipv4 -S wifi-ipv6

Note, current approach is also approved 🙂

[tejlmand]

should this title include IPv4 ?

[tejlmand]

Comment looks wrong.
This file enables ipv6, not ipv4.

[tejlmand]

Comment needs an update.

[tejlmand]

Small thought, would it be better to have two snippets specifically for ipv4 and ipv6, and then users can use them like this ?

Nevermind, just noticed there's already a https://github.com/zephyrproject-rtos/zephyr/tree/main/snippets/wifi-ipv4 snippet.

Perhaps we could just have the wifi-ipv6 snippet and user combines them ?
Note: one for each combo like now is also ok.

[jukkar]

Perhaps we could just have the wifi-ipv6 snippet and user combines them ?

This would have been my preferred option too, but unfortunately this is not really possible as for example the wifi-ipv4 snippet disables IPv6, so combining them is not really practical. So because of this I created the dual IPv4/6 snippet.

[jukkar]

• Updated according to comments

[tejlmand]

as for example the wifi-ipv4 snippet disables IPv6,

yes I noticed that, but I would like to question if that is correct.

I could have an existing app which uses IPv6, and on top want to use the IPv4 enabling snippet.
In such case I would not expect an IPv4 enabling snippet to disable IPv6, because I never requested that to happen.

[tejlmand]

Approved.

But I do think it's slightly wrong that an IPv6 snippet has opinion regarding IPv4 settings, such as disabling IPv4.
After all, the snippet is named wifi-ipv6 and not wifi-ipv6-only.
(and same goes for the existing IPv4 snippet).