Re: [PATCH] test/py: Run simple dm commands without checking

[Michal Simek]

On 7/7/22 12:59, Michal Simek wrote:

From: Michal Simek 

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 
Signed-off-by: Michal Simek 
---

  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

Re: [PATCH] test/py: Run simple dm commands without checking

[Michal Simek]

Hi Simon,

On 7/12/22 12:58, Simon Glass wrote:

Hi Michal,

On Thu, 7 Jul 2022 at 05:00, Michal Simek  wrote:


From: Michal Simek 

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 
Signed-off-by: Michal Simek 


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 

You can drop the 'response' var if you are not using it.


Someone can parse it in future if they like.

M

Re: [PATCH] test/py: Run simple dm commands without checking

[Simon Glass]

Hi Michal,

On Thu, 7 Jul 2022 at 05:00, Michal Simek  wrote:
>
> From: Michal Simek 
>
> 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 
> Signed-off-by: Michal Simek 

Make up your mind :-)

> ---
>
>  test/py/tests/test_dm.py | 8 
>  1 file changed, 8 insertions(+)

Reviewed-by: Simon Glass 

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

[PATCH] test/py: Run simple dm commands without checking

[Michal Simek]

From: Michal Simek 

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 
Signed-off-by: Michal Simek 
---

 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")
-- 
2.36.1