Improved battery status reporting for units which support usb power / charge

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@8886 a1c6a512-1295-4272-9138-f99709370657

2 files changed, 10 insertions, 7 deletions

diff --git a/firmware/drivers/power.c b/firmware/drivers/power.c
index b62121298d..c6d49f5aa9 100644
--- a/firmware/drivers/power.c
+++ b/firmware/drivers/power.c
@@ -120,8 +120,7 @@ bool charger_inserted(void)
     return adc_read(ADC_EXT_POWER) > 0x100;
 #elif defined (HAVE_FMADC)
     /* FM or V2, can also charge from the USB port */
-    return (adc_read(ADC_CHARGE_REGULATOR) < 0x1FF) ||
-        (adc_read(ADC_USB_POWER) < 0x1FF);
+    return (adc_read(ADC_CHARGE_REGULATOR) < 0x1FF);
 #elif defined(TOSHIBA_GIGABEAT_F)
     return false;
 #else
diff --git a/firmware/powermgmt.c b/firmware/powermgmt.c
index b61aed7366..ff635486c1 100644
--- a/firmware/powermgmt.c
+++ b/firmware/powermgmt.c
@@ -353,7 +353,7 @@ static void battery_status_update(void)
     /* calculate estimated remaining running time */
     /* discharging: remaining running time */
     /* charging:    remaining charging time */
-#ifdef HAVE_CHARGE_CTRL
+#if defined(HAVE_CHARGE_CTRL) || defined(HAVE_CHARGE_STATE)
     if (charge_state == CHARGING) {
         powermgmt_est_runningtime_min = (100 - level) * battery_capacity / 100
                                       * 60 / (CURRENT_MAX_CHG - runcurrent());
@@ -497,7 +497,11 @@ static void power_thread_sleep(int ticks)
          * loop (including the subroutines), and end up back here where we
          * transition to the appropriate steady state charger on/off state.
          */
-        if(charger_inserted()) {
+        if(charger_inserted()
+#ifdef HAVE_USB_POWER
+                || usb_powered()
+#endif
+                ) {
             switch(charger_input_state) {
                 case NO_CHARGER:
                 case CHARGER_UNPLUGGED:
@@ -572,6 +576,9 @@ static void power_thread_sleep(int ticks)
              */
             battery_centivolts = avgbat / BATT_AVE_SAMPLES / 10000;
 
+            /* update battery status every time an update is available */
+            battery_status_update();
+
         }
 
 #if defined(DEBUG_FILE) && defined(HAVE_CHARGE_CTRL)
@@ -628,9 +635,6 @@ static void power_thread(void)
         /* insert new value at the start, in centivolts 8-) */
         power_history[0] = battery_centivolts;
 
-        /* update battery status every minute */
-        battery_status_update();
-
 #if CONFIG_BATTERY == BATT_LIION2200
         /* We use the information from the ADC_EXT_POWER ADC channel, which
            tells us the charging current from the LTC1734. When DC is