From: Kevin McKinney 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-Tag: firefly_0821_release~3680^2~3347^2~160 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=00da8eddf9eb8d432533c3a83da3a6470a45dfc4;p=firefly-linux-kernel-4.4.55.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 Signed-off-by: Greg Kroah-Hartman --- 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