diff options
| author | Jinjie Ruan <ruanjinjie@huawei.com> | 2024-08-22 14:25:46 +0800 |
|---|---|---|
| committer | Lee Jones <lee@kernel.org> | 2024-10-02 13:46:34 +0100 |
| commit | 9b216717248e1acf270d76174faa2f8e4a789459 (patch) | |
| tree | 2ee43d8bba3bd17fea5d3fdb57696434d56092aa /drivers/video | |
| parent | 0d580d99749e759b62dc8e28f511310e9235da7a (diff) | |
| download | lwn-9b216717248e1acf270d76174faa2f8e4a789459.tar.gz lwn-9b216717248e1acf270d76174faa2f8e4a789459.zip | |
backlight: 88pm860x_bl: Simplify with scoped for each OF child loop
Use scoped for_each_child_of_node_scoped() when iterating over device
nodes to make code a bit simpler.
Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Link: https://lore.kernel.org/r/20240822062546.3490076-1-ruanjinjie@huawei.com
Signed-off-by: Lee Jones <lee@kernel.org>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/backlight/88pm860x_bl.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c index 25e409bbb1a2..720b5ada7fe8 100644 --- a/drivers/video/backlight/88pm860x_bl.c +++ b/drivers/video/backlight/88pm860x_bl.c @@ -151,7 +151,7 @@ static int pm860x_backlight_dt_init(struct platform_device *pdev, struct pm860x_backlight_data *data, char *name) { - struct device_node *nproot, *np; + struct device_node *nproot; int iset = 0; nproot = of_get_child_by_name(pdev->dev.parent->of_node, "backlights"); @@ -159,14 +159,13 @@ static int pm860x_backlight_dt_init(struct platform_device *pdev, dev_err(&pdev->dev, "failed to find backlights node\n"); return -ENODEV; } - for_each_child_of_node(nproot, np) { + for_each_child_of_node_scoped(nproot, np) { if (of_node_name_eq(np, name)) { of_property_read_u32(np, "marvell,88pm860x-iset", &iset); data->iset = PM8606_WLED_CURRENT(iset); of_property_read_u32(np, "marvell,88pm860x-pwm", &data->pwm); - of_node_put(np); break; } } |