Fix (unintentional) linkage of HAVE_BUTTON_DATA with HAVE_TOUCHSCREEN

Defining HAVE_BUTTON_DATA without simultaneously defining HAVE_TOUCHSCREEN
caused compile errors. (I need them separated for a scrollstrip driver.)

Change-Id: I945d3437d840035ccc0c147f8155029b403c6ec2
Reviewed-on: http://gerrit.rockbox.org/771
Reviewed-by: Amaury Pouly <amaury.pouly@gmail.com>
Reviewed-by: Thomas Martitz <kugel@rockbox.org>

1 files changed, 5 insertions, 1 deletions

diff --git a/firmware/target/hosted/sdl/button-sdl.c b/firmware/target/hosted/sdl/button-sdl.c
index 9677f1dd23..2a3202b6bc 100644
--- a/firmware/target/hosted/sdl/button-sdl.c
+++ b/firmware/target/hosted/sdl/button-sdl.c
@@ -444,11 +444,15 @@ static void button_event(int key, bool pressed)
     else
         btn &= ~new_btn;
 }
-#if defined(HAVE_BUTTON_DATA) && defined(HAVE_TOUCHSCREEN)
+#if defined(HAVE_BUTTON_DATA)
 int button_read_device(int* data)
 {
+#if defined(HAVE_TOUCHSCREEN)
     *data = mouse_coords;
 #else
+    (void) *data;   /* suppress compiler warnings */
+#endif
+#else
 int button_read_device(void)
 {
 #endif