diff options
author | Devendra Naga <devendra.aaru@gmail.com> | 2012-07-11 05:50:32 -0400 |
---|---|---|
committer | Bryan Wu <bryan.wu@canonical.com> | 2012-07-23 19:52:40 -0400 |
commit | 5ecf6e40bbb8649b8418c8038a2af1faefbc56e4 (patch) | |
tree | 0f18e577dfdb0a3179f46e4a4d37a4e9e213eba7 /drivers/leds/leds-s3c24xx.c | |
parent | a209f7669870c461932515eba147b961692de37f (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/leds-s3c24xx.c')
-rw-r--r-- | drivers/leds/leds-s3c24xx.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/leds/leds-s3c24xx.c b/drivers/leds/leds-s3c24xx.c index e1206407616d..942f0ea18178 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 | ||