Re: [igt-dev] [PATCH i-g-t] kms_content_protection: add check for DPMS off state

Gupta, Anshuman
2020-01-03T10:45:25Z

On 1/3/2020 1:41 PM, Ramalingam C wrote:
> During DPMS off HDCP state should be DESIRED.
> Lets add a check for that.
Not related to this patch but DPMS is connector property
and test name is DPMS but sub-test is setting crtc property.
slightly confusing. Leaving up to you to change it.
>
> Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> cc: Anshuman Gupta <anshuman.gupta@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> ---
> tests/kms_content_protection.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index 8b361fdefb2e..eb6cd9ec50b1 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -513,6 +513,11 @@ static void test_content_protection_on_output(igt_output_t *output,
> IGT_CRTC_ACTIVE, 0);
> igt_display_commit2(display, s);
>
> + /* Wait for HDCP to be disabled, before crtc off */
> + ret = wait_for_prop_value(output, CP_DESIRED,
> + KERNEL_DISABLE_TIME_ALLOWED_MSEC);
> + igt_assert_f(ret, "Content Protection not cleared\n");
> +
> igt_pipe_set_prop_value(display, pipe,
> IGT_CRTC_ACTIVE, 1);
> igt_display_commit2(display, s);
>