Revert "luci-proto-ppp: pppoe.so has no MTU option; remove it"

This reverts commit 995d3a1801cf3b850e8c7c35937fbdd0526ff53e.

The `mtu` option controls the pppd mru and mtu arguments, it is not
specific to pppoe.so.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>

diff --git a/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js b/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
index b025a3eeb2f67f27e963c1c3b32a8aa74caa811b..057e17fe81b1906c1e0c2a3d8b5cc460fbde03b0 100644 (file)
--- a/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
+++ b/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
@@ -93,6 +93,8 @@ return network.registerProtocol('pppoe', {
                o.placeholder = '0';
                o.datatype    = 'uinteger';
 
-               /* pppoe.so has no MTU option - only underlying device MTU is possible */
+               o = s.taboption('advanced', form.Value, 'mtu', _('Override MTU'));
+               o.placeholder = dev ? (dev.getMTU() || '1500') : '1500';
+               o.datatype    = 'max(9200)';
        }
 });