aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/leds/leds-gpio.c
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2016-02-06 09:37:33 -0500
committerJacek Anaszewski <j.anaszewski@samsung.com>2016-03-14 04:22:20 -0400
commit2d88a331e48095cf60ad9bdf3177bd401bf99727 (patch)
tree76d06c4421982a56793a5edf97c021b9514243fc /drivers/leds/leds-gpio.c
parentc9eec908a1ec7829497a0914514badec95cc8461 (diff)
leds: leds-gpio: Fix return value check in create_gpio_led()
In case of error, the function gpio_to_desc() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Diffstat (limited to 'drivers/leds/leds-gpio.c')
-rw-r--r--drivers/leds/leds-gpio.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c
index 7bc53280dbfd..5883dede3423 100644
--- a/drivers/leds/leds-gpio.c
+++ b/drivers/leds/leds-gpio.c
@@ -104,8 +104,8 @@ static int create_gpio_led(const struct gpio_led *template,
104 return ret; 104 return ret;
105 105
106 led_dat->gpiod = gpio_to_desc(template->gpio); 106 led_dat->gpiod = gpio_to_desc(template->gpio);
107 if (IS_ERR(led_dat->gpiod)) 107 if (!led_dat->gpiod)
108 return PTR_ERR(led_dat->gpiod); 108 return -EINVAL;
109 } 109 }
110 110
111 led_dat->cdev.name = template->name; 111 led_dat->cdev.name = template->name;