From: Kevin McKinney <klmckinney1@gmail.com>
Date: Mon, 13 Feb 2012 03:24:53 +0000 (-0500)
Subject: Staging: bcm: Move directives for the preprocessor statement to enum value in led_con... 
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=00da8eddf9eb8d432533c3a83da3a6470a45dfc4;p=GitHub%2FLineageOS%2FG12%2Fandroid_kernel_amlogic_linux-4.9.git

Staging: bcm: Move directives for the preprocessor statement to enum value in led_control.h

DRIVER_HALT is a driver state that was originally
defined as a #define statement. This patch moves
it to the LedEvents type as an enumerated
value for the purpose of removing a compile time warning:

drivers/staging/bcm/led_control.c: In function ‘LEDControlThread’:
drivers/staging/bcm/led_control.c:817:3: warning: case value ‘255’ not in enumerated type ‘LedEventInfo_t’ [-Wswitch]

Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/bcm/led_control.h b/drivers/staging/bcm/led_control.h
index 84d1a028e1e2..ed8fbc091115 100644
--- a/drivers/staging/bcm/led_control.h
+++ b/drivers/staging/bcm/led_control.h
@@ -63,12 +63,10 @@ typedef enum LedEvents {
 	IDLEMODE_CONTINUE = 0x40,
 	IDLEMODE_EXIT = 0x80,
 	LED_THREAD_INACTIVE = 0x100,  /* Makes the LED thread Inactivce. It wil be equivallent to putting the thread on hold. */
-	LED_THREAD_ACTIVE = 0x200    /* Makes the LED Thread Active back. */
+	LED_THREAD_ACTIVE = 0x200,    /* Makes the LED Thread Active back. */
+	DRIVER_HALT = 0xff
 } LedEventInfo_t;	/* Enumerated values of different driver states */
 
-#define DRIVER_HALT 0xff
-
-
 /*
  * Structure which stores the information of different LED types
  * and corresponding LED state information of driver states