leds: 88pm860x: Add missing of_node_put()
of_find_node_by_name() returns a node pointer with refcount incremented, use of_node_put() on it when done. of_find_node_by_name() will call of_node_put() against from parameter, thus we also need to call of_node_get(from) before calling of_find_node_by_name(). Signed-off-by: Axel Lin <axel.lin@ingics.com> Signed-off-by: Bryan Wu <cooloney@gmail.com>
This commit is contained in:
parent
ef754e88e3
commit
61d4eb2724
|
@ -128,8 +128,10 @@ static void pm860x_led_set(struct led_classdev *cdev,
|
||||||
static int pm860x_led_dt_init(struct platform_device *pdev,
|
static int pm860x_led_dt_init(struct platform_device *pdev,
|
||||||
struct pm860x_led *data)
|
struct pm860x_led *data)
|
||||||
{
|
{
|
||||||
struct device_node *nproot = pdev->dev.parent->of_node, *np;
|
struct device_node *nproot, *np;
|
||||||
int iset = 0;
|
int iset = 0;
|
||||||
|
|
||||||
|
nproot = of_node_get(pdev->dev.parent->of_node);
|
||||||
if (!nproot)
|
if (!nproot)
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
nproot = of_find_node_by_name(nproot, "leds");
|
nproot = of_find_node_by_name(nproot, "leds");
|
||||||
|
@ -145,6 +147,7 @@ static int pm860x_led_dt_init(struct platform_device *pdev,
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
of_node_put(nproot);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
#else
|
#else
|
||||||
|
|
Loading…
Reference in New Issue