diff options
| author | Frank Rowand <frank.rowand@am.sony.com> | 2016-10-28 23:26:31 -0700 |
|---|---|---|
| committer | Rob Herring <robh@kernel.org> | 2016-11-15 09:17:17 -0600 |
| commit | 4458db4c9b7363a60d9ad07275fb2eea4ecf3b52 (patch) | |
| tree | 0b15a593cf04da5e9580099e1780a032f75baed5 /drivers/of | |
| parent | d9181b20a83bd8de207e0cb54be499d9880b00e3 (diff) | |
| download | op-kernel-dev-4458db4c9b7363a60d9ad07275fb2eea4ecf3b52.zip op-kernel-dev-4458db4c9b7363a60d9ad07275fb2eea4ecf3b52.tar.gz | |
of: Move setting of pointer to beside test for non-null
Value of pointer was calculated in an earlier block than
where it was used. Move it down into the block where it
is used, immediately before where is is checked to be valid.
Signed-off-by: Frank Rowand <frank.rowand@am.sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/resolver.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index eb78010..53353cc 100644 --- a/drivers/of/resolver.c +++ b/drivers/of/resolver.c @@ -320,8 +320,6 @@ int of_resolve_phandles(struct device_node *overlay) overlay_symbols = NULL; overlay_fixups = NULL; - tree_symbols = of_find_node_by_path("/__symbols__"); - for_each_child_of_node(overlay, child) { if (!of_node_cmp(child->name, "__symbols__")) overlay_symbols = child; @@ -334,6 +332,7 @@ int of_resolve_phandles(struct device_node *overlay) goto out; } + tree_symbols = of_find_node_by_path("/__symbols__"); if (!tree_symbols) { pr_err("no symbols in root of device tree.\n"); err = -EINVAL; |
