edtlib: Express Node.matching_compat and Node.binding_path as @Property

This simplifies the code and makes it clearer that both properties are
defined in terms of the Binding object matched to a given DT node.

Signed-off-by: Grzegorz Swiderski <[email protected]>

Author: 57300

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

@@ -1019,8 +1019,6 @@ def __init__(
         # Public, some of which are initialized properly later:
         self.edt: 'EDT' = edt
         self.dep_ordinal: int = -1
-        self.matching_compat: Optional[str] = None
-        self.binding_path: Optional[str] = None
         self.compats: List[str] = compats
         self.ranges: List[Range] = []
         self.regs: List[Register] = []
@@ -1139,6 +1137,20 @@ def read_only(self) -> bool:
         "See the class docstring"
         return "read-only" in self._node.props
 
+    @property
+    def matching_compat(self) -> Optional[str]:
+        "See the class docstring"
+        if self._binding:
+            return self._binding.compatible
+        return None
+
+    @property
+    def binding_path(self) -> Optional[str]:
+        "See the class docstring"
+        if self._binding:
+            return self._binding.path
+        return None
+
     @property
     def aliases(self) -> List[str]:
         "See the class docstring"
@@ -1258,12 +1270,9 @@ def __repr__(self) -> str:
         return f"<Node {self.path} in '{self.edt.dts_path}', {binding}>"
 
     def _init_binding(self) -> None:
-        # Initializes Node.matching_compat, Node._binding, and
-        # Node.binding_path.
-        #
-        # Node._binding holds the data from the node's binding file, in the
-        # format returned by PyYAML (plain Python lists, dicts, etc.), or None
-        # if the node has no binding.
+        # Initializes Node._binding. It holds data from the node's binding file,
+        # in the format returned by PyYAML (plain Python lists, dicts, etc.), or
+        # None if the node has no binding.
 
         # This relies on the parent of the node having already been
         # initialized, which is guaranteed by going through the nodes in
@@ -1295,8 +1304,6 @@ def _init_binding(self) -> None:
                     else:
                         continue
 
-                self.binding_path = binding.path
-                self.matching_compat = compat
                 self._binding = binding
                 return
         else:
@@ -1307,13 +1314,10 @@ def _init_binding(self) -> None:
             binding_from_parent = self._binding_from_parent()
             if binding_from_parent:
                 self._binding = binding_from_parent
-                self.binding_path = self._binding.path
-                self.matching_compat = self._binding.compatible
-
                 return
 
         # No binding found
-        self._binding = self.binding_path = self.matching_compat = None
+        self._binding = None
 
     def _binding_from_properties(self) -> None:
         # Sets up a Binding object synthesized from the properties in the node.
@@ -1354,8 +1358,6 @@ def _binding_from_properties(self) -> None:
             raw['properties'][name] = pp
 
         # Set up Node state.
-        self.binding_path = None
-        self.matching_compat = None
         self.compats = []
         self._binding = Binding(None, {}, raw=raw, require_compatible=False)