From 6e80ac39a5a3e7091c14db19cac791c9ddb8849f Mon Sep 17 00:00:00 2001
From: Linus Nielsen Feltzing <linus@haxx.se>
Date: Tue, 5 Jul 2005 00:03:18 +0000
Subject: New poweroff handling, using the SYS_POWEROFF event, allowing plugins
 to save their settings before powering off.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@7019 a1c6a512-1295-4272-9138-f99709370657
---
 apps/main_menu.c            |  9 ++++++++-
 apps/screens.c              |  2 +-
 firmware/export/powermgmt.h |  1 +
 firmware/powermgmt.c        | 17 +++++++++++------
 4 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/apps/main_menu.c b/apps/main_menu.c
index 68d6699cd5..87518cff28 100644
--- a/apps/main_menu.c
+++ b/apps/main_menu.c
@@ -361,6 +361,13 @@ bool info_menu(void)
     return result;
 }
 
+#ifdef HAVE_LCD_CHARCELLS
+static void do_shutdown(void)
+{
+    sys_poweroff(false);
+}
+#endif
+    
 bool main_menu(void)
 {
     int m;
@@ -405,7 +412,7 @@ bool main_menu(void)
 
 #ifdef HAVE_LCD_CHARCELLS
     items[i].desc = ID2P(LANG_SHUTDOWN);
-    items[i++].function = clean_shutdown;
+    items[i++].function = do_shutdown;
 #endif
     
     m=menu_init( items, i, NULL, NULL, NULL, NULL );
diff --git a/apps/screens.c b/apps/screens.c
index e188d341d8..6d9b299c54 100644
--- a/apps/screens.c
+++ b/apps/screens.c
@@ -1253,7 +1253,7 @@ bool shutdown_screen(void)
         switch(button)
         {
             case BUTTON_OFF:
-                clean_shutdown();
+                sys_poweroff(false);
                 break;
 
             default:
diff --git a/firmware/export/powermgmt.h b/firmware/export/powermgmt.h
index 29ef9eba95..e32867a188 100644
--- a/firmware/export/powermgmt.h
+++ b/firmware/export/powermgmt.h
@@ -130,5 +130,6 @@ int get_sleep_timer(void);
 void set_car_adapter_mode(bool setting);
 void reset_poweroff_timer(void);
 void shutdown_hw(void);
+void sys_poweroff(bool halt);
 
 #endif
diff --git a/firmware/powermgmt.c b/firmware/powermgmt.c
index 519785f9c6..fdc77f680b 100644
--- a/firmware/powermgmt.c
+++ b/firmware/powermgmt.c
@@ -44,6 +44,7 @@
 #ifdef HAVE_UDA1380
 #include "uda1380.h"
 #endif
+#include "logf.h"
 
 /*
  * Define DEBUG_FILE to create a csv (spreadsheet) with battery information
@@ -389,7 +390,7 @@ static void handle_auto_poweroff(void)
         if(TIME_AFTER(current_tick, last_event_tick    + timeout) &&
            TIME_AFTER(current_tick, last_disk_activity + timeout))
         {
-            shutdown_hw();
+            sys_poweroff(true);
         }
     }
     else
@@ -412,11 +413,7 @@ static void handle_auto_poweroff(void)
 #endif
                 {
                     DEBUGF("Sleep timer timeout. Shutting off...\n");
-                    /* Make sure that the disk isn't spinning when
-                       we cut the power */
-                    while(ata_disk_is_active())
-                       sleep(HZ);
-                    shutdown_hw();
+                    sys_poweroff(true);
                 }
             }
         }
@@ -877,6 +874,14 @@ void powermgmt_init(void)
 
 #endif /* SIMULATOR */
 
+void sys_poweroff(bool halt)
+{
+    logf("sys_poweroff(%d)", halt);
+    queue_post(&button_queue, SYS_POWEROFF, NULL);
+    while(halt)
+        yield();
+}
+
 /* Various hardware housekeeping tasks relating to shutting down the jukebox */
 void shutdown_hw(void) 
 {
-- 
cgit v1.2.3