diff options
| author | Olof Johansson <olof@lixom.net> | 2013-12-11 16:11:42 -0800 |
|---|---|---|
| committer | Bryan Wu <cooloney@gmail.com> | 2014-01-27 17:28:51 -0800 |
| commit | 33fc94506bae75341f083b79e6c2a0ff086a8810 (patch) | |
| tree | 1687f2774aa295bd3675672abeedf81244baac03 | |
| parent | fc1aee038b609dc33067ca9dd477b16ea893cae0 (diff) | |
leds: pwm: Remove a warning on non-DT platforms
This removes a warning on non-DT-enabled platforms:
drivers/leds/leds-pwm.c: In function 'led_pwm_create_of':
drivers/leds/leds-pwm.c:88:22: warning: unused variable 'node'
Really caused by the local variable that is assigned to and then never
used. Just do away with the local var, it's not needed.
Technically this code path can never be entered without DT enabled,
since there's an earlier check about number of children in the calling
function, but the compiler can't see that.
Signed-off-by: Olof Johansson <olof@lixom.net>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
| -rw-r--r-- | drivers/leds/leds-pwm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c index 3fbd28e99b2..605047428b5 100644 --- a/drivers/leds/leds-pwm.c +++ b/drivers/leds/leds-pwm.c @@ -87,11 +87,10 @@ static inline size_t sizeof_pwm_leds_priv(int num_leds) static int led_pwm_create_of(struct platform_device *pdev, struct led_pwm_priv *priv) { - struct device_node *node = pdev->dev.of_node; struct device_node *child; int ret; - for_each_child_of_node(node, child) { + for_each_child_of_node(pdev->dev.of_node, child) { struct led_pwm_data *led_dat = &priv->leds[priv->num_leds]; led_dat->cdev.name = of_get_property(child, "label", |