Fixed state machine to accept roaming as valid registration.

Teppo Järvelin · Teppo Järvelin · commit dcdb7a5da21f · 2018-04-11T09:38:13.000+03:00
diff --git a/features/cellular/easy_cellular/CellularConnectionFSM.cpp b/features/cellular/easy_cellular/CellularConnectionFSM.cpp
@@ -570,9 +570,12 @@ void CellularConnectionFSM::network_callback(nsapi_event_t ev, intptr_t ptr)
 {
 
     tr_info("FSM: network_callback called with event: %d, intptr: %d", ev, ptr);
-    if ((cellular_connection_status_t)ev == CellularRegistrationStatusChanged && ptr == CellularNetwork::RegisteredHomeNetwork && _state == STATE_REGISTERING_NETWORK) {
-        _queue.cancel(_eventID);
-        continue_from_state(STATE_ATTACHING_NETWORK);
+    if ((cellular_connection_status_t)ev == CellularRegistrationStatusChanged && _state == STATE_REGISTERING_NETWORK) {
+        // expect packet data so only these states are valid
+        if (ptr == CellularNetwork::RegisteredHomeNetwork && CellularNetwork::RegisteredRoaming) {
+            _queue.cancel(_eventID);
+            continue_from_state(STATE_ATTACHING_NETWORK);
+        }
     }
 
     if (_event_status_cb) {

Original file line number	Diff line number	Diff line change
`@@ -570,9 +570,12 @@ void CellularConnectionFSM::network_callback(nsapi_event_t ev, intptr_t ptr)`
`570`	`570`	`{`
`571`	`571`
`572`	`572`	`tr_info("FSM: network_callback called with event: %d, intptr: %d", ev, ptr);`
`573`		`- if ((cellular_connection_status_t)ev == CellularRegistrationStatusChanged && ptr == CellularNetwork::RegisteredHomeNetwork && _state == STATE_REGISTERING_NETWORK) {`
`574`		`- _queue.cancel(_eventID);`
`575`		`- continue_from_state(STATE_ATTACHING_NETWORK);`
	`573`	`+ if ((cellular_connection_status_t)ev == CellularRegistrationStatusChanged && _state == STATE_REGISTERING_NETWORK) {`
	`574`	`+ // expect packet data so only these states are valid`
	`575`	`+ if (ptr == CellularNetwork::RegisteredHomeNetwork && CellularNetwork::RegisteredRoaming) {`
	`576`	`+ _queue.cancel(_eventID);`
	`577`	`+ continue_from_state(STATE_ATTACHING_NETWORK);`
	`578`	`+ }`
`576`	`579`	`}`
`577`	`580`
`578`	`581`	`if (_event_status_cb) {`