<div dir="ltr">This looks good, and I can confirm dts-check-format.sh is passing with this patch.<div><br></div><div>Reviewed-by: Patrick Robb</div><div>Tested-by: Patrick Robb</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 26, 2024 at 10:09 AM Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com">luca.vizzarro@arm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The "remove redundant test suite" removed an unused test suite and some<br>
dead code with it. Some dead code which references now-removed symbols,<br>
remained though. This removes this code, therefore fixing the related<br>
mypy errors.<br>
<br>
Fixes: e3ab9dd5cd5d ("dts: remove redundant test suite")<br>
<br>
Signed-off-by: Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com" target="_blank">luca.vizzarro@arm.com</a>><br>
Reviewed-by: Paul Szczepanek <<a href="mailto:paul.szczepanek@arm.com" target="_blank">paul.szczepanek@arm.com</a>><br>
---<br>
 dts/framework/testbed_model/node.py | 26 --------------------------<br>
 1 file changed, 26 deletions(-)<br>
<br>
diff --git a/dts/framework/testbed_model/node.py b/dts/framework/testbed_model/node.py<br>
index 85144f6f4e..c1844ecd5d 100644<br>
--- a/dts/framework/testbed_model/node.py<br>
+++ b/dts/framework/testbed_model/node.py<br>
@@ -14,8 +14,6 @@<br>
 """<br>
<br>
 from abc import ABC<br>
-from ipaddress import IPv4Interface, IPv6Interface<br>
-from typing import Union<br>
<br>
 from framework.config import (<br>
     OS,<br>
@@ -192,30 +190,6 @@ def _setup_hugepages(self) -> None:<br>
                 self.config.hugepages.force_first_numa,<br>
             )<br>
<br>
-    def configure_port_state(self, port: Port, enable: bool = True) -> None:<br>
-        """Enable/disable `port`.<br>
-<br>
-        Args:<br>
-            port: The port to enable/disable.<br>
-            enable: :data:`True` to enable, :data:`False` to disable.<br>
-        """<br>
-        self.main_session.configure_port_state(port, enable)<br>
-<br>
-    def configure_port_ip_address(<br>
-        self,<br>
-        address: Union[IPv4Interface, IPv6Interface],<br>
-        port: Port,<br>
-        delete: bool = False,<br>
-    ) -> None:<br>
-        """Add an IP address to `port` on this node.<br>
-<br>
-        Args:<br>
-            address: The IP address with mask in CIDR format. Can be either IPv4 or IPv6.<br>
-            port: The port to which to add the address.<br>
-            delete: If :data:`True`, will delete the address from the port instead of adding it.<br>
-        """<br>
-        self.main_session.configure_port_ip_address(address, port, delete)<br>
-<br>
     def close(self) -> None:<br>
         """Close all connections and free other resources."""<br>
         if self.main_session:<br>
-- <br>
2.43.0<br>
<br>
</blockquote></div>