led: Add dependency on CONFIG_LEDS_TRIGGERS to led_trigger_rename_static()
This patch fixes build warnings when CONFIG_LEDS_TRIGGERS is disabled as below: include/linux/leds.h:158:18: warning: 'struct led_trigger' declared inside parameter list [enabled by default] include/linux/leds.h:158:18: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default] Signed-off-by: Jingoo Han <jg1.han@samsung.com> Acked-by: Fabio Baltieri <fabio.baltieri@gmail.com> Signed-off-by: Bryan Wu <cooloney@gmail.com>
This commit is contained in:
		
					parent
					
						
							
								a8df7b1ab7
							
						
					
				
			
			
				commit
				
					
						057407c732
					
				
			
		
					 1 changed files with 17 additions and 18 deletions
				
			
		| 
						 | 
				
			
			@ -139,24 +139,6 @@ extern void led_blink_set_oneshot(struct led_classdev *led_cdev,
 | 
			
		|||
extern void led_set_brightness(struct led_classdev *led_cdev,
 | 
			
		||||
			       enum led_brightness brightness);
 | 
			
		||||
 | 
			
		||||
/**
 | 
			
		||||
 * led_trigger_rename_static - rename a trigger
 | 
			
		||||
 * @name: the new trigger name
 | 
			
		||||
 * @trig: the LED trigger to rename
 | 
			
		||||
 *
 | 
			
		||||
 * Change a LED trigger name by copying the string passed in
 | 
			
		||||
 * name into current trigger name, which MUST be large
 | 
			
		||||
 * enough for the new string.
 | 
			
		||||
 *
 | 
			
		||||
 * Note that name must NOT point to the same string used
 | 
			
		||||
 * during LED registration, as that could lead to races.
 | 
			
		||||
 *
 | 
			
		||||
 * This is meant to be used on triggers with statically
 | 
			
		||||
 * allocated name.
 | 
			
		||||
 */
 | 
			
		||||
extern void led_trigger_rename_static(const char *name,
 | 
			
		||||
				      struct led_trigger *trig);
 | 
			
		||||
 | 
			
		||||
/*
 | 
			
		||||
 * LED Triggers
 | 
			
		||||
 */
 | 
			
		||||
| 
						 | 
				
			
			@ -197,6 +179,23 @@ extern void led_trigger_blink_oneshot(struct led_trigger *trigger,
 | 
			
		|||
				      unsigned long *delay_on,
 | 
			
		||||
				      unsigned long *delay_off,
 | 
			
		||||
				      int invert);
 | 
			
		||||
/**
 | 
			
		||||
 * led_trigger_rename_static - rename a trigger
 | 
			
		||||
 * @name: the new trigger name
 | 
			
		||||
 * @trig: the LED trigger to rename
 | 
			
		||||
 *
 | 
			
		||||
 * Change a LED trigger name by copying the string passed in
 | 
			
		||||
 * name into current trigger name, which MUST be large
 | 
			
		||||
 * enough for the new string.
 | 
			
		||||
 *
 | 
			
		||||
 * Note that name must NOT point to the same string used
 | 
			
		||||
 * during LED registration, as that could lead to races.
 | 
			
		||||
 *
 | 
			
		||||
 * This is meant to be used on triggers with statically
 | 
			
		||||
 * allocated name.
 | 
			
		||||
 */
 | 
			
		||||
extern void led_trigger_rename_static(const char *name,
 | 
			
		||||
				      struct led_trigger *trig);
 | 
			
		||||
 | 
			
		||||
#else
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue