From: Grazvydas Ignotas <notasas@gmail.com>
Date: Thu, 4 Aug 2011 22:24:26 +0000 (+0300)
Subject: pndevmapperd: don't do charge control too often
X-Git-Tag: sz_beta3~41
X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;ds=sidebyside;h=471022009f9f9ce5d41a5034aaeb48c4264c3e20;p=pandora-libraries.git

pndevmapperd: don't do charge control too often

when led blinking is going on sigalrm_handler is called quite often,
and we don't want to bother kernel with this that much.
---

diff --git a/apps/pndevmapperd.c b/apps/pndevmapperd.c
index 09293a1..b5301f1 100644
--- a/apps/pndevmapperd.c
+++ b/apps/pndevmapperd.c
@@ -869,8 +869,11 @@ void sigalrm_handler ( int n ) {
 
   pnd_log ( pndn_debug, "---[ SIGALRM ]---\n" );
 
+  static time_t last_charge_check;
   int batlevel = pnd_device_get_battery_gauge_perc();
   int uamps = 0;
+  time_t now;
+
   pnd_device_get_charge_current ( &uamps );
 
   if ( batlevel < 0 ) {
@@ -916,7 +919,8 @@ void sigalrm_handler ( int n ) {
   }
 
   // charge monitoring
-  if ( bc_enable && bc_charge_device != NULL ) {
+  now = time(NULL);
+  if ( bc_enable && bc_charge_device != NULL && (unsigned int)(now - last_charge_check) > 60 ) {
 
     int charge_enabled = pnd_device_get_charger_enable ( bc_charge_device );
     if ( charge_enabled < 0 )
@@ -932,6 +936,7 @@ void sigalrm_handler ( int n ) {
         pnd_device_set_charger_enable ( bc_charge_device, 1 );
       }
     }
+    last_charge_check = now;
   }
 
   // is battery warning already active?