dtlib: allow dangling aliases with DT(..., force=True)

As a first step towards being more forgiving on invalid inputs, allow
string-valued aliases properties that do not point to valid nodes when
the user requests permissiveness.

Signed-off-by: Martí Bolívar <[email protected]>

Author: mbolivar-nordic

scripts/dts/python-devicetree/src/devicetree/dtlib.py

@@ -1684,8 +1684,15 @@ def _register_aliases(self):
                     _err("/aliases: alias property name '{}' should include "
                          "only characters from [0-9a-z-]".format(prop.name))
 
-                # Property.to_path() already checks that the node exists
-                alias2node[prop.name] = prop.to_path()
+                # Property.to_path() checks that the node exists, has
+                # the right type, etc. Swallow errors for invalid
+                # aliases with self._force.
+                try:
+                    alias2node[prop.name] = prop.to_path()
+                except DTError:
+                    if self._force:
+                        continue
+                    raise
 
         self.alias2node = alias2node
 

scripts/dts/python-devicetree/tests/test_dtlib.py

@@ -24,13 +24,15 @@
 #   - to run a particular test function or functions, use
 #     '-k test_function_pattern_goes_here'
 
-def parse(dts, include_path=()):
-    '''Parse a DTS string 'dts', using the given include path.'''
+def parse(dts, include_path=(), **kwargs):
+    '''Parse a DTS string 'dts', using the given include path.
+
+    Any kwargs are passed on to DT().'''
 
     fd, path = tempfile.mkstemp(prefix='pytest-', suffix='.dts')
     try:
         os.write(fd, dts.encode('utf-8'))
-        return dtlib.DT(path, include_path)
+        return dtlib.DT(path, include_path, **kwargs)
     finally:
         os.close(fd)
         os.unlink(path)
@@ -2250,3 +2252,13 @@ def test_misc():
 / {
 };
 """)
+
+def test_dangling_alias():
+    dt = parse('''
+/dts-v1/;
+
+/ {
+	aliases { foo = "/missing"; };
+};
+''', force=True)
+    assert dt.get_node('/aliases').props['foo'].to_string() == '/missing'