~aleteoryx/muditaos: [EGD-8121] Fix cellular sleep mode enter without SIM

4a1e7e14be76eda1e6a1af90358c84151a68b2c5 — Maciej Gibowicz 4 years ago e9ee2af

[EGD-8121] Fix cellular sleep mode enter without SIM

Without a SIM card, the modem will be able to enter sleep mode,
which will significantly extend the battery life of the phone.

patch browse .tar.gz

1 files changed, 2 insertions(+), 2 deletions(-)

M module-services/service-cellular/ServiceCellular.cpp

M module-services/service-cellular/ServiceCellular.cpp => module-services/service-cellular/ServiceCellular.cpp +2 -2

@@ 202,7 202,7 @@ void ServiceCellular::SleepTimerHandler()
                                           ? currentTime - lastCommunicationTimestamp
                                           : std::numeric_limits<TickType_t>::max() - lastCommunicationTimestamp + currentTime;
 
-    if (!ongoingCall.isValid() && priv->state->get() == State::ST::Ready &&
+    if (!ongoingCall.isValid() && priv->state->get() >= State::ST::URCReady &&
         timeOfInactivity >= constants::enterSleepModeTime.count()) {
         cmux->enterSleepMode();
         cpuSentinel->ReleaseMinimumFrequency();


@@ 1258,6 1258,7 @@ bool ServiceCellular::handle_URCReady()
     ret = ret && channel->cmd(at::AT::ENABLE_NETWORK_REGISTRATION_URC);
 
     bus.sendMulticast<cellular::msg::notification::ModemStateChanged>(cellular::api::ModemState::Ready);
+    sleepTimer.start();
 
     LOG_DEBUG("%s", priv->state->c_str());
     return ret;


@@ 1370,7 1371,6 @@ bool ServiceCellular::handle_ready()
 {
 
     LOG_DEBUG("%s", priv->state->c_str());
-    sleepTimer.start();
     return true;
 }