[OpenWrt-Devel] [PATCH 3/4] ncm: Remove unnecessary proto_set_available commands

[Matti Laakso]

Interface should not be set unavailable in all error cases,
returning 1 is enough.

Signed-off-by: Matti Laakso <malaakso at elisanet.fi>
---
 package/network/utils/comgt/files/ncm.sh | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/package/network/utils/comgt/files/ncm.sh b/package/network/utils/comgt/files/ncm.sh
index 2c2835f..4c3f161 100644
--- a/package/network/utils/comgt/files/ncm.sh
+++ b/package/network/utils/comgt/files/ncm.sh
@@ -41,7 +41,6 @@ proto_ncm_setup() {
 	[ -n "$apn" ] || {
 		echo "No APN specified"
 		proto_notify_error "$interface" NO_APN
-		proto_set_available "$interface" 0
 		return 1
 	}
 
@@ -69,7 +68,6 @@ proto_ncm_setup() {
 	[ $? -ne 0 ] && {
 		echo "Failed to get modem information"
 		proto_notify_error "$interface" GETINFO_FAILED
-		proto_set_available "$interface" 0
 		return 1
 	}
 
@@ -86,7 +84,6 @@ proto_ncm_setup() {
 		eval COMMAND="$i" gcom -d "$device" -s /etc/gcom/runcommand.gcom || {
 			echo "Failed to initialize modem"
 			proto_notify_error "$interface" INITIALIZE_FAILED
-			proto_set_available "$interface" 0
 			return 1
 		}
 	done
@@ -105,7 +102,6 @@ proto_ncm_setup() {
 		COMMAND="$setmode" gcom -d "$device" -s /etc/gcom/runcommand.gcom || {
 			echo "Failed to set operating mode"
 			proto_notify_error "$interface" SETMODE_FAILED
-			proto_set_available "$interface" 0
 			return 1
 		}
 		json_select ..
@@ -115,7 +111,6 @@ proto_ncm_setup() {
 	eval COMMAND="$connect" gcom -d "$device" -s /etc/gcom/runcommand.gcom || {
 		echo "Failed to connect"
 		proto_notify_error "$interface" CONNECT_FAILED
-		proto_set_available "$interface" 0
 		return 1
 	}
 
-- 
2.1.0
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel