[PATCH 0/2] Small formatting issue fixes in DTS

[PATCH 0/2] Small formatting issue fixes in DTS

[Patrick Robb]

There are a few formatting issues in DTS which are being
reported by the devtools dts-check-format script. The 2
commits in this series should resolve these issues.

Patrick Robb (2):
  dts: fix incorrect import order for RSS testsuite
  dts: shorten a too long line

 dts/api/testpmd/__init__.py    | 2 +-
 dts/tests/TestSuite_pmd_rss.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.49.0

[PATCH 1/2] dts: fix incorrect import order for RSS testsuite

[Patrick Robb]

Fix the order of the imports on the RSS testsuite such that
the dts check format script is no longer reporting issues.

Fixes: 367e950f6d24 ("dts: add RSS test suite")

Signed-off-by: Patrick Robb <probb@iol.unh.edu>
---
 dts/tests/TestSuite_pmd_rss.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dts/tests/TestSuite_pmd_rss.py b/dts/tests/TestSuite_pmd_rss.py
index 80f6e310ae..f6adf262c3 100644
--- a/dts/tests/TestSuite_pmd_rss.py
+++ b/dts/tests/TestSuite_pmd_rss.py
@@ -20,8 +20,8 @@
     requires_link_topology,
     requires_nic_capability,
 )
-from api.test import verify
 from api.packet import send_packets_and_capture
+from api.test import verify
 from api.testpmd import TestPmd
 from api.testpmd.config import SimpleForwardingModes
 from api.testpmd.types import (
-- 
2.49.0

[PATCH 2/2] dts: shorten a too long line

[Patrick Robb]

Bring a line which is too long under 100 characters, which
is the limit.

Fixes: f6fb8c6238db ("dts: add link check on testpmd port start")

Signed-off-by: Patrick Robb <probb@iol.unh.edu>
---
 dts/api/testpmd/__init__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dts/api/testpmd/__init__.py b/dts/api/testpmd/__init__.py
index b6f5a8f0da..e98f882b14 100644
--- a/dts/api/testpmd/__init__.py
+++ b/dts/api/testpmd/__init__.py
@@ -96,7 +96,7 @@ def _requires_started_ports(func: TestPmdMethod) -> TestPmdMethod:
         func: The :class:`TestPmd` method to decorate.
 
     Raises:
-        InteractiveCommandExecutionError: If the ports has been started but a port link will not come up.
+        InteractiveCommandExecutionError: If ports have been started but the link won't come up.
     """
 
     @functools.wraps(func)
-- 
2.49.0

Re: [PATCH 1/2] dts: fix incorrect import order for RSS testsuite

[Andrew Bailey]

[-- Attachment #1: Type: text/plain, Size: 1025 bytes --]

Reviewed-by: Andrew Bailey <abailey@iol.unh.edu>

On Thu, Nov 13, 2025 at 6:31 PM Patrick Robb <probb@iol.unh.edu> wrote:

> Fix the order of the imports on the RSS testsuite such that
> the dts check format script is no longer reporting issues.
>
> Fixes: 367e950f6d24 ("dts: add RSS test suite")
>
> Signed-off-by: Patrick Robb <probb@iol.unh.edu>
> ---
>  dts/tests/TestSuite_pmd_rss.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dts/tests/TestSuite_pmd_rss.py
> b/dts/tests/TestSuite_pmd_rss.py
> index 80f6e310ae..f6adf262c3 100644
> --- a/dts/tests/TestSuite_pmd_rss.py
> +++ b/dts/tests/TestSuite_pmd_rss.py
> @@ -20,8 +20,8 @@
>      requires_link_topology,
>      requires_nic_capability,
>  )
> -from api.test import verify
>  from api.packet import send_packets_and_capture
> +from api.test import verify
>  from api.testpmd import TestPmd
>  from api.testpmd.config import SimpleForwardingModes
>  from api.testpmd.types import (
> --
> 2.49.0
>
>

[-- Attachment #2: Type: text/html, Size: 1504 bytes --]

Re: [PATCH 2/2] dts: shorten a too long line

[Andrew Bailey]

[-- Attachment #1: Type: text/plain, Size: 1076 bytes --]

Reviewed-by: Andrew Bailey <abailey@iol.unh.edu>


On Thu, Nov 13, 2025 at 6:31 PM Patrick Robb <probb@iol.unh.edu> wrote:

> Bring a line which is too long under 100 characters, which
> is the limit.
>
> Fixes: f6fb8c6238db ("dts: add link check on testpmd port start")
>
> Signed-off-by: Patrick Robb <probb@iol.unh.edu>
> ---
>  dts/api/testpmd/__init__.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dts/api/testpmd/__init__.py b/dts/api/testpmd/__init__.py
> index b6f5a8f0da..e98f882b14 100644
> --- a/dts/api/testpmd/__init__.py
> +++ b/dts/api/testpmd/__init__.py
> @@ -96,7 +96,7 @@ def _requires_started_ports(func: TestPmdMethod) ->
> TestPmdMethod:
>          func: The :class:`TestPmd` method to decorate.
>
>      Raises:
> -        InteractiveCommandExecutionError: If the ports has been started
> but a port link will not come up.
> +        InteractiveCommandExecutionError: If ports have been started but
> the link won't come up.
>      """
>
>      @functools.wraps(func)
> --
> 2.49.0
>
>

[-- Attachment #2: Type: text/html, Size: 1595 bytes --]