diff options
author | Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> | 2021-08-27 10:32:46 +0530 |
---|---|---|
committer | Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> | 2021-08-30 12:01:42 +0530 |
commit | e9292b533691784f46eeb9bae522ca7a8710c920 (patch) | |
tree | 803173fb96e313a9d1286e16cb76327d31b9e8fe | |
parent | 925655c30f4dcc0fa750ba7f827776509f8b9700 (diff) |
tests/kms_setmode: Handle unsupported modes for invalid crtc test
Unsupported modes in crtc need to return proper error in order
to report back that applied config was failed or not. Thus
previously added ENOENT error but later decided to remove
crtc_supports_mode check which is also possible solution.
Test affected (fail -> pass after change) :
igt@kms_setmode@invalid-clone-single-[crtc|crtc-stealing]
invalid-clone-single-crtc-stealing still fails which is not
related and will be handled with different follow up patch.
Changes since V1:
- Replaced previous logic with crtc_supports_mode check removal
Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com>
-rw-r--r-- | tests/kms_setmode.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c index 89220b83e..80665204c 100644 --- a/tests/kms_setmode.c +++ b/tests/kms_setmode.c @@ -569,9 +569,6 @@ int __test_crtc_config(struct crtc_config *crtcs, int crtc_count, for (int i = 0; i < crtc->cconfs->connector->count_modes; i++) { uint32_t *ids; - if (!crtc_supports_mode(crtc, &crtc->cconfs->connector->modes[i])) - continue; - crtc->mode = crtc->cconfs->connector->modes[i]; get_crtc_config_str(crtc, str_buf[base], sizeof(str_buf[base])); |