[PATCH 03/06] Fix compilation warning for drivers/leds/leds_gpio.c

[Subrata Modak]

gcc 4.4.1 generates the following build warning on i386:

drivers/leds/leds-gpio.c:75: warning: â??create_gpio_ledâ?? defined but not used
drivers/leds/leds-gpio.c:126: warning: â??delete_gpio_ledâ?? defined but not used
drivers/leds/leds-gpio.c: In function â??gpio_led_initâ??:
drivers/leds/leds-gpio.c:316: warning: â??retâ?? is used uninitialized in this function

Fix them as below:
Put the definitions of create_gpio_led() and delete_gpio_led()
inside
	#ifdef CONFIG_LEDS_GPIO_PLATFORM
when you actually call them inside
	#ifdef CONFIG_LEDS_GPIO_PLATFORM

And the remaining fix is inspired by David Howells fix few days back:
http://lkml.org/lkml/2009/7/9/109,

Signed-off-by: Subrata Modak<subrata@xxxxxxxxxxxxxxxxxx>,
---

--- a/drivers/leds/leds-gpio.c	2009-06-29 19:20:42.000000000 +0530
+++ b/drivers/leds/leds-gpio.c	2009-07-15 05:51:21.000000000 +0530
@@ -29,6 +29,7 @@ struct gpio_led_data {
 			unsigned long *delay_on, unsigned long *delay_off);
 };
 
+#ifdef CONFIG_LEDS_GPIO_PLATFORM
 static void gpio_led_work(struct work_struct *work)
 {
 	struct gpio_led_data	*led_dat =
@@ -132,7 +133,6 @@ static void delete_gpio_led(struct gpio_
 	gpio_free(led->gpio);
 }
 
-#ifdef CONFIG_LEDS_GPIO_PLATFORM
 static int __devinit gpio_led_probe(struct platform_device *pdev)
 {
 	struct gpio_led_platform_data *pdata = pdev->dev.platform_data;
@@ -298,7 +298,7 @@ static struct of_platform_driver of_gpio
 
 static int __init gpio_led_init(void)
 {
-	int ret;
+	int uninitialized_var(ret);
 
 #ifdef CONFIG_LEDS_GPIO_PLATFORM	
 	ret = platform_driver_register(&gpio_led_driver);

---
Regards--
Subrata

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/