summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2019-09-04 14:58:28 +1000
committerDavid Gibson <david@gibson.dropbear.id.au>2019-09-04 14:58:28 +1000
commit60e0db3d65a1218b0d5a29474e769f28a18e3ca6 (patch)
tree65c4aa55e50d43fb3b581b2160ff3600439974b1
parent95ce19c14064170e2a3998e487892f07fc5cbdac (diff)
downloaddevice-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.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/checks.c b/checks.c
index d7986ee..756f0fa 100644
--- a/checks.c
+++ b/checks.c
@@ -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);