liblights: Fix variable button brightness code
authorPaul Keith <javelinanddart@gmail.com>
Fri, 2 Jun 2017 22:52:14 +0000 (17:52 -0500)
committerPaul Keith <javelinanddart@gmail.com>
Sat, 3 Jun 2017 02:23:51 +0000 (21:23 -0500)
* Because we're using #ifdef in set_light_buttons(),
  defining VAR_BUTTON_BRIGHTNESS to anything, including 0,
  results in the button brightness code being enabled
* That results in broken touchkey backlight for devices
  that don't override the samsung_lights header, or defined
  this macro to 0 in their device's samsung_lights header
* Comment out the macro to correct this logic

Change-Id: Icf554b0deac975f729f1a70e517341dae74f3657

liblights/include/samsung_lights.h

index f9392c1f26645eb10ed2782df46b02f88b50a9a9..fce13081c69973f1291a9db2ad33f1dbc83cc1a4 100644 (file)
@@ -30,8 +30,8 @@
 #define BUTTON_BRIGHTNESS_NODE "/sys/class/sec/sec_touchkey/brightness"
 #define LED_BLINK_NODE "/sys/class/sec/led/led_blink"
 
-// Set to 1 to enable variable button brightness
-#define VAR_BUTTON_BRIGHTNESS 0
+// Uncomment to enable variable button brightness
+//#define VAR_BUTTON_BRIGHTNESS 1
 
 /*
  * Brightness adjustment factors