Skip to content

Commit 2ea99da

Browse files
krzkthierryreding
authored andcommitted
clk: tegra: tegra124-emc: Simplify with scoped for each OF child loop
Use scoped for-each loop when iterating over device nodes to make code a bit simpler. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> Reviewed-by: Jon Hunter <jonathanh@nvidia.com> Signed-off-by: Thierry Reding <treding@nvidia.com>
1 parent 1acce02 commit 2ea99da

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

drivers/clk/tegra/clk-tegra124-emc.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,6 @@ static int load_timings_from_dt(struct tegra_clk_emc *tegra,
444444
u32 ram_code)
445445
{
446446
struct emc_timing *timings_ptr;
447-
struct device_node *child;
448447
int child_count = of_get_child_count(node);
449448
int i = 0, err;
450449
size_t size;
@@ -458,12 +457,11 @@ static int load_timings_from_dt(struct tegra_clk_emc *tegra,
458457
timings_ptr = tegra->timings + tegra->num_timings;
459458
tegra->num_timings += child_count;
460459

461-
for_each_child_of_node(node, child) {
460+
for_each_child_of_node_scoped(node, child) {
462461
struct emc_timing *timing = timings_ptr + (i++);
463462

464463
err = load_one_timing_from_dt(tegra, timing, child);
465464
if (err) {
466-
of_node_put(child);
467465
kfree(tegra->timings);
468466
return err;
469467
}

0 commit comments

Comments
 (0)