[igt-dev] [PATCH i-g-t] tests/kms_cdclk: Drop basic subtest

[Shankar, Uma]

> -----Original Message-----
> From: Shankar, Uma
> Sent: Monday, July 5, 2021 8:26 PM
> To: Sharma, Swati2 <swati2.sharma at intel.com>; igt-dev at lists.freedesktop.org
> Subject: RE: [PATCH i-g-t] tests/kms_cdclk: Drop basic subtest
> 
> 
> 
> > -----Original Message-----
> > From: Sharma, Swati2 <swati2.sharma at intel.com>
> > Sent: Monday, July 5, 2021 7:48 PM
> > To: igt-dev at lists.freedesktop.org
> > Cc: Sharma, Swati2 <swati2.sharma at intel.com>; Shankar, Uma
> > <uma.shankar at intel.com>
> > Subject: [PATCH i-g-t] tests/kms_cdclk: Drop basic subtest
> >
> > This drops basic subtest since use case is already getting covered by
> > mode transition subtest.
> >
> > Cc: Uma Shankar <uma.shankar at intel.com>
> > Signed-off-by: Swati Sharma <swati2.sharma at intel.com>
> > ---
> >  tests/kms_cdclk.c | 61
> > +++++------------------------------------------
> >  1 file changed, 6 insertions(+), 55 deletions(-)
> >
> > diff --git a/tests/kms_cdclk.c b/tests/kms_cdclk.c index
> > b2e32d25..d2a9fa5f 100644
> > --- a/tests/kms_cdclk.c
> > +++ b/tests/kms_cdclk.c
> > @@ -26,7 +26,7 @@
> >
> >  #include "igt.h"
> >
> > -IGT_TEST_DESCRIPTION("Test cdclk features : squasher and crawling");
> > +IGT_TEST_DESCRIPTION("Test cdclk features : crawling");
> >
> >  #define HDISPLAY_4K     3840
> >  #define VDISPLAY_4K     2160
> > @@ -34,9 +34,8 @@ IGT_TEST_DESCRIPTION("Test cdclk features : squasher
> > and crawling");
> >
> >  /* Test flags */
> >  enum {
> > -	TEST_BASIC = 1 << 0,
> > -	TEST_PLANESCALING = 1 << 1,
> > -	TEST_MODETRANSITION = 1 << 2,
> > +	TEST_PLANESCALING = 1 << 0,
> > +	TEST_MODETRANSITION = 1 << 1,
> >  };
> >
> >  typedef struct {
> > @@ -132,45 +131,6 @@ static void do_cleanup_display(igt_display_t *dpy)
> >  	igt_display_commit2(dpy, dpy->is_atomic ? COMMIT_ATOMIC :
> > COMMIT_LEGACY);  }
> >
> > -static void test_basic(data_t *data, enum pipe pipe, igt_output_t *output) -{
> > -	igt_display_t *display = &data->display;
> > -	int debugfs_fd = data->debugfs_fd;
> > -	int cdclk_ref, cdclk_new;
> > -	struct igt_fb fb;
> > -	igt_plane_t *primary;
> > -	drmModeModeInfo *mode;
> > -
> > -	do_cleanup_display(display);
> > -	igt_display_reset(display);
> > -
> > -	igt_output_set_pipe(output, pipe);
> > -	mode = get_highres_mode(output);
> > -	igt_require(mode != NULL);
> > -	igt_output_override_mode(output, mode);
> > -
> > -	primary = igt_output_get_plane_type(output,
> > DRM_PLANE_TYPE_PRIMARY);
> > -
> > -	igt_create_color_pattern_fb(display->drm_fd,
> > -				    mode->hdisplay, mode->vdisplay,
> > -				    DRM_FORMAT_XRGB8888,
> > -				    I915_TILING_NONE,
> > -				    0.0, 0.0, 0.0, &fb);
> > -
> > -	igt_plane_set_fb(primary, &fb);
> > -	cdclk_ref = get_current_cdclk_freq(debugfs_fd);
> > -	igt_display_commit_atomic(display,
> > DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
> > -	cdclk_new = get_current_cdclk_freq(debugfs_fd);
> > -	igt_info("CD clock frequency %d -> %d\n", cdclk_ref, cdclk_new);
> > -
> > -	/* cdclk should bump */
> > -	igt_assert_lt(cdclk_ref, cdclk_new);
> > -
> > -	/* cleanup */
> > -	do_cleanup_display(display);
> > -	igt_remove_fb(display->drm_fd, &fb);
> > -}
> > -
> >  static void test_plane_scaling(data_t *data, enum pipe pipe,
> > igt_output_t *output) {
> >  	igt_display_t *display = &data->display; @@ -284,8 +244,6 @@ static
> > void run_cdclk_test(data_t *data, uint32_t flags)
> >  	for_each_pipe_with_valid_output(display, pipe, output) {
> >  		igt_dynamic_f("%s-pipe-%s", output->name,
> > kmstest_pipe_name(pipe))
> >  			if (igt_pipe_connector_valid(pipe, output)) {
> > -				if (flags & TEST_BASIC)
> > -					test_basic(data, pipe, output);
> >  				if (flags & TEST_PLANESCALING)
> >  					test_plane_scaling(data, pipe, output);
> >  				if (flags & TEST_MODETRANSITION)
> > @@ -306,22 +264,15 @@ igt_main
> >  		kmstest_set_vt_graphics_mode();
> >  		data.devid = intel_get_drm_devid(data.drm_fd);
> >  		igt_require_f(hardware_supported(&data),
> > -			      "Hardware doesn't support either squashing "\
> > -			      "or crawling.\n");
> > +			      "Hardware doesn't support crawling.\n");
> 
> We can still use both, whether to use squashing or crawling depends if a PLL ratio is
> required or not. So let this test cover both crawling and squashing.
> 
> With that fixed.
> Acked-by: Uma Shankar <uma.shankar at intel.com>

You also have my:
Reviewed-by: Uma Shankar <uma.shankar at intel.com>

> 
> >  		igt_display_require(&data.display, data.drm_fd);
> >  		igt_display_require_output(&data.display);
> >  	}
> >
> > -	igt_describe("Basic test to validate cdclk frequency change w/o "\
> > -		     "requiring full modeset.");
> > -	igt_subtest_with_dynamic("basic")
> > -		run_cdclk_test(&data, TEST_BASIC);
> > -	igt_describe("Plane scaling test to validate cdclk frequency change w/o "\
> > -		     "requiring full modeset.");
> > +	igt_describe("Plane scaling test to validate cdclk frequency
> > +change");
> >  	igt_subtest_with_dynamic("plane-scaling")
> >  		run_cdclk_test(&data, TEST_PLANESCALING);
> > -	igt_describe("Mode transition (low to high) test to validate cdclk frequency
> > "\
> > -		     "change w/o requiring full modeset.");
> > +	igt_describe("Mode transition (low to high) test to validate cdclk
> > +frequency change");
> >  	igt_subtest_with_dynamic("mode-transition")
> >  		run_cdclk_test(&data, TEST_MODETRANSITION);
> >
> > --
> > 2.25.1