diff options
| author | Jean-Louis Biasini <jlbiasini@gmail.com> | 2013-09-02 11:03:56 +0200 |
|---|---|---|
| committer | Amaury Pouly <amaury.pouly@gmail.com> | 2013-09-05 20:02:07 +0200 |
| commit | df6eb82f5156256e1999374ce9b1a159610ff9a0 (patch) | |
| tree | ec79d078c7deaba74ad0b83d6dc48c3776e8f7f0 /firmware/drivers | |
| parent | ef2dd06d5ee7415684f45afadad6deaf0fb0c615 (diff) | |
| download | rockbox-df6eb82f5156256e1999374ce9b1a159610ff9a0.zip rockbox-df6eb82f5156256e1999374ce9b1a159610ff9a0.tar.gz rockbox-df6eb82f5156256e1999374ce9b1a159610ff9a0.tar.bz2 rockbox-df6eb82f5156256e1999374ce9b1a159610ff9a0.tar.xz | |
touch devices: Disable touch on softlock.
Target that have a touchpad/touchscreen should disable it while
being locked (In order to avoid LCD to drain battery power due to
"key locked" constant reporting messages. If they a have a keylock
button this was already handled at driver level. If not (e.g. fuze+),
they will have to implement a switch at driver level that action.c
can operate on softlock.
This patch does the following for any target having a touchpad
or a touchscreen and no HAS_BUTTON_HOLD (ie any softlock target)
1) it implements the code to call button_enable_touch(bool en) in
action.c.
2) button_enable_touch is implemented in button.c and call
either touchpad_enable or touchscreen_enable
3) those two function are implemented respectively in touchscreen.c
and a new touchpad.c file. They provide a generic way to silents touch's
device and call a function at driver level where target specific code
can be implemented if possible/needed (for power saving for instance).
Those function name are touchpad_enable_device and touchscreen_enable_device
4) we implement an empty function at driver level of targets that need it
to have them still being able to compiled.
Change-Id: I9ead78a25bd33466a8533f5b9f259b395cb5ce49
Reviewed-on: http://gerrit.rockbox.org/569
Reviewed-by: Thomas Martitz <kugel@rockbox.org>
Reviewed-by: Amaury Pouly <amaury.pouly@gmail.com>
Diffstat (limited to 'firmware/drivers')
| -rw-r--r-- | firmware/drivers/button.c | 12 | ||||
| -rw-r--r-- | firmware/drivers/touchpad.c | 43 | ||||
| -rw-r--r-- | firmware/drivers/touchscreen.c | 16 |
3 files changed, 71 insertions, 0 deletions
diff --git a/firmware/drivers/button.c b/firmware/drivers/button.c index c165e8f..1cc9542 100644 --- a/firmware/drivers/button.c +++ b/firmware/drivers/button.c @@ -672,3 +672,15 @@ int button_apply_acceleration(const unsigned int data) return delta; } #endif /* HAVE_WHEEL_ACCELERATION */ + +#if (defined(HAVE_TOUCHPAD) || defined(HAVE_TOUCHSCREEN)) && !defined(HAS_BUTTON_HOLD) +void button_enable_touch(bool en) +{ +#ifdef HAVE_TOUCHPAD + touchpad_enable(en); +#endif +#ifdef HAVE_TOUCHSCREEN + touchscreen_enable(en); +#endif +} +#endif diff --git a/firmware/drivers/touchpad.c b/firmware/drivers/touchpad.c new file mode 100644 index 0000000..1d78ee1 --- /dev/null +++ b/firmware/drivers/touchpad.c @@ -0,0 +1,43 @@ +/*************************************************************************** + * __________ __ ___. + * Open \______ \ ____ ____ | | _\_ |__ _______ ___ + * Source | _// _ \_/ ___\| |/ /| __ \ / _ \ \/ / + * Jukebox | | ( <_> ) \___| < | \_\ ( <_> > < < + * Firmware |____|_ /\____/ \___ >__|_ \|___ /\____/__/\_ \ + * \/ \/ \/ \/ \/ + * $Id$ + * + * Copyright (C) 2013 by Jean-Louis Biasini + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This software is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY + * KIND, either express or implied. + * + ****************************************************************************/ + +#include <stdbool.h> +#include "button.h" +#include "touchpad.h" +#include "button-target.h" + +static bool touch_enabled = true; + +void touchpad_enable(bool en) +{ + if(en != touch_enabled) + { + touch_enabled = en; + touchpad_enable_device(en); + } +} + +int touchpad_filter(int button) +{ + if(!touch_enabled) + button &= ~BUTTON_TOUCHPAD; + return button; +} diff --git a/firmware/drivers/touchscreen.c b/firmware/drivers/touchscreen.c index 823c2e7..8ce2400 100644 --- a/firmware/drivers/touchscreen.c +++ b/firmware/drivers/touchscreen.c @@ -31,6 +31,7 @@ #define BUTTON_MARGIN_X (int)(LCD_WIDTH * 0.03) #define BUTTON_MARGIN_Y (int)(LCD_HEIGHT * 0.03) +static bool touch_enabled = true; static enum touchscreen_mode current_mode = TOUCHSCREEN_POINT; static const int touchscreen_buttons[3][3] = { @@ -121,6 +122,8 @@ static void map_pixels(int *x, int *y) /* TODO: add jitter (and others) filter */ int touchscreen_to_pixels(int x, int y, int *data) { + if(!touch_enabled) + return 0; x &= 0xFFFF; y &= 0xFFFF; @@ -169,6 +172,19 @@ enum touchscreen_mode touchscreen_get_mode(void) return current_mode; } +void touchscreen_enable(bool en) +{ + if(en != touch_enabled) + { + touch_enabled = en; + touchscreen_enable_device(en); + } +} + +bool touchscreen_is_enabled(void) +{ + return touch_enabled; +} #if ((CONFIG_PLATFORM & PLATFORM_ANDROID) == 0) /* android has an API for this */ |