diff options
| author | Mark Yacoub <markyacoub@google.com> | 2021-03-23 11:08:51 -0400 |
|---|---|---|
| committer | Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> | 2021-08-31 09:55:22 -0400 |
| commit | 062230e67a0c5e1f5ac50c215185b379af02b115 (patch) | |
| tree | 3f26478bf86993cadaa2fd8599451a2458960f60 /tests/kms_atomic_transition.c | |
| parent | b7de28791dd197d01de1512ab5960c6162564815 (diff) | |
tests/kms_atomic_transition: Enable Primary Plane on Enabling CRTC
It is common to find drivers that require the primary plane enabled
whenever the CRTC is (almost all drm_tiny drivers and amdgpu need this).
Nevertheless, kms_atomic_transition does not consider this case and, for
this reason, fails in hardware that has the aforementioned restriction.
Since this is not part of the test, this commit ensures that the test
setup all plane params before committing anything. This commit was
tested on: Delbin, Volteer, Raven, and VKMS.
Changes since V1:
- Update commit message
- Rebase
Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Nicholas Choi <Nicholas.Choi@amd.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Mark Yacoub <markyacoub@chromium.org>
Cc: Melissa Wen <melissa.srw@gmail.com>
Cc: Arkadiusz Hiler <arek@hiler.eu>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Reviewed-By: Nicholas Choi <Nicholas.Choi@amd.com>
Diffstat (limited to 'tests/kms_atomic_transition.c')
| -rw-r--r-- | tests/kms_atomic_transition.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c index cef6187e..bc0a1c81 100644 --- a/tests/kms_atomic_transition.c +++ b/tests/kms_atomic_transition.c @@ -174,7 +174,7 @@ wm_setup_plane(data_t *data, enum pipe pipe, continue; if (!mask || !(parms[i].mask & mask)) { - if (plane->values[IGT_PLANE_FB_ID]) { + if (plane->values[IGT_PLANE_FB_ID] && plane->type != DRM_PLANE_TYPE_PRIMARY) { igt_plane_set_fb(plane, NULL); planes_set_up++; } @@ -521,10 +521,10 @@ run_transition_test(data_t *data, enum pipe pipe, igt_output_t *output, igt_output_set_pipe(output, pipe); } - igt_display_commit2(&data->display, COMMIT_ATOMIC); - setup_parms(data, pipe, mode, &data->fb, &data->argb_fb, &data->sprite_fb, parms, &iter_max); + igt_display_commit2(&data->display, COMMIT_ATOMIC); + /* * In some configurations the tests may not run to completion with all * sprite planes lit up at 4k resolution, try decreasing width/size of secondary |