<div dir="ltr">Thanks! applied!</div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 8, 2021 at 11:48 AM Petri Latvala <<a href="mailto:petri.latvala@intel.com">petri.latvala@intel.com</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Dec 08, 2021 at 11:34:40AM -0500, Mark Yacoub wrote: > From: Mark Yacoub <<a href="mailto:markyacoub@google.com" target="_blank">markyacoub@google.com</a>> > > [Why] > flip_vs_cursor_crc and flip_vs_cursor_busy_crc use CRC in debugfs which is > not supported by some drivers. > > [How] > igt_require_pipe_crc in igt_fixture before the subtests start. > > Test: igt@kms_cursor_legacy@flip-vs-cursor{-busy}-crc-{atomic,legacy} on > ChromeOS Jacuzzi (MTK) > > Signed-off-by: Mark Yacoub <<a href="mailto:markyacoub@chromium.org" target="_blank">markyacoub@chromium.org</a>> Reviewed-by: Petri Latvala <<a href="mailto:petri.latvala@intel.com" target="_blank">petri.latvala@intel.com</a>> > --- > tests/kms_cursor_legacy.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c > index d50155e8..47b9b4e1 100644 > --- a/tests/kms_cursor_legacy.c > +++ b/tests/kms_cursor_legacy.c > @@ -1533,6 +1533,8 @@ igt_main > /*Test description for cursor-crc tests*/ > igt_describe("Test will first does a page flip and then cursor update"); > igt_subtest_group { > + igt_fixture > + igt_require_pipe_crc(display.drm_fd); > > igt_subtest("flip-vs-cursor-crc-legacy") > flip_vs_cursor_crc(&display, false); > @@ -1544,6 +1546,8 @@ igt_main > /*Test description for busy-crc tests*/ > igt_describe("this test perform a busy bo update followed by a cursor update"); > igt_subtest_group { > + igt_fixture > + igt_require_pipe_crc(display.drm_fd); > > igt_subtest("flip-vs-cursor-busy-crc-legacy") > flip_vs_cursor_busy_crc(&display, false); > -- > 2.34.1.400.ga245620fadb-goog > </blockquote></div>