leds: gpio: simplify gpio_leds_create
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 14 Sep 2016 18:55:06 +0000 (20:55 +0200)
committerJacek Anaszewski <j.anaszewski@samsung.com>
Thu, 15 Sep 2016 14:49:41 +0000 (16:49 +0200)
Definition of np can be moved into the loop as well to simplify
the code a little.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
drivers/leds/leds-gpio.c

index da4aa8e..171ba2f 100644 (file)
@@ -159,7 +159,6 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev)
        struct fwnode_handle *child;
        struct gpio_leds_priv *priv;
        int count, ret;
-       struct device_node *np;
 
        count = device_get_child_node_count(dev);
        if (!count)
@@ -173,6 +172,7 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev)
                struct gpio_led_data *led_dat = &priv->leds[priv->num_leds];
                struct gpio_led led = {};
                const char *state = NULL;
+               struct device_node *np = to_of_node(child);
 
                led.gpiod = devm_get_gpiod_from_child(dev, NULL, child);
                if (IS_ERR(led.gpiod)) {
@@ -181,8 +181,6 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev)
                        goto err;
                }
 
-               np = to_of_node(child);
-
                if (fwnode_property_present(child, "label")) {
                        fwnode_property_read_string(child, "label", &led.name);
                } else {