Code review comment for lp:~awe/phablet-extras/ofono-lp1188404

Tony Espy (awe) wrote :

The following patch will force the status to ROAMING, instead of REGISTERED:

=== modified file 'drivers/rilmodem/gprs.c'
--- drivers/rilmodem/gprs.c 2013-07-10 23:08:34 +0000
+++ drivers/rilmodem/gprs.c 2013-07-13 00:50:38 +0000
@@ -166,6 +166,12 @@
   ofono_gprs_set_cid_range(gprs, 1, max_cids);
  }

+ if (status == NETWORK_REGISTRATION_STATUS_REGISTERED) {
+ DBG("converting status to ROAMING!");
+ status = NETWORK_REGISTRATION_STATUS_ROAMING;
+ }
+
+ /* Do we need to save either???*/
  gd->status = status;
  gd->tech = tech;

=== modified file 'drivers/rilmodem/network-registration.c'
--- drivers/rilmodem/network-registration.c 2013-06-14 16:11:01 +0000
+++ drivers/rilmodem/network-registration.c 2013-07-13 00:50:20 +0000
@@ -104,6 +104,11 @@
   return;
  }

+ if (status == NETWORK_REGISTRATION_STATUS_REGISTERED) {
+ DBG("converting status to ROAMING!");
+ status = NETWORK_REGISTRATION_STATUS_ROAMING;
+ }
+
  nd->tech = tech;
  cb(&error, status, lac, ci, tech, cbd->data);
 }
@@ -118,6 +123,11 @@
                 return;
         }

+ if (status == NETWORK_REGISTRATION_STATUS_REGISTERED) {
+ DBG("converting status to ROAMING!");
+ status = NETWORK_REGISTRATION_STATUS_ROAMING;
+ }
+
  ofono_netreg_status_notify(netreg, status, lac, ci, tech);
 }

« Back to merge proposal