aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/leds
diff options
context:
space:
mode:
authorDevendra Naga <devendra.aaru@gmail.com>2012-07-11 05:50:32 -0400
committerBryan Wu <bryan.wu@canonical.com>2012-07-23 19:52:40 -0400
commit5ecf6e40bbb8649b8418c8038a2af1faefbc56e4 (patch)
tree0f18e577dfdb0a3179f46e4a4d37a4e9e213eba7 /drivers/leds
parenta209f7669870c461932515eba147b961692de37f (diff)
leds/leds-s3c24xx: use devm_gpio_request
why? because if the led_classdev_register fails we wont do gpio free because using devm_ functions, there is no need for error paths Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
Diffstat (limited to 'drivers/leds')
-rw-r--r--drivers/leds/leds-s3c24xx.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/leds/leds-s3c24xx.c b/drivers/leds/leds-s3c24xx.c
index e1206407616..942f0ea1817 100644
--- a/drivers/leds/leds-s3c24xx.c
+++ b/drivers/leds/leds-s3c24xx.c
@@ -65,7 +65,6 @@ static int s3c24xx_led_remove(struct platform_device *dev)
65 struct s3c24xx_gpio_led *led = pdev_to_gpio(dev); 65 struct s3c24xx_gpio_led *led = pdev_to_gpio(dev);
66 66
67 led_classdev_unregister(&led->cdev); 67 led_classdev_unregister(&led->cdev);
68 gpio_free(led->pdata->gpio);
69 68
70 return 0; 69 return 0;
71} 70}
@@ -92,7 +91,7 @@ static int s3c24xx_led_probe(struct platform_device *dev)
92 91
93 led->pdata = pdata; 92 led->pdata = pdata;
94 93
95 ret = gpio_request(pdata->gpio, "S3C24XX_LED"); 94 ret = devm_gpio_request(&dev->dev, pdata->gpio, "S3C24XX_LED");
96 if (ret < 0) 95 if (ret < 0)
97 return ret; 96 return ret;
98 97