summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Zhang <dingchen.zhang@amd.com>2022-04-26 10:29:04 -0400
committerDavid Zhang <dingchen.zhang@amd.com>2022-04-27 09:13:55 -0400
commita2c91509a4d990ee3c95e5b2baedee974c68de8d (patch)
tree3a73e7f0389ee6284b842934b35270efd7a3ccfb
parent1bc4e7c3759ec8bc50f2633839d8edd23ab426ae (diff)
tests/amdgpu/amd_psr: set visual-confirm default to disabled
[why & how] For CI machine, by default it would directly run test case w/o any manual confirmation. And visual confirm option is to validate the PSR-SU feature as manual approach, which is for debugging purpose and not for CI automated scheme. Pre-set the "visual_confirm" to disable and only to enable the visual confirm by manual. changes in v2: ---------------------- - pre-set visual confirm option from enabled to disabled, and set it as required argument. Cc: Rodrigo Siqueira <rodrigo.siqueira@amd.com> Cc: Harry Wentland <harry.wentland@amd.com> Cc: Leo Li <sunpeng.li@amd.com> Cc: Jay Pillai <aurabindo.pillai@amd.com> Cc: Wayne Lin <wayne.lin@amd.com> Signed-off-by: David Zhang <dingchen.zhang@amd.com> Reviewed-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
-rw-r--r--tests/amdgpu/amd_psr.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/amdgpu/amd_psr.c b/tests/amdgpu/amd_psr.c
index e3a2cb4ad848..8f47e816dab3 100644
--- a/tests/amdgpu/amd_psr.c
+++ b/tests/amdgpu/amd_psr.c
@@ -66,7 +66,7 @@ typedef struct data {
struct {
bool visual_confirm;
} opt = {
- .visual_confirm = true, /* visual confirm debug option */
+ .visual_confirm = false, /* visual confirm debug option */
};
static void draw_color_alpha(igt_fb_t *fb, int x, int y, int w, int h,
@@ -491,7 +491,7 @@ const char *help_str =
" --visual-confirm PSR visual confirm debug option enable\n";
struct option long_options[] = {
- {"visual-confirm", optional_argument, NULL, 'v'},
+ {"visual-confirm", required_argument, NULL, 'v'},
{ 0, 0, 0, 0 }
};