umbim: explicitly check for PIN1 state

PIN2 is used only to restrict changing of fixed dialling feature,
does not affect network registration. Therefore explicitly check for
PIN1 state during connection setup, which is required for network
registration.

Signed-off-by: Martin Schiller <[email protected]>
[Updated commit description]
Signed-off-by: Lech Perczak <[email protected]>

diff --git a/package/network/utils/umbim/files/lib/netifd/proto/mbim.sh b/package/network/utils/umbim/files/lib/netifd/proto/mbim.sh
index db716c3dbf0b38283b191c1f480158cd17e4b263..a33759cd6d93092e3724c0e9448181d7900d4ce5 100755 (executable)
--- a/package/network/utils/umbim/files/lib/netifd/proto/mbim.sh
+++ b/package/network/utils/umbim/files/lib/netifd/proto/mbim.sh
@@ -86,7 +86,8 @@ _proto_mbim_setup() {
        tid=$((tid + 1))
 
        echo "mbim[$$]" "Checking pin"
-       umbim $DBG -n -t $tid -d $device pinstate || {
+       umbim $DBG -n -t $tid -d $device pinstate
+       [ $? -eq 2 ] && {
                echo "mbim[$$]" "PIN required"
                tid=$((tid + 1))
                umbim $DBG -t $tid -d "$device" disconnect