Index: apps/action.c
===================================================================
--- apps/action.c (revision 30969)
+++ apps/action.c (working copy)
@@ -150,6 +150,35 @@
items[i].action_code;
}
+static bool backlight_on_action(int action)
+{
+ /* no backlight on volume change and pause/play */
+ return (action != ACTION_WPS_VOLUP)
+ && (action != ACTION_WPS_VOLDOWN)
+ && (action != ACTION_WPS_PLAY);
+}
+
+static bool backlight_on_keypress_oracle(int btn)
+{
+ /* check whether the button would fire an action */
+ /* If yes, check whether that action should turn on the backlight */
+ /* Keep this function fast! It's called from the button ticktask */
+ /* or the scrollwheel interrupt handler! */
+
+ const struct button_mapping *items = NULL;
+ int i = 0;
+
+ if (!global_settings.backlight_on_volume_change)
+ {
+ items = get_context_mapping(CONTEXT_WPS);
+ /* also check action for button release */
+ return backlight_on_action(do_button_check(items, btn, BUTTON_NONE, &i))
+ && backlight_on_action(do_button_check(items, btn | BUTTON_REL, btn, &i));
+ }
+ else
+ return true;
+}
+
#if defined(HAVE_GUI_BOOST) && defined(HAVE_ADJUSTABLE_CPU_FREQ)
/* Helper function which is called to boost / unboost CPU. This function
* avoids to increase boost_count with each call of gui_boost(). */
@@ -197,6 +226,16 @@
send_event(GUI_EVENT_ACTIONUPDATE, NULL);
+ /* In CONTEXT_WPS or CONTEXT_FM we may want to keep the backlight off
+ /* on certain actions. */
+ /* Thus, a callback function is used to check if it should turn on. */
+ /* See also button.c */
+ if ( (context & ~ALLOW_SOFTLOCK) == CONTEXT_WPS
+ || (context & ~ALLOW_SOFTLOCK) == CONTEXT_FM )
+ set_backlight_on_keypress_oracle(backlight_on_keypress_oracle);
+ else
+ set_backlight_on_keypress_oracle(NULL);
+
if (timeout == TIMEOUT_NOBLOCK)
button = button_get(false);
else if (timeout == TIMEOUT_BLOCK)
Index: apps/lang/deutsch.lang
===================================================================
--- apps/lang/deutsch.lang (revision 30969)
+++ apps/lang/deutsch.lang (working copy)
@@ -2595,6 +2595,20 @@
+ id: LANG_BACKLIGHT_ON_VOLUME_CHANGE
+ desc: Backlight on volume change and on pause/play
+ user: core
+
+ *: "Backlight On Volume Change"
+
+
+ *: "Beleuchtung bei Änderung der Lautstärke"
+
+
+ *: "Beleuchtung bei Änderung der Lautstärke"
+
+
+
id: LANG_LCD_SLEEP_AFTER_BACKLIGHT_OFF
desc: In display settings, time to switch LCD chip into power saving state
user: core
Index: apps/lang/english.lang
===================================================================
--- apps/lang/english.lang (revision 30969)
+++ apps/lang/english.lang (working copy)
@@ -2678,6 +2678,20 @@
+ id: LANG_BACKLIGHT_ON_VOLUME_CHANGE
+ desc: Backlight on volume change and on pause/play
+ user: core
+
+ *: "Backlight on Volume Change"
+
+
+ *: "Backlight on Volume Change"
+
+
+ *: "Backlight on Volume Change"
+
+
+
id: LANG_LCD_SLEEP_AFTER_BACKLIGHT_OFF
desc: In display settings, time to switch LCD chip into power saving state
user: core
Index: apps/settings.h
===================================================================
--- apps/settings.h (revision 30969)
+++ apps/settings.h (working copy)
@@ -682,6 +682,8 @@
then according to timeout_values[] */
bool caption_backlight; /* turn on backlight at end and start of track */
bool bl_filter_first_keypress; /* filter first keypress when dark? */
+ bool backlight_on_volume_change; /* turn on the backlight on volume change
+ and on pause/play? */
#if CONFIG_CHARGING
int backlight_timeout_plugged;
#endif
Index: apps/menus/display_menu.c
===================================================================
--- apps/menus/display_menu.c (revision 30969)
+++ apps/menus/display_menu.c (working copy)
@@ -101,6 +101,9 @@
MENUITEM_SETTING(bl_filter_first_keypress,
&global_settings.bl_filter_first_keypress,
filterfirstkeypress_callback);
+MENUITEM_SETTING(backlight_on_volume_change,
+ &global_settings.backlight_on_volume_change,
+ NULL);
#ifdef HAVE_LCD_SLEEP_SETTING
MENUITEM_SETTING(lcd_sleep_after_backlight_off,
&global_settings.lcd_sleep_after_backlight_off, NULL);
@@ -138,6 +141,7 @@
,&backlight_fade_in, &backlight_fade_out
#endif
,&bl_filter_first_keypress
+ ,&backlight_on_volume_change
# ifdef HAVE_LCD_SLEEP_SETTING
,&lcd_sleep_after_backlight_off
# endif
Index: apps/settings_list.c
===================================================================
--- apps/settings_list.c (revision 30969)
+++ apps/settings_list.c (working copy)
@@ -908,6 +908,9 @@
OFFON_SETTING(0, bl_filter_first_keypress,
LANG_BACKLIGHT_FILTER_FIRST_KEYPRESS, false,
"backlight filters first keypress", NULL),
+ OFFON_SETTING(0, backlight_on_volume_change,
+ LANG_BACKLIGHT_ON_VOLUME_CHANGE, true,
+ "backlight on volume change", NULL),
#ifdef HAVE_REMOTE_LCD
OFFON_SETTING(0, remote_bl_filter_first_keypress,
LANG_BACKLIGHT_FILTER_FIRST_KEYPRESS, false,
Index: firmware/export/button.h
===================================================================
--- firmware/export/button.h (revision 30969)
+++ firmware/export/button.h (working copy)
@@ -46,6 +46,23 @@
#endif
#ifdef HAVE_BACKLIGHT
void set_backlight_filter_keypress(bool value);
+
+/* Callback function to tell whether the backlight should be turned on
+ * when a key is pressed. Called just after a key press.
+ *
+ * @param btn The button that was pressed
+ *
+ * Returns true iff the backlight should be turned on on key press.
+ */
+typedef bool (*backlight_on_keypress_oracle_func)(int btn);
+
+/* Sets the new 'oracle' to tell whether the backlight should be turned
+ * on when a key is pressed. Passing NULL as parameter will turn on the BL
+ * on every key press.
+ */
+void set_backlight_on_keypress_oracle(backlight_on_keypress_oracle_func func);
+void backlight_on_by_button(int btn);
+
#ifdef HAVE_REMOTE_LCD
void set_remote_backlight_filter_keypress(bool value);
#endif
Index: firmware/target/arm/as3525/sansa-fuzev2/button-fuzev2.c
===================================================================
--- firmware/target/arm/as3525/sansa-fuzev2/button-fuzev2.c (revision 30969)
+++ firmware/target/arm/as3525/sansa-fuzev2/button-fuzev2.c (working copy)
@@ -160,7 +160,7 @@
(wheel_delta << 24) | wheel_velocity*360/WHEELCLICKS_PER_ROTATION);
/* message posted - reset delta and poke backlight on*/
wheel_delta = 1;
- backlight_on();
+ backlight_on_by_button(btn);
buttonlight_on();
}
else
Index: firmware/target/arm/ipod/button-clickwheel.c
===================================================================
--- firmware/target/arm/ipod/button-clickwheel.c (revision 30969)
+++ firmware/target/arm/ipod/button-clickwheel.c (working copy)
@@ -137,8 +137,7 @@
new_wheel_value = (status >> 16) & 0x7f;
whl = new_wheel_value;
- /* switch on backlight (again), reset power-off timer */
- backlight_on();
+ /* reset power-off timer */
reset_poweroff_timer();
/* Check whether the scrollwheel was untouched by accident or by will. */
@@ -181,6 +180,9 @@
if (wheel_keycode != BUTTON_NONE)
{
+ /* switch on backlight (again) */
+ backlight_on_by_button(wheel_keycode);
+
long v = (usec - last_wheel_usec) & 0x7fffffff;
/* undo signedness */
Index: firmware/target/arm/ipod/button-1g-3g.c
===================================================================
--- firmware/target/arm/ipod/button-1g-3g.c (revision 30969)
+++ firmware/target/arm/ipod/button-1g-3g.c (working copy)
@@ -95,13 +95,6 @@
count = 0;
}
- /* poke backlight every 1/4s of activity */
- if (TIME_AFTER(current_tick, next_backlight_on)) {
- backlight_on();
- reset_poweroff_timer();
- next_backlight_on = current_tick + HZ/4;
- }
-
/* has wheel travelled far enough? */
if (++count < WHEEL_BASE_SENSITIVITY) {
return;
@@ -126,6 +119,13 @@
/* have a keycode */
+ /* poke backlight every 1/4s of activity */
+ if (TIME_AFTER(current_tick, next_backlight_on)) {
+ backlight_on_by_button(wheel_keycode);
+ reset_poweroff_timer();
+ next_backlight_on = current_tick + HZ/4;
+ }
+
usec = USEC_TIMER;
v = usec - last_wheel_usec;
Index: firmware/target/arm/ipod/button-mini1g.c
===================================================================
--- firmware/target/arm/ipod/button-mini1g.c (revision 30969)
+++ firmware/target/arm/ipod/button-mini1g.c (working copy)
@@ -91,13 +91,6 @@
return;
}
- /* poke backlight every 1/4s of activity */
- if (TIME_AFTER(current_tick, next_backlight_on)) {
- backlight_on();
- reset_poweroff_timer();
- next_backlight_on = current_tick + HZ/4;
- }
-
if (++count < WHEEL_BASE_SENSITIVITY)
return;
@@ -118,6 +111,13 @@
/* have a keycode */
+ /* poke backlight every 1/4s of activity */
+ if (TIME_AFTER(current_tick, next_backlight_on)) {
+ backlight_on_by_button(wheel_keycode);
+ reset_poweroff_timer();
+ next_backlight_on = current_tick + HZ/4;
+ }
+
usec = USEC_TIMER;
v = usec - last_wheel_usec;
Index: firmware/target/arm/sandisk/sansa-e200/button-e200.c
===================================================================
--- firmware/target/arm/sandisk/sansa-e200/button-e200.c (revision 30969)
+++ firmware/target/arm/sandisk/sansa-e200/button-e200.c (working copy)
@@ -170,7 +170,7 @@
/* Poke backlight to turn it on or maintain it no more often
* than every 1/4 second */
next_backlight_on = current_tick + HZ/4;
- backlight_on();
+ backlight_on_by_button(keycode);
#ifdef HAVE_BUTTON_LIGHT
buttonlight_on();
#endif
Index: firmware/drivers/button.c
===================================================================
--- firmware/drivers/button.c (revision 30969)
+++ firmware/drivers/button.c (working copy)
@@ -41,6 +41,7 @@
#ifdef HAVE_REMOTE_LCD
#include "lcd-remote.h"
#endif
+#include "debug.h"
struct event_queue button_queue SHAREDBSS_ATTR;
@@ -52,6 +53,7 @@
#endif
#ifdef HAVE_BACKLIGHT
static bool filter_first_keypress;
+static backlight_on_keypress_oracle_func backlight_on_keypress_oracle = NULL;
#ifdef HAVE_REMOTE_LCD
static bool remote_filter_first_keypress;
#endif
@@ -138,6 +140,28 @@
return ret;
}
+void backlight_on_by_button(int btn)
+{
+ bool bl_on;
+ if (backlight_on_keypress_oracle != NULL)
+ {
+ bl_on = (*backlight_on_keypress_oracle)(btn);
+ DEBUGF("BL oracle (%d) returned: %d\n", btn, bl_on);
+ }
+ else
+ {
+ bl_on = true;
+ DEBUGF("BL callback is NULL -> bl ON\n");
+ }
+ if (bl_on)
+ {
+ backlight_on();
+#ifdef HAVE_BUTTON_LIGHT
+ buttonlight_on();
+#endif
+ }
+}
+
static void button_tick(void)
{
static int count = 0;
@@ -308,10 +332,13 @@
skip_remote_release = true;
}
else
-#endif
- if (!filter_first_keypress || is_backlight_on(false)
+#endif /* HAVE_REMOTE_LCD */
+ if (!filter_first_keypress
+ || is_backlight_on(false)
+ || ((backlight_on_keypress_oracle != NULL)
+ && !(*backlight_on_keypress_oracle)(btn))
#if BUTTON_REMOTE
- || (btn & BUTTON_REMOTE)
+ || (btn & BUTTON_REMOTE)
#endif
)
button_try_post(btn, data);
@@ -327,12 +354,7 @@
remote_backlight_on();
else
#endif
- {
- backlight_on();
-#ifdef HAVE_BUTTON_LIGHT
- buttonlight_on();
-#endif
- }
+ backlight_on_by_button(btn);
reset_poweroff_timer();
}
@@ -447,6 +469,7 @@
#endif
#ifdef HAVE_BACKLIGHT
filter_first_keypress = false;
+ set_backlight_on_keypress_oracle(NULL);
#ifdef HAVE_REMOTE_LCD
remote_filter_first_keypress = false;
#endif
@@ -558,6 +581,12 @@
{
filter_first_keypress = value;
}
+
+void set_backlight_on_keypress_oracle(backlight_on_keypress_oracle_func new_value)
+{
+ backlight_on_keypress_oracle = new_value;
+}
+
#ifdef HAVE_REMOTE_LCD
void set_remote_backlight_filter_keypress(bool value)
{