mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-18 22:14:16 +00:00
memory: tegra20-emc: 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. Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Link: https://lore.kernel.org/r/20240816-cleanup-h-of-node-put-memory-v2-9-9eed0ee16b78@linaro.org Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
This commit is contained in:
parent
2bc48f6971
commit
818902cb74
1 changed files with 2 additions and 5 deletions
|
@ -410,7 +410,6 @@ static int cmp_timings(const void *_a, const void *_b)
|
||||||
static int tegra_emc_load_timings_from_dt(struct tegra_emc *emc,
|
static int tegra_emc_load_timings_from_dt(struct tegra_emc *emc,
|
||||||
struct device_node *node)
|
struct device_node *node)
|
||||||
{
|
{
|
||||||
struct device_node *child;
|
|
||||||
struct emc_timing *timing;
|
struct emc_timing *timing;
|
||||||
int child_count;
|
int child_count;
|
||||||
int err;
|
int err;
|
||||||
|
@ -428,15 +427,13 @@ static int tegra_emc_load_timings_from_dt(struct tegra_emc *emc,
|
||||||
|
|
||||||
timing = emc->timings;
|
timing = emc->timings;
|
||||||
|
|
||||||
for_each_child_of_node(node, child) {
|
for_each_child_of_node_scoped(node, child) {
|
||||||
if (of_node_name_eq(child, "lpddr2"))
|
if (of_node_name_eq(child, "lpddr2"))
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
err = load_one_timing_from_dt(emc, timing++, child);
|
err = load_one_timing_from_dt(emc, timing++, child);
|
||||||
if (err) {
|
if (err)
|
||||||
of_node_put(child);
|
|
||||||
return err;
|
return err;
|
||||||
}
|
|
||||||
|
|
||||||
emc->num_timings++;
|
emc->num_timings++;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue