Re: [PATCH] drm/i915/hdcp: Take force_hdcp14 into account during check_link

["Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>]

On 2/25/2026 12:20 PM, Suraj Kandpal wrote:
> During intel_hdcp_check_link phase we need to take into account
> if we are currently forcing HDCP 1.4 or not. This is because
> we check for HDCP 2.x Link first and only if HDCP 2.x is not being
> used check for HDCP 1.4. With force_hdcp14 in picture we should not
> be going into intel_hdcp2_check_link because of which we may end
> up trying to disable HDCP2.x even if HDCP 1.4 was enabled causing
> a lot of issues while IGT tests this.
>
> Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>

Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>


> ---
>   drivers/gpu/drm/i915/display/intel_hdcp.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
> index 116d2e0af878..892eab4b6f92 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdcp.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
> @@ -2231,7 +2231,7 @@ static void intel_hdcp_check_work(struct work_struct *work)
>   	if (drm_connector_is_unregistered(&connector->base))
>   		return;
>   
> -	if (!intel_hdcp2_check_link(connector))
> +	if (!hdcp->force_hdcp14 && !intel_hdcp2_check_link(connector))
>   		queue_delayed_work(display->wq.unordered, &hdcp->check_work,
>   				   DRM_HDCP2_CHECK_PERIOD_MS);
>   	else if (!intel_hdcp_check_link(connector))