From a452c7410dcc50cd12ad6d09a639c18b435d7ec9 Mon Sep 17 00:00:00 2001
From: Brandon Low <lostlogic@rockbox.org>
Date: Thu, 19 Jan 2006 19:39:51 +0000
Subject: Start cleaning up some #ifdefs in charging code

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@8391 a1c6a512-1295-4272-9138-f99709370657
---
 apps/debug_menu.c        |  2 ++
 apps/gui/gwps-common.c   | 12 +++---------
 apps/gui/statusbar.c     |  2 --
 firmware/drivers/power.c | 15 ++++-----------
 4 files changed, 9 insertions(+), 22 deletions(-)

diff --git a/apps/debug_menu.c b/apps/debug_menu.c
index 69c0e2fceb..66b4fdbb8d 100644
--- a/apps/debug_menu.c
+++ b/apps/debug_menu.c
@@ -1333,11 +1333,13 @@ static bool view_runtime(void)
 #endif
 
         if (state & 1) {
+#ifdef HAVE_CHARGING
             if (charger_inserted())
             {
                 global_settings.runtime = 0;
             }
             else
+#endif
             {
                 global_settings.runtime += ((current_tick - lasttime) / HZ);
             }
diff --git a/apps/gui/gwps-common.c b/apps/gui/gwps-common.c
index c9d093963e..b9b3e9bcd0 100644
--- a/apps/gui/gwps-common.c
+++ b/apps/gui/gwps-common.c
@@ -653,27 +653,21 @@ static char* get_tag(struct wps_data* wps_data,
                     return buf;
                 }
 
+#ifdef HAVE_CHARGING
                 case 'p': /* External power plugged in? */
                 {
-#ifdef HAVE_CHARGING             
                     if(charger_input_state==CHARGER)
-#else
-                    if(charger_inserted())
-#endif
                         return "p";
                     else
                         return NULL;
                 }
+#endif
 #if defined(HAVE_CHARGE_CTRL) || \
     defined (HAVE_CHARGE_STATE) || \
     CONFIG_BATTERY == BATT_LIION2200
                 case 'c': /* Charging */
                 {
-                    if (charge_state == CHARGING
-#ifdef HAVE_CHARGE_CTRL
-                            || charge_state == TOPOFF
-#endif
-                            ) {
+                    if (charge_state == CHARGING || charge_state == TOPOFF) {
                         return "c";
                     } else {
                         return NULL;
diff --git a/apps/gui/statusbar.c b/apps/gui/statusbar.c
index 57308c52f2..d8e611af85 100644
--- a/apps/gui/statusbar.c
+++ b/apps/gui/statusbar.c
@@ -129,8 +129,6 @@ void gui_statusbar_draw(struct gui_statusbar * bar, bool force_redraw)
     bar->info.volume = sound_val2phys(SOUND_VOLUME, global_settings.volume);
 #ifdef HAVE_CHARGING
     bar->info.inserted = (charger_input_state == CHARGER);
-#else
-    bar->info.inserted = charger_inserted();
 #endif
     bar->info.battlevel = battery_level();
     bar->info.battery_safe = battery_level_safe();
diff --git a/firmware/drivers/power.c b/firmware/drivers/power.c
index 8a20890228..b5d4d7d22a 100644
--- a/firmware/drivers/power.c
+++ b/firmware/drivers/power.c
@@ -104,33 +104,26 @@ void power_init(void)
 #endif
 }
 
+#ifdef HAVE_CHARGING
 bool charger_inserted(void)
 {     
 #if CONFIG_CPU == MCF5249
     return (GPIO1_READ & 0x00400000)?true:false;
 #elif defined(GMINI_ARCH)
     return (P7 & 0x80) == 0;
-#else
-#ifdef HAVE_CHARGING
-#ifdef HAVE_CHARGE_CTRL
+#elif defined(HAVE_CHARGE_CTRL)
     /* Recorder */
     return adc_read(ADC_EXT_POWER) > 0x100;
-#else
-#ifdef HAVE_FMADC
+#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);
 #else
     /* Player */
     return (PADR & 1) == 0;
-#endif /* HAVE_FMADC */
-#endif /* HAVE_CHARGE_CTRL */
-#else
-    /* Ondio */
-    return false;
-#endif /* HAVE_CHARGING */
 #endif
 }
+#endif /* HAVE_CHARGING */
 
 #ifdef HAVE_CHARGE_CTRL
 void charger_enable(bool on)
-- 
cgit v1.2.3