<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Dec 3, 2014 at 11:23 AM, Steven Barth <span dir="ltr"><<a href="mailto:cyrus@openwrt.org" target="_blank">cyrus@openwrt.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 03.12.2014 11:15, Hans Dedecker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Config support to force the IGMP host version on device level; possible values are:<br>
     1|igmpv1: IGMP version 1<br>
     2|igmpv2: IGMP version 2<br>
     3|igmpv3: IGMP version 3<br>
</blockquote>
<br>
<br></span>
Thanks Hans.<br>
<br>
However, I don't really see the point of the string logic, it seems a bit bloated and the simple integer 1 2 or 3 should suffice imo.<br>
<br>
Also I'm wondering if this should set MLD version too (i.e. 1 if IGMP is 2 and 2 if IGMP is 3).<br>
<br>
Cheers,<br>
<br>
Steven</blockquote><div>I will simplify the configuration format and restrict it to the numerical format as requested by you and Felix.</div><div>Regarding the MLD version wouldn't it be preferable to have this as a different UCI parameter as I could imagine different versions in use for MLD and IGMP. Quite a lot of ISP's are still stuck to IGMPv2 in their network while they're using MLDv2 for IPV6.</div><div><br></div><div>Hans </div></div></div></div>