On Thu, 16 Jan 2025, Suraj Kandpal <suraj.kandpal@xxxxxxxxx> wrote: > Use intel_encoder_is_hdmi function which was recently introduced to > see if encoder is HDMI or not. Does this mean we always returned early from intel_hdcp_adjust_hdcp_line_rekeying()? Fixes: ? BR, Jani. > > Signed-off-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx> > --- > 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 7464b44c8bb3..91d5629cd9f9 100644 > --- a/drivers/gpu/drm/i915/display/intel_hdcp.c > +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c > @@ -41,7 +41,7 @@ intel_hdcp_adjust_hdcp_line_rekeying(struct intel_encoder *encoder, > u32 rekey_bit = 0; > > /* Here we assume HDMI is in TMDS mode of operation */ > - if (encoder->type != INTEL_OUTPUT_HDMI) > + if (!intel_encoder_is_hdmi(encoder)) > return; > > if (DISPLAY_VER(display) >= 30) { -- Jani Nikula, Intel