Skip to content

Commit

Permalink
Staging: bcm: Move directives for the preprocessor statement to enum …
Browse files Browse the repository at this point in the history
…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>
  • Loading branch information
Kevin McKinney authored and Greg Kroah-Hartman committed Feb 14, 2012
1 parent 5fa5a90 commit 00da8ed
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions drivers/staging/bcm/led_control.h
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 00da8ed

Please sign in to comment.