diff options
author | Swati Sharma <swati2.sharma@intel.com> | 2020-03-13 18:55:56 +0530 |
---|---|---|
committer | Swati Sharma <swati2.sharma@intel.com> | 2020-03-19 19:42:13 +0530 |
commit | cf6d524007ac51a7d5a48503ea3dd5f01fd4ebab (patch) | |
tree | 2d8db820ef4638cf8e85a1a02e4bc980940c7bcc /tests/kms_hdr.c | |
parent | bd2b01af69c9720d54e68a8702a23e4ff3637746 (diff) |
tests/kms_hdr: Made debug info more evident
In present scenario, whenever subtest gets SKIPPED we get debug info
"connector property not found or panel is non-HDR" which is quite vague.
Added igt_info() stating the actual reason of skip. This will help in
bug assessment too.
Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/kms_hdr.c')
-rw-r--r-- | tests/kms_hdr.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c index 677936c63..713725564 100644 --- a/tests/kms_hdr.c +++ b/tests/kms_hdr.c @@ -285,7 +285,7 @@ static void test_bpc_switch(data_t *data, uint32_t flags) valid_tests++; } - igt_require_f(valid_tests, "No connector found with MAX BPC connector property\n"); + igt_require_f(valid_tests, "No connector found with max_bpc connector property\n"); } static bool cta_block(const char *edid_ext) @@ -609,24 +609,26 @@ static void test_hdr(data_t *data, const char *test_name, uint32_t flags) * set MAX_BPC property to 10bpc prior to setting * HDR metadata property. Therefore, checking. */ - if (!has_max_bpc(output)) - continue; - - if (!has_hdr(output)) + if (!has_max_bpc(output) || !has_hdr(output)) { + igt_info("%s connector not found with HDR metadata/max_bpc connector property\n", output->name); continue; + } - if (!is_panel_hdr(data, output)) + if (!is_panel_hdr(data, output)) { + igt_info("Panel attached via %s connector is non-HDR\n", output->name); continue; + } igt_info("HDR %s test execution on %s\n", test_name, output->name); if (flags & TEST_NONE || flags & TEST_DPMS || flags & TEST_SUSPEND) test_static_toggle(data, output, flags); if (flags & TEST_SWAP) test_static_swap(data, output); + valid_tests++; } - igt_require_f(valid_tests, "No connector found with HDR metadata/MAX BPC connector property (or) panel is Non-HDR\n"); + igt_require_f(valid_tests, "No connector found with HDR metadata/max_bpc connector property (or) panel is non-HDR\n"); } igt_main |