No need to query 'up' and 'auto' when they are not going to be used.

Signed-off-by: Oswald Buddenhagen <oswald.buddenha...@gmx.de>
---
 target/linux/lantiq/base-files/sbin/dsl_notify.sh | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/target/linux/lantiq/base-files/sbin/dsl_notify.sh 
b/target/linux/lantiq/base-files/sbin/dsl_notify.sh
index ba197f1..11ada92 100755
--- a/target/linux/lantiq/base-files/sbin/dsl_notify.sh
+++ b/target/linux/lantiq/base-files/sbin/dsl_notify.sh
@@ -31,22 +31,24 @@ interfaces=`ubus list network.interface.\* | cut -d"." -f3`
 for ifc in $interfaces; do
 
        json_load "$(ifstatus $ifc)"
-       json_get_var up up
-
-       config_get_bool auto "$ifc" auto 1
 
        json_get_var proto proto
+       if [ "$proto" != "pppoa" ]; then
+               continue
+       fi
 
+       json_get_var up up
+       config_get_bool auto "$ifc" auto 1
        if [ "$DSL_INTERFACE_STATUS" = "UP" ]; then
-               if [ "$proto" = "pppoa" ] && [ "$up" != 1 ] && [ "$auto" = 1 ]; 
then
+               if [ "$up" != 1 ] && [ "$auto" = 1 ]; then
                        ( sleep 1; ifup "$ifc" ) &
                fi
        else
-               if [ "$proto" = "pppoa" ] && [ "$up" = 1 ] && [ "$auto" = 1 ]; 
then
+               if [ "$up" = 1 ] && [ "$auto" = 1 ]; then
                        ( sleep 1; ifdown "$ifc" ) &
                else
                        json_get_var autostart autostart
-                       if [ "$proto" = "pppoa" ] && [ "$up" != 1 ] && [ 
"$autostart" = 1 ]; then
+                       if [ "$up" != 1 ] && [ "$autostart" = 1 ]; then
                                ( sleep 1; ifdown "$ifc" ) &
                        fi
                fi
-- 
2.8.3.1.g1cc7b6a


_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev

Reply via email to