Re: [PATCH i-g-t] tests/kms_pipe_crc_basic: Don't suspend the machine if

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 04, 2016 at 04:25:41PM +0200, Marius Vlad wrote:
> suspend-read-crc-pipe will perform a suspend and then skip the test in case the
> pipe is not present on the platform. Skip the test before doing the suspend.
> 
> 
> Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx>
> ---
>  tests/kms_pipe_crc_basic.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c
> index 3c51ba0..dbb2919 100644
> --- a/tests/kms_pipe_crc_basic.c
> +++ b/tests/kms_pipe_crc_basic.c
> @@ -271,6 +271,7 @@ igt_main
>  			test_read_crc(&data, i, TEST_SEQUENCE | TEST_NONBLOCK);
>  
>  		igt_subtest_f("suspend-read-crc-pipe-%c", 'A'+i) {
> +			igt_skip_on(i >= data.display.n_pipes);

We should place this into an igt_fixture at the top of the loop. Hitting
igt_skip outside of any testcase means we'll skip all the subsequent ones
without doing any work.

Otoh this is a bit a bikeshed, so Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
either way.
-Daniel

>  			igt_system_suspend_autoresume();
>  
>  			test_read_crc(&data, i, 0);
> -- 
> 2.7.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux