Enabled the quickscreen for iAudio X5. Globally, REC enters the menu and holding REC enters the quickscreen. Also moved the quickscreen #define to model config files, which simplifies the checking for it and makes checking more consistent. Cleaned up keymappings in quickscreen.h.

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

7 files changed, 36 insertions, 42 deletions

diff --git a/apps/gui/gwps.c b/apps/gui/gwps.c
index a43f2e4..db92961 100644
--- a/apps/gui/gwps.c
+++ b/apps/gui/gwps.c
@@ -569,9 +569,7 @@ long gui_wps_show(void)
                 break;
 #endif
 
-#if (CONFIG_KEYPAD == RECORDER_PAD) || (CONFIG_KEYPAD == IRIVER_H100_PAD) ||\
-    (CONFIG_KEYPAD == IRIVER_H300_PAD) || (CONFIG_KEYPAD == IPOD_3G_PAD) ||\
-    (CONFIG_KEYPAD == IPOD_4G_PAD)
+#ifdef HAVE_QUICKSCREEN
                 /* play settings */
             case WPS_QUICK:
 #ifdef WPS_RC_QUICK
diff --git a/apps/gui/gwps.h b/apps/gui/gwps.h
index cce2a2b..d9f7224 100644
--- a/apps/gui/gwps.h
+++ b/apps/gui/gwps.h
@@ -236,11 +236,13 @@
 #define WPS_INCVOL     BUTTON_UP
 #define WPS_DECVOL     BUTTON_DOWN
 #define WPS_PAUSE      BUTTON_PLAY
-#define WPS_MENU       BUTTON_REC
+#define WPS_MENU       (BUTTON_REC|BUTTON_REL)
+#define WPS_MENU_PRE   BUTTON_REC
 #define WPS_BROWSE     (BUTTON_SELECT | BUTTON_REL)
 #define WPS_BROWSE_PRE BUTTON_SELECT
 #define WPS_EXIT       (BUTTON_PLAY | BUTTON_REPEAT)
 #define WPS_CONTEXT    (BUTTON_SELECT | BUTTON_REPEAT)
+#define WPS_QUICK      (BUTTON_REC | BUTTON_REPEAT)
 
 #elif CONFIG_KEYPAD == GIGABEAT_PAD
 
diff --git a/apps/gui/quickscreen.c b/apps/gui/quickscreen.c
index 24ab725..981199d 100644
--- a/apps/gui/quickscreen.c
+++ b/apps/gui/quickscreen.c
@@ -18,7 +18,8 @@
  ****************************************************************************/
 
 #include "quickscreen.h"
-#ifdef HAS_QUICKSCREEN
+
+#ifdef HAVE_QUICKSCREEN
 
 #include <stdio.h>
 #include "system.h"
@@ -83,7 +84,7 @@ void gui_quickscreen_draw(struct gui_quickscreen * qs, struct screen * display)
     else
     {
         display->putsxy(display->width - w - 12, 24, line_text);
-        display->mono_bitmap(bitmap_icons_7x8[Icon_FastForward], 
+        display->mono_bitmap(bitmap_icons_7x8[Icon_FastForward],
                         display->width - 8, 24, 7, 8);
     }
     display->getstringsize(option, &w, NULL);
@@ -217,5 +218,5 @@ bool gui_syncquickscreen_run(struct gui_quickscreen * qs, int button_enter)
     }
 }
 
-#endif /* HAS_QUICKSCREEN */
+#endif /* HAVE_QUICKSCREEN */
 
diff --git a/apps/gui/quickscreen.h b/apps/gui/quickscreen.h
index e6e04c9..55e5a2d 100644
--- a/apps/gui/quickscreen.h
+++ b/apps/gui/quickscreen.h
@@ -18,32 +18,26 @@
  ****************************************************************************/
 #include "button.h"
 #include "config.h"
-#if (CONFIG_KEYPAD == RECORDER_PAD) || (CONFIG_KEYPAD == IRIVER_H100_PAD) ||\
-    (CONFIG_KEYPAD == IRIVER_H300_PAD) || (CONFIG_KEYPAD == IPOD_4G_PAD) ||\
-    (CONFIG_KEYPAD == IPOD_3G_PAD)
+
+#ifdef HAVE_QUICKSCREEN
 
 #ifndef _GUI_QUICKSCREEN_H_
 #define _GUI_QUICKSCREEN_H_
 
-#define HAS_QUICKSCREEN
-
 #include "option_select.h"
 #include "screen_access.h"
 
 #define QUICKSCREEN_LEFT        BUTTON_LEFT
 #define QUICKSCREEN_RIGHT       BUTTON_RIGHT
 
