From e28379253e1d33024957181cb9c7401217989ccb Mon Sep 17 00:00:00 2001
From: Juha Heinanen <jh@tutpro.com>
Date: Mon, 13 Apr 2026 10:40:41 +0300
Subject: [PATCH] Removed brief wakelock acquire

---
 .../com/tutpro/baresip/BaresipService.kt      | 36 +++++++------------
 1 file changed, 12 insertions(+), 24 deletions(-)

diff --git a/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt b/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt
index 00897ba1..d3c26a28 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt
@@ -115,8 +115,6 @@ class BaresipService: Service() {
     private var isNotificationInCall = false
     private var isServiceClean = false
 
-    private val TAG = "BaresipService"
-
     @SuppressLint("WakelockTimeout")
     override fun onCreate() {
         super.onCreate()
@@ -748,7 +746,7 @@ class BaresipService: Service() {
             if (uas.value[accountIndex].account.aor == aor) {
                 when (ev[0]) {
                     "registering", "unregistering" -> {
-                        updatePartialWakeLock(brief = true)
+                        updatePartialWakeLock()
                         ua.updateStatus(circleYellow.getValue(colorblind))
                         updateStatusNotification()
                         if (isMainVisible)
@@ -756,7 +754,7 @@ class BaresipService: Service() {
                         return
                     }
                     "registered" -> {
-                        updatePartialWakeLock(brief = true)
+                        updatePartialWakeLock()
                         ua.updateStatus(
                             if (Api.account_regint(ua.account.accp) == 0)
                                 R.drawable.circle_white
@@ -769,7 +767,7 @@ class BaresipService: Service() {
                         return
                     }
                     "registering failed" -> {
-                        updatePartialWakeLock(brief = true)
+                        updatePartialWakeLock()
                         ua.updateStatus(if (Api.account_regint(ua.account.accp) == 0)
                             R.drawable.circle_white
                         else
@@ -1642,25 +1640,13 @@ class BaresipService: Service() {
     }
 
     @SuppressLint("WakelockTimeout")
-    private fun updatePartialWakeLock(brief: Boolean = false) {
-        // Only hold a permanent wake lock during active calls.
-        // For registration standby, rely on the foreground service and kernel socket wakes.
-        // Also hold it briefly for registration events or network transitions.
-        val needsToStayAwake = calls.isNotEmpty()
+    private fun updatePartialWakeLock() {
+        // Hold the wake lock as long as the service is active to ensure
+        // native SIP timers (registration, keep-alives) continue to run.
         try {
-            if (needsToStayAwake) {
-                if (!partialWakeLock.isHeld) {
-                    Log.d(TAG, "Acquiring Partial Wake Lock (Call in progress)")
-                    partialWakeLock.acquire()
-                }
-            } else if (brief) {
-                Log.d(TAG, "Acquiring Brief Partial Wake Lock (2s)")
-                partialWakeLock.acquire(2000L)
-            } else {
-                if (partialWakeLock.isHeld) {
-                    Log.d(TAG, "Releasing Partial Wake Lock (No calls in progress)")
-                    partialWakeLock.release()
-                }
+            if (!partialWakeLock.isHeld) {
+                Log.d(TAG, "Acquiring Partial Wake Lock")
+                partialWakeLock.acquire()
             }
         } catch (e: Exception) {
             Log.e(TAG, "Error managing partialWakeLock: ${e.message}")
@@ -1963,7 +1949,7 @@ class BaresipService: Service() {
     private fun updateNetwork() {
         if (!isNativeReady) return
 
-        updatePartialWakeLock(brief = true)
+        updatePartialWakeLock()
         updateDnsServers()
 
         val addresses = linkAddresses()
@@ -2160,6 +2146,8 @@ class BaresipService: Service() {
     @SuppressLint("MutableCollectionMutableState")
     companion object {
 
+        private const val TAG = "BaresipService"
+
         var instance: BaresipService? = null
         var isServiceRunning = false
         var isNativeReady = false