Browse Source

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>
Kevin McKinney 13 years ago
parent
commit
00da8eddf9
1 changed files with 2 additions and 4 deletions
  1. 2 4
      drivers/staging/bcm/led_control.h

+ 2 - 4
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