@@ -474,16 +474,15 @@ tegra_emc_find_node_by_ram_code(struct tegra_emc *emc)
ram_code = tegra_read_ram_code();
- of_node_get(dev->of_node);
- for (np = of_find_node_by_name(dev->of_node, "emc-tables"); np;
- np = of_find_node_by_name(np, "emc-tables")) {
+ for_each_child_of_node(dev->of_node, np) {
+ if (!of_node_name_eq(np, "emc-tables"))
+ continue;
err = of_property_read_u32(np, "nvidia,ram-code", &value);
if (err || value != ram_code) {
struct device_node *lpddr2_np;
bool cfg_mismatches = false;
- of_node_get(np);
- lpddr2_np = of_find_node_by_name(np, "lpddr2");
+ lpddr2_np = of_get_child_by_name(np, "lpddr2");
if (lpddr2_np) {
const struct lpddr2_info *info;
According to the yaml file [1] and the dts files [2-4], the "emc-tables" node is a child of a node with the property "nvidia,use-ram-code", and the "lpddr2" node is a child of the "emc-tables" node. Thus utilize the for_each_child_of_node() macro and of_get_child_by_name() instead of of_find_node_by_name() to simplify the code. This bug was found by an experimental verification tool that I am developing. [1]: Documentation/devicetree/bindings/memory-controllers/nvidia,tegra20-emc.yaml [2]: arch/arm/boot/dts/nvidia/tegra20-acer-a500-picasso.dts [3]: arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts [4]: arch/arm/boot/dts/nvidia/tegra20-paz00.dts Signed-off-by: Joe Hattori <joe@pf.is.s.u-tokyo.ac.jp> --- drivers/memory/tegra/tegra20-emc.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)