[PATCH] test/py: Run simple dm commands without checking
From: Michal Simek michal.simek@xilinx.com
Just to make sure that dm commands can operate. This was the problem on Microblaze in past without fixing manual relocation.
Signed-off-by: Michal Simek michal.simek@xilinx.com Signed-off-by: Michal Simek michal.simek@amd.com ---
test/py/tests/test_dm.py | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/test/py/tests/test_dm.py b/test/py/tests/test_dm.py index 97203b536e17..ea93061fdfa8 100644 --- a/test/py/tests/test_dm.py +++ b/test/py/tests/test_dm.py @@ -33,3 +33,11 @@ def test_dm_static(u_boot_console): response = u_boot_console.run_command('dm drivers') for driver in drivers: assert driver in response + +@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_uclass(u_boot_console): + response = u_boot_console.run_command("dm uclass") + +@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_devres(u_boot_console): + response = u_boot_console.run_command("dm devres")
Hi Michal,
On Thu, 7 Jul 2022 at 05:00, Michal Simek michal.simek@amd.com wrote:
From: Michal Simek michal.simek@xilinx.com
Just to make sure that dm commands can operate. This was the problem on Microblaze in past without fixing manual relocation.
Signed-off-by: Michal Simek michal.simek@xilinx.com Signed-off-by: Michal Simek michal.simek@amd.com
Make up your mind :-)
test/py/tests/test_dm.py | 8 ++++++++ 1 file changed, 8 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org
You can drop the 'response' var if you are not using it.
diff --git a/test/py/tests/test_dm.py b/test/py/tests/test_dm.py index 97203b536e17..ea93061fdfa8 100644 --- a/test/py/tests/test_dm.py +++ b/test/py/tests/test_dm.py @@ -33,3 +33,11 @@ def test_dm_static(u_boot_console): response = u_boot_console.run_command('dm drivers') for driver in drivers: assert driver in response
+@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_uclass(u_boot_console):
- response = u_boot_console.run_command("dm uclass")
+@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_devres(u_boot_console):
- response = u_boot_console.run_command("dm devres")
-- 2.36.1
Regards, Simon
Hi Simon,
On 7/12/22 12:58, Simon Glass wrote:
Hi Michal,
On Thu, 7 Jul 2022 at 05:00, Michal Simek michal.simek@amd.com wrote:
From: Michal Simek michal.simek@xilinx.com
Just to make sure that dm commands can operate. This was the problem on Microblaze in past without fixing manual relocation.
Signed-off-by: Michal Simek michal.simek@xilinx.com Signed-off-by: Michal Simek michal.simek@amd.com
Make up your mind :-)
the patch is in xilinx repo for quite a long time and in the meantime company has changed. That's why two SoB lines.
test/py/tests/test_dm.py | 8 ++++++++ 1 file changed, 8 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org
You can drop the 'response' var if you are not using it.
Someone can parse it in future if they like.
M
On 7/7/22 12:59, Michal Simek wrote:
From: Michal Simek michal.simek@xilinx.com
Just to make sure that dm commands can operate. This was the problem on Microblaze in past without fixing manual relocation.
Signed-off-by: Michal Simek michal.simek@xilinx.com Signed-off-by: Michal Simek michal.simek@amd.com
test/py/tests/test_dm.py | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/test/py/tests/test_dm.py b/test/py/tests/test_dm.py index 97203b536e17..ea93061fdfa8 100644 --- a/test/py/tests/test_dm.py +++ b/test/py/tests/test_dm.py @@ -33,3 +33,11 @@ def test_dm_static(u_boot_console): response = u_boot_console.run_command('dm drivers') for driver in drivers: assert driver in response
+@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_uclass(u_boot_console):
- response = u_boot_console.run_command("dm uclass")
+@pytest.mark.buildconfigspec("cmd_dm") +def test_dm_devres(u_boot_console):
- response = u_boot_console.run_command("dm devres")
Applied. M
participants (2)
-
Michal Simek -
Simon Glass