From 987ea5b7c27d1ab3de2f83f99b2da721ff5744fd Mon Sep 17 00:00:00 2001
From: Jussi Kivilinna <jussi.kivilinna@haltian.com>
Date: Fri, 25 Sep 2015 15:43:42 +0300
Subject: [PATCH] ubgps: fix power-management threshold comparisons

Signed-off-by: Jussi Kivilinna <jussi.kivilinna@haltian.com>
---
 apps/system/ubgps/ubgps.c       | 2 +-
 apps/system/ubgps/ubgps_state.c | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/apps/system/ubgps/ubgps.c b/apps/system/ubgps/ubgps.c
index 675d9d34..6491d446 100644
--- a/apps/system/ubgps/ubgps.c
+++ b/apps/system/ubgps/ubgps.c
@@ -464,7 +464,7 @@ int ubgps_config(gps_config_t const config, void const * const value)
               gps->state.current_state == GPS_STATE_FIX_ACQUIRED)
             {
 #ifdef CONFIG_UBGPS_PSM_MODE
-              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD)
+              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD * 1000)
                 {
                   /* Use default navigation rate for SW controlled PSM */
 
diff --git a/apps/system/ubgps/ubgps_state.c b/apps/system/ubgps/ubgps_state.c
index 1a95b01e..6b87662c 100644
--- a/apps/system/ubgps/ubgps_state.c
+++ b/apps/system/ubgps/ubgps_state.c
@@ -450,7 +450,7 @@ static int ubgps_sm_global(struct ubgps_s * const gps, struct sm_event_s const *
 
               /* Construct power save mode event */
 
-              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD)
+              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD * 1000)
                 {
                   struct sm_event_psm_event_s psm;
 
@@ -577,12 +577,12 @@ static int ubgps_sm_global(struct ubgps_s * const gps, struct sm_event_s const *
             {
               /* Start SW controlled power save mode (PSM). */
 
-              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD)
+              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD * 1000)
                 {
                   struct timespec ts = {};
 
                   clock_gettime(CLOCK_MONOTONIC, &ts);
-                  ts.tv_sec += gps->state.navigation_rate/1000;
+                  ts.tv_sec += gps->state.navigation_rate / 1000;
                   gps->state.psm_timer_id = ts_core_timer_setup_date(&ts,
                       ubgps_psm_timer_cb, gps);
 
@@ -999,7 +999,7 @@ static int ubgps_init_process_phase(struct ubgps_s * const gps, bool next)
               /* Set GPS navigation rate */
 
 #ifdef CONFIG_UBGPS_PSM_MODE
-              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD*1000)
+              if (gps->state.navigation_rate >= CONFIG_UBGPS_PSM_MODE_THRESHOLD * 1000)
                 {
                   /* Use default navigation rate for SW controlled PSM */
 
-- 
GitLab