Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Assign correct hdcp content type

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

LGTM.

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

Please add Fixes tag.

Regards,

Ankit

On 6/22/2023 2:02 PM, Suraj Kandpal wrote:
> Currently hdcp->content_type is being assigned the content_type field
> in drm_connector_state which is wrong and instead it needs to be
> assigned hdcp_content_type field from drm_connector_state
>
> Cc: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
> Signed-off-by: Suraj Kandpal <suraj.kandpal@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 5ed450111f77..34fabadefaf6 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdcp.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
> @@ -2358,7 +2358,7 @@ int intel_hdcp_enable(struct intel_atomic_state *state,
>   	mutex_lock(&dig_port->hdcp_mutex);
>   	drm_WARN_ON(&i915->drm,
>   		    hdcp->value == DRM_MODE_CONTENT_PROTECTION_ENABLED);
> -	hdcp->content_type = (u8)conn_state->content_type;
> +	hdcp->content_type = (u8)conn_state->hdcp_content_type;
>   
>   	if (intel_crtc_has_type(pipe_config, INTEL_OUTPUT_DP_MST)) {
>   		hdcp->cpu_transcoder = pipe_config->mst_master_transcoder;