[LEDE-DEV] uqmi: Set data format after clearing previous autoconnect state.

Message ID 1483096336-31290-1-git-send-email-nledovskikh@gmail.com
State New
Delegated to: Felix Fietkau
Headers show

Commit Message

Nickolay Ledovskikh Dec. 30, 2016, 11:12 a.m.
Clearing previous autoconnect state when using Sierra Airprime MC7304
drops data format to 'raw-ip' (MC7710 is not behave the same). This
leads to dhcp freeze. So data format must be set after stopping
previous network.

Signed-off-by: Nickolay Ledovskikh <nledovskikh@gmail.com>
---
 package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Nickolay Ledovskikh Jan. 27, 2017, 7:48 a.m. | #1
Hi. Any interest?

2016-12-30 14:12 GMT+03:00 Nickolay Ledovskikh <nledovskikh@gmail.com>:
> Clearing previous autoconnect state when using Sierra Airprime MC7304
> drops data format to 'raw-ip' (MC7710 is not behave the same). This
> leads to dhcp freeze. So data format must be set after stopping
> previous network.
>
> Signed-off-by: Nickolay Ledovskikh <nledovskikh@gmail.com>
> ---
>  package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh b/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
> index 35de6c5..47475f6 100755
> --- a/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
> +++ b/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
> @@ -96,8 +96,6 @@ proto_qmi_setup() {
>                 }
>         }
>
> -       uqmi -s -d "$device" --set-data-format 802.3
> -       uqmi -s -d "$device" --wda-set-data-format 802.3
>         uqmi -s -d "$device" --sync
>
>         echo "Waiting for network registration"
> @@ -134,6 +132,9 @@ proto_qmi_setup() {
>                         --stop-network 0xffffffff \
>                         --autoconnect > /dev/null
>
> +               uqmi -s -d "$device" --set-data-format 802.3
> +               uqmi -s -d "$device" --wda-set-data-format 802.3
> +
>                 pdh_4=`uqmi -s -d "$device" --set-client-id wds,"$cid_4" \
>                         --start-network \
>                         ${apn:+--apn $apn} \
> @@ -165,6 +166,9 @@ proto_qmi_setup() {
>                         --stop-network 0xffffffff \
>                         --autoconnect > /dev/null
>
> +               uqmi -s -d "$device" --set-data-format 802.3
> +               uqmi -s -d "$device" --wda-set-data-format 802.3
> +
>                 pdh_6=`uqmi -s -d "$device" --set-client-id wds,"$cid_6" \
>                         --start-network \
>                         ${apn:+--apn $apn} \
> --
> 2.7.3
>

Patch

diff --git a/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh b/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
index 35de6c5..47475f6 100755
--- a/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
+++ b/package/network/utils/uqmi/files/lib/netifd/proto/qmi.sh
@@ -96,8 +96,6 @@  proto_qmi_setup() {
 		}
 	}
 
-	uqmi -s -d "$device" --set-data-format 802.3
-	uqmi -s -d "$device" --wda-set-data-format 802.3
 	uqmi -s -d "$device" --sync
 
 	echo "Waiting for network registration"
@@ -134,6 +132,9 @@  proto_qmi_setup() {
 			--stop-network 0xffffffff \
 			--autoconnect > /dev/null
 
+		uqmi -s -d "$device" --set-data-format 802.3
+		uqmi -s -d "$device" --wda-set-data-format 802.3
+
 		pdh_4=`uqmi -s -d "$device" --set-client-id wds,"$cid_4" \
 			--start-network \
 			${apn:+--apn $apn} \
@@ -165,6 +166,9 @@  proto_qmi_setup() {
 			--stop-network 0xffffffff \
 			--autoconnect > /dev/null
 
+		uqmi -s -d "$device" --set-data-format 802.3
+		uqmi -s -d "$device" --wda-set-data-format 802.3
+
 		pdh_6=`uqmi -s -d "$device" --set-client-id wds,"$cid_6" \
 			--start-network \
 			${apn:+--apn $apn} \