All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup
@ 2018-01-04 14:07 Marta Lofstedt
  2018-01-04 23:23 ` Pandiyan, Dhinakaran
  2018-01-05  8:12 ` Lofstedt, Marta
  0 siblings, 2 replies; 4+ messages in thread
From: Marta Lofstedt @ 2018-01-04 14:07 UTC (permalink / raw)
  To: intel-gfx

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104260

Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com>
---
 tests/kms_psr_sink_crc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index 83a69f0b..26cf434a 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -532,11 +532,13 @@ int main(int argc, char *argv[])
 	igt_subtest("psr_basic") {
 		setup_test_plane(&data);
 		igt_assert(wait_psr_entry(&data));
+		test_cleanup(&data);
 	}
 
 	igt_subtest("psr_drrs") {
 		setup_test_plane(&data);
 		igt_assert(drrs_disabled(&data));
+		test_cleanup(&data);
 	}
 
 	for (op = PAGE_FLIP; op <= RENDER; op++) {
-- 
2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup
  2018-01-04 14:07 [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup Marta Lofstedt
@ 2018-01-04 23:23 ` Pandiyan, Dhinakaran
  2018-01-05  8:12 ` Lofstedt, Marta
  1 sibling, 0 replies; 4+ messages in thread
From: Pandiyan, Dhinakaran @ 2018-01-04 23:23 UTC (permalink / raw)
  To: Lofstedt, Marta; +Cc: intel-gfx


Including a line about the atomic update failure in the commit message
would be nice.

Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>

On Thu, 2018-01-04 at 16:07 +0200, Marta Lofstedt wrote:
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104260
> 
> Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com>
> ---
>  tests/kms_psr_sink_crc.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
> index 83a69f0b..26cf434a 100644
> --- a/tests/kms_psr_sink_crc.c
> +++ b/tests/kms_psr_sink_crc.c
> @@ -532,11 +532,13 @@ int main(int argc, char *argv[])
>  	igt_subtest("psr_basic") {
>  		setup_test_plane(&data);
>  		igt_assert(wait_psr_entry(&data));
> +		test_cleanup(&data);
>  	}
>  
>  	igt_subtest("psr_drrs") {
>  		setup_test_plane(&data);
>  		igt_assert(drrs_disabled(&data));
> +		test_cleanup(&data);
>  	}
>  
>  	for (op = PAGE_FLIP; op <= RENDER; op++) {
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup
  2018-01-04 14:07 [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup Marta Lofstedt
  2018-01-04 23:23 ` Pandiyan, Dhinakaran
@ 2018-01-05  8:12 ` Lofstedt, Marta
  1 sibling, 0 replies; 4+ messages in thread
From: Lofstedt, Marta @ 2018-01-05  8:12 UTC (permalink / raw)
  To: intel-gfx

Arek,
I didn't get any PW for this maybe because I sent it together with a HACK patch to get a test run on BAT.

Should I rename both patches and re-send to the list?

/Marta

> -----Original Message-----
> From: Lofstedt, Marta
> Sent: Thursday, January 4, 2018 4:07 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Latvala, Petri <petri.latvala@intel.com>; Lofstedt, Marta
> <marta.lofstedt@intel.com>
> Subject: [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and
> psr_drrs need test cleanup
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104260
> 
> Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com>
> ---
>  tests/kms_psr_sink_crc.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c index
> 83a69f0b..26cf434a 100644
> --- a/tests/kms_psr_sink_crc.c
> +++ b/tests/kms_psr_sink_crc.c
> @@ -532,11 +532,13 @@ int main(int argc, char *argv[])
>  	igt_subtest("psr_basic") {
>  		setup_test_plane(&data);
>  		igt_assert(wait_psr_entry(&data));
> +		test_cleanup(&data);
>  	}
> 
>  	igt_subtest("psr_drrs") {
>  		setup_test_plane(&data);
>  		igt_assert(drrs_disabled(&data));
> +		test_cleanup(&data);
>  	}
> 
>  	for (op = PAGE_FLIP; op <= RENDER; op++) {
> --
> 2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup
@ 2018-01-05  9:04 Marta Lofstedt
  0 siblings, 0 replies; 4+ messages in thread
From: Marta Lofstedt @ 2018-01-05  9:04 UTC (permalink / raw)
  To: intel-gfx

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104260

Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com>
---
 tests/kms_psr_sink_crc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index 83a69f0b..26cf434a 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -532,11 +532,13 @@ int main(int argc, char *argv[])
 	igt_subtest("psr_basic") {
 		setup_test_plane(&data);
 		igt_assert(wait_psr_entry(&data));
+		test_cleanup(&data);
 	}
 
 	igt_subtest("psr_drrs") {
 		setup_test_plane(&data);
 		igt_assert(drrs_disabled(&data));
+		test_cleanup(&data);
 	}
 
 	for (op = PAGE_FLIP; op <= RENDER; op++) {
-- 
2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-01-05  9:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-04 14:07 [PATCH i-g-t 1/2] test/kms_psr_sink_crc - subtests psr_basic and psr_drrs need test cleanup Marta Lofstedt
2018-01-04 23:23 ` Pandiyan, Dhinakaran
2018-01-05  8:12 ` Lofstedt, Marta
2018-01-05  9:04 Marta Lofstedt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.