diff options
author | Saurabh Sengar <saurabh.truth@gmail.com> | 2015-11-23 04:28:16 -0500 |
---|---|---|
committer | Jacek Anaszewski <j.anaszewski@samsung.com> | 2016-01-04 03:57:38 -0500 |
commit | 5a7c536f26186c313e521502f942d5cdaf05b8ee (patch) | |
tree | dd028818f11adf0857f6c78c2b42628ea09f3911 | |
parent | b964c5ba6d52475d70e3550fbb3780ebabedf7db (diff) |
leds: powernv: removing NULL check
No need to explicitly check for pointer to be null,
of_prop_next_string anyways returns NULL, if passed pointer is NULL
and hence loop will continue.
Signed-off-by: Saurabh Sengar <saurabh.truth@gmail.com>
Acked-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
-rw-r--r-- | drivers/leds/leds-powernv.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/leds/leds-powernv.c b/drivers/leds/leds-powernv.c index 1e75e1fe9b72..bc2d76e3be14 100644 --- a/drivers/leds/leds-powernv.c +++ b/drivers/leds/leds-powernv.c | |||
@@ -256,8 +256,6 @@ static int powernv_led_classdev(struct platform_device *pdev, | |||
256 | 256 | ||
257 | for_each_child_of_node(led_node, np) { | 257 | for_each_child_of_node(led_node, np) { |
258 | p = of_find_property(np, "led-types", NULL); | 258 | p = of_find_property(np, "led-types", NULL); |
259 | if (!p) | ||
260 | continue; | ||
261 | 259 | ||
262 | while ((cur = of_prop_next_string(p, cur)) != NULL) { | 260 | while ((cur = of_prop_next_string(p, cur)) != NULL) { |
263 | powernv_led = devm_kzalloc(dev, sizeof(*powernv_led), | 261 | powernv_led = devm_kzalloc(dev, sizeof(*powernv_led), |