diff options
author | Simon Glass <sjg@chromium.org> | 2021-03-21 18:24:34 +1300 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2021-03-27 16:26:48 +1300 |
commit | acd98611c3ca26f10b997ab09aeacd9f4b91d5f3 (patch) | |
tree | 7d3ba8287d6293f8d0d5d3101ba103dd503403f0 | |
parent | 5ff9fedc9bc02e01d73d57e9c7d7eac9fd6320d4 (diff) | |
download | u-boot-acd98611c3ca26f10b997ab09aeacd9f4b91d5f3.zip u-boot-acd98611c3ca26f10b997ab09aeacd9f4b91d5f3.tar.gz u-boot-acd98611c3ca26f10b997ab09aeacd9f4b91d5f3.tar.bz2 |
dtoc: Improve internal error for Refresh()
Add the node name too so it is easy to see which node failed.
Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r-- | tools/dtoc/fdt.py | 4 | ||||
-rwxr-xr-x | tools/dtoc/test_fdt.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/tools/dtoc/fdt.py b/tools/dtoc/fdt.py index 25ce513..f0d1384 100644 --- a/tools/dtoc/fdt.py +++ b/tools/dtoc/fdt.py @@ -339,8 +339,8 @@ class Node: p = fdt_obj.get_property_by_offset(poffset) prop = self.props.get(p.name) if not prop: - raise ValueError("Internal error, property '%s' missing, " - 'offset %d' % (p.name, poffset)) + raise ValueError("Internal error, node '%s' property '%s' missing, " + 'offset %d' % (self.path, p.name, poffset)) prop.RefreshOffset(poffset) poffset = fdt_obj.next_property_offset(poffset, QUIET_NOTFOUND) diff --git a/tools/dtoc/test_fdt.py b/tools/dtoc/test_fdt.py index 1c3a8a2..72095b0 100755 --- a/tools/dtoc/test_fdt.py +++ b/tools/dtoc/test_fdt.py @@ -209,7 +209,7 @@ class TestNode(unittest.TestCase): del self.node.props['notstring'] with self.assertRaises(ValueError) as e: self.dtb.Refresh() - self.assertIn("Internal error, property 'notstring' missing, offset ", + self.assertIn("Internal error, node '/spl-test' property 'notstring' missing, offset ", str(e.exception)) def testLookupPhandle(self): |