diff options
| author | Igor Grinberg <grinberg@compulab.co.il> | 2013-11-08 01:03:51 +0200 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2013-11-25 10:41:52 -0500 |
| commit | 6516f81b64c71ab2be61f304d0e13dd84f730484 (patch) | |
| tree | 03460a528001726d6c5badc36622b6adf168db19 /drivers | |
| parent | 1df7bbba24b5f8f97331e30066358b7ab97ff1df (diff) | |
| download | blackbird-obmc-uboot-6516f81b64c71ab2be61f304d0e13dd84f730484.tar.gz blackbird-obmc-uboot-6516f81b64c71ab2be61f304d0e13dd84f730484.zip | |
gpio_led: check gpio_request() return value
Add a check for the gpio_request() function return value and do not try
to configure the GPIO if the gpio_request() call fails.
Also, print an error message indicating the gpio_request() has failed.
Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
Tested-by: Ilya Ledvich <ilya@compulab.co.il>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/misc/gpio_led.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/misc/gpio_led.c b/drivers/misc/gpio_led.c index 3fedddc8b5..de2041956b 100644 --- a/drivers/misc/gpio_led.c +++ b/drivers/misc/gpio_led.c @@ -11,7 +11,11 @@ void __led_init(led_id_t mask, int state) { - gpio_request(mask, "gpio_led"); + if (gpio_request(mask, "gpio_led") != 0) { + printf("%s: failed requesting GPIO%lu!\n", __func__, mask); + return; + } + gpio_direction_output(mask, state == STATUS_LED_ON); } |