-#if (CONFIG_KEYPAD == IRIVER_H100_PAD) || (CONFIG_KEYPAD == IRIVER_H300_PAD) ||\
-    (CONFIG_KEYPAD == RECORDER_PAD)
+#if (CONFIG_KEYPAD == RECORDER_PAD)
+#define QUICKSCREEN_BOTTOM     BUTTON_DOWN
+#define QUICKSCREEN_BOTTOM_INV BUTTON_UP
+#define QUICKSCREEN_QUIT       BUTTON_F3
+#define QUICKSCREEN_QUIT2      BUTTON_OFF
+#elif (CONFIG_KEYPAD == IRIVER_H100_PAD) || (CONFIG_KEYPAD == IRIVER_H300_PAD)
 #define QUICKSCREEN_BOTTOM     BUTTON_DOWN
 #define QUICKSCREEN_BOTTOM_INV BUTTON_UP
-#elif (CONFIG_KEYPAD == IPOD_4G_PAD) || (CONFIG_KEYPAD == IPOD_3G_PAD)
-#define QUICKSCREEN_BOTTOM     BUTTON_PLAY
-#define QUICKSCREEN_QUIT  BUTTON_MENU
-#define QUICKSCREEN_QUIT2 BUTTON_SELECT
-#endif
-
-#if (CONFIG_KEYPAD == IRIVER_H100_PAD) || (CONFIG_KEYPAD == IRIVER_H300_PAD)
 #define QUICKSCREEN_QUIT  BUTTON_MODE
 #define QUICKSCREEN_QUIT2 BUTTON_OFF
 #define QUICKSCREEN_RC_QUIT     BUTTON_RC_MODE
@@ -53,7 +47,14 @@
 #define QUICKSCREEN_RC_BOTTOM_INV  BUTTON_RC_VOL_UP
 #define QUICKSCREEN_RC_RIGHT       BUTTON_RC_FF
 #endif
-
+#elif (CONFIG_KEYPAD == IPOD_4G_PAD) || (CONFIG_KEYPAD == IPOD_3G_PAD)
+#define QUICKSCREEN_BOTTOM BUTTON_PLAY
+#define QUICKSCREEN_QUIT   BUTTON_MENU
+#define QUICKSCREEN_QUIT2  BUTTON_SELECT
+#elif (CONFIG_KEYPAD == IAUDIO_X5_PAD)
+#define QUICKSCREEN_BOTTOM BUTTON_DOWN
+#define QUICKSCREEN_QUIT   BUTTON_REC
+#define QUICKSCREEN_QUIT2  BUTTON_POWER
 #endif
 
 struct gui_quickscreen;
@@ -118,4 +119,4 @@ void gui_syncquickscreen_draw(struct gui_quickscreen * qs);
 bool gui_syncquickscreen_run(struct gui_quickscreen * qs, int button_enter);
 
 #endif /*_GUI_QUICK_SCREEN_H_*/
-#endif /* CONFIG_KEYPAD */
+#endif /* HAVE_QUICKSCREEN */
diff --git a/apps/screens.c b/apps/screens.c
index 4da5677..5d1a3f0 100644
--- a/apps/screens.c
+++ b/apps/screens.c
@@ -460,9 +460,8 @@ bool pitch_screen(void)
 }
 #endif
 
-#if (CONFIG_KEYPAD == RECORDER_PAD) || (CONFIG_KEYPAD == IRIVER_H100_PAD) ||\
-    (CONFIG_KEYPAD == IRIVER_H300_PAD) || (CONFIG_KEYPAD == IPOD_4G_PAD) ||\
-    (CONFIG_KEYPAD == IPOD_3G_PAD)
+#ifdef HAVE_QUICKSCREEN
+
 #define bool_to_int(b)\
     b?1:0
 #define int_to_bool(i)\
diff --git a/apps/tree.c b/apps/tree.c
index f523b1c..2777970 100644
--- a/apps/tree.c
+++ b/apps/tree.c
@@ -365,7 +365,7 @@ static int update_dir(void)
 void resume_directory(const char *dir)
 {
     bool id3db = *tc.dirfilter == SHOW_ID3DB;
-    
+
     if (ft_load(&tc, dir) < 0)
         return;
     lastdir[0] = 0;
@@ -460,7 +460,7 @@ void set_current_file(char *path)
     }
 
     strcpy(lastfile, name);
-    
+
     /* undefined item selected */
     tc.selected_item = -1;
 
@@ -641,7 +641,7 @@ static bool dirbrowse(void)
                     }
 #if defined(HAVE_CHARGING) && \
     (CONFIG_KEYPAD == RECORDER_PAD) && !defined(HAVE_SW_POWEROFF)
-                    else { 
+                    else {
                         if (!charger_inserted()) {
                             if(shutdown_screen())
                                 reload_dir = true;
@@ -719,7 +719,7 @@ static bool dirbrowse(void)
                 }
                 break;
 
-#ifdef TREE_QUICK
+#ifdef HAVE_QUICKSCREEN
             case TREE_QUICK:
 #ifdef TREE_RC_QUICK
             case TREE_RC_QUICK:
@@ -905,7 +905,7 @@ static bool dirbrowse(void)
                 lastextra = -1;
                 reload_root = false;
             }
-            
+
             if (! reload_dir )
             {
                 gui_synclist_select_item(&tree_lists, 0);
@@ -1073,7 +1073,7 @@ static bool add_dir(char* dirname, int len, int fd)
                             plsize++;
                             if(TIME_AFTER(current_tick, pltick+HZ/4)) {
                                 pltick = current_tick;
-                                
+
                                 snprintf(buf, sizeof buf, "%d", plsize);
 #ifdef HAVE_LCD_BITMAP
                                 FOR_NB_SCREENS(i)
@@ -1117,7 +1117,7 @@ bool create_playlist(void)
     char filename[MAX_PATH];
 
     pltick = current_tick;
-    
+
     snprintf(filename, sizeof filename, "%s.m3u",
              tc.currdir[1] ? tc.currdir : "/root");
     FOR_NB_SCREENS(i)
diff --git a/apps/tree.h b/apps/tree.h
index c811b5d..29532b5 100644
--- a/apps/tree.h
+++ b/apps/tree.h
@@ -37,7 +37,6 @@
 #define TREE_WPS_PRE   BUTTON_ON
 #define TREE_CONTEXT   (BUTTON_SELECT | BUTTON_REPEAT)
 #define TREE_CONTEXT2  (BUTTON_ON | BUTTON_SELECT)
-#define TREE_POWER_BTN BUTTON_ON
 #define TREE_QUICK     (BUTTON_MODE | BUTTON_REPEAT)
 
 /* Remote keys */
@@ -67,7 +66,6 @@
 #define TREE_WPS_PRE   BUTTON_ON
 #define TREE_CONTEXT   (BUTTON_PLAY | BUTTON_REPEAT)
 #define TREE_CONTEXT2  (BUTTON_ON | BUTTON_PLAY)
-#define TREE_POWER_BTN BUTTON_ON
 #define TREE_QUICK     BUTTON_F2
 
 #define TREE_RC_NEXT   BUTTON_RC_RIGHT
@@ -85,7 +83,6 @@
 #define TREE_WPS_PRE   BUTTON_ON
 #define TREE_CONTEXT   (BUTTON_PLAY | BUTTON_REPEAT)
 #define TREE_CONTEXT2  (BUTTON_ON | BUTTON_PLAY)
-#define TREE_POWER_BTN BUTTON_ON
 
 #define TREE_RC_NEXT   BUTTON_RC_RIGHT
 #define TREE_RC_PREV   BUTTON_RC_LEFT
@@ -103,7 +100,6 @@
 #define TREE_WPS       (BUTTON_MENU | BUTTON_REL)
 #define TREE_WPS_PRE   BUTTON_MENU
 #define TREE_CONTEXT   (BUTTON_RIGHT | BUTTON_REPEAT)
-#define TREE_POWER_BTN BUTTON_OFF
 
 #elif CONFIG_KEYPAD == GMINI100_PAD
 #define TREE_EXIT      BUTTON_LEFT
@@ -115,7 +111,6 @@
 #define TREE_WPS_PRE   BUTTON_ON
 #define TREE_CONTEXT   (BUTTON_PLAY | BUTTON_REPEAT)
 #define TREE_CONTEXT2  (BUTTON_ON | BUTTON_PLAY)
-#define TREE_POWER_BTN BUTTON_ON
 
 #elif (CONFIG_KEYPAD == IPOD_3G_PAD) || (CONFIG_KEYPAD == IPOD_4G_PAD)
 
@@ -142,7 +137,6 @@
 #define TREE_WPS       (BUTTON_SELECT | BUTTON_REL)
 #define TREE_WPS_PRE   BUTTON_SELECT
 #define TREE_CONTEXT   (BUTTON_RIGHT | BUTTON_REPEAT)
-#define TREE_POWER_BTN (BUTTON_PLAY | BUTTON_REPEAT)
 
 #elif CONFIG_KEYPAD == IAUDIO_X5_PAD
 
@@ -151,12 +145,12 @@
 #define TREE_ENTER     BUTTON_RIGHT
 #define TREE_RUN       (BUTTON_SELECT | BUTTON_REL)
 #define TREE_RUN_PRE   BUTTON_SELECT
-#define TREE_MENU      (BUTTON_REC | BUTTON_REPEAT)
+#define TREE_MENU      (BUTTON_REC | BUTTON_REL)
 #define TREE_MENU_PRE  BUTTON_REC
 #define TREE_WPS       (BUTTON_PLAY | BUTTON_REL)
 #define TREE_WPS_PRE   BUTTON_PLAY
 #define TREE_CONTEXT   (BUTTON_SELECT | BUTTON_REPEAT)
-#define TREE_POWER_BTN (BUTTON_REC | BUTTON_REPEAT)
+#define TREE_QUICK     (BUTTON_REC | BUTTON_REPEAT)
 
 #elif CONFIG_KEYPAD == GIGABEAT_PAD
 
@@ -172,7 +166,6 @@
 #define TREE_WPS       (BUTTON_POWER | BUTTON_REL)
 #define TREE_WPS_PRE   BUTTON_POWER
 #define TREE_CONTEXT   (BUTTON_SELECT | BUTTON_REPEAT)
-#define TREE_POWER_BTN BUTTON_POWER
 
 #endif