mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2025-01-07 13:43:51 +00:00
pmdomain: qcom-cpr: Use helper function for_each_available_child_of_node()
for_each_available_child_of_node() can help to iterate through the device_node, and we don't need to use while loop. Besides, the purpose of the while loop is to find a device_node which fits the condition "child_req_np == ref_np", we can just read the property of "child_np" directly in for_each_available_child_of_node(). No functional change with such conversion. Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com> Link: https://lore.kernel.org/r/20240821034022.27394-2-zhangzekun11@huawei.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
This commit is contained in:
parent
9e5eb7403c
commit
f253f6d922
@ -1054,14 +1054,14 @@ static unsigned long cpr_get_opp_hz_for_req(struct dev_pm_opp *ref,
|
||||
if (!ref_np)
|
||||
goto out_ref;
|
||||
|
||||
do {
|
||||
for_each_available_child_of_node(desc_np, child_np) {
|
||||
of_node_put(child_req_np);
|
||||
child_np = of_get_next_available_child(desc_np, child_np);
|
||||
child_req_np = of_parse_phandle(child_np, "required-opps", 0);
|
||||
} while (child_np && child_req_np != ref_np);
|
||||
|
||||
if (child_np && child_req_np == ref_np)
|
||||
of_property_read_u64(child_np, "opp-hz", &rate);
|
||||
if (child_req_np == ref_np) {
|
||||
of_property_read_u64(child_np, "opp-hz", &rate);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
of_node_put(child_req_np);
|
||||
of_node_put(child_np);
|
||||
|
Loading…
Reference in New Issue
Block a user