diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2019-09-04 14:58:28 +1000 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2019-09-04 14:58:28 +1000 |
commit | 60e0db3d65a1218b0d5a29474e769f28a18e3ca6 (patch) | |
tree | 65c4aa55e50d43fb3b581b2160ff3600439974b1 | |
parent | 95ce19c14064170e2a3998e487892f07fc5cbdac (diff) | |
download | device-tree-compiler-60e0db3d65a1218b0d5a29474e769f28a18e3ca6.tar.gz |
Ignore phandle properties in /aliasesv1.5.1
The 'alias_paths' check verifies that each property in /aliases is a valid
path to another node. However this can cans false positives trees where
the /aliases node has a phandle property, which isn't in this format but
is allowed. In particular this situation can be common with trees dumped
from some real OF systems (which typically generate a phandle for every
node).
Special case this to avoid the spurious error.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r-- | checks.c | 5 |
1 files changed, 5 insertions, 0 deletions
@@ -691,6 +691,11 @@ static void check_alias_paths(struct check *c, struct dt_info *dti, return; for_each_property(node, prop) { + if (streq(prop->name, "phandle") + || streq(prop->name, "linux,phandle")) { + continue; + } + if (!prop->val.val || !get_node_by_path(dti->dt, prop->val.val)) { FAIL_PROP(c, dti, node, prop, "aliases property is not a valid node (%s)", prop->val.val); |