Staging: bcm: Move directives for the preprocessor statement to enum value in led_con...
authorKevin McKinney <klmckinney1@gmail.com>
Mon, 13 Feb 2012 03:24:53 +0000 (22:24 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Feb 2012 04:16:14 +0000 (20:16 -0800)
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>
drivers/staging/bcm/led_control.h

index 84d1a02..ed8fbc0 100644 (file)
@@ -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