Re: [Intel-gfx] [PATCH v2] drm/i915/hdmi: Prune modes that require HDMI2.1 FRL

2022-07-27 Thread Shankar, Uma



> -Original Message-
> From: Intel-gfx  On Behalf Of Ankit
> Nautiyal
> Sent: Thursday, July 21, 2022 2:17 PM
> To: intel-gfx@lists.freedesktop.org
> Subject: [Intel-gfx] [PATCH v2] drm/i915/hdmi: Prune modes that require 
> HDMI2.1
> FRL
> 
> HDMI2.1 requires some higher resolution video modes to be enumerated only if
> HDMI2.1 Fixed Rate Link (FRL) is supported.
> Current platforms do not support FRL transmission so prune modes that require
> HDMI2.1 FRL.
> 
> v2: Fixed the condition to check for dotclock > 600.
> Return MODE_CLOCK_HIGH as mode status.

Pushed the change to drm-intel-next. Thanks for the patch and reviews.

Regards,
Uma Shankar

> Signed-off-by: Ankit Nautiyal 
> Reviewed-by: Arun R Murthy  (v1)
> ---
>  drivers/gpu/drm/i915/display/intel_hdmi.c | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c
> b/drivers/gpu/drm/i915/display/intel_hdmi.c
> index ebd91aa69dd2..a88f589351fa 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdmi.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdmi.c
> @@ -2001,6 +2001,15 @@ intel_hdmi_mode_valid(struct drm_connector
> *connector,
>   clock *= 2;
>   }
> 
> + /*
> +  * HDMI2.1 requires higher resolution modes like 8k60, 4K120 to be
> +  * enumerated only if FRL is supported. Current platforms do not support
> +  * FRL so prune the higher resolution modes that require doctclock more
> +  * than 600MHz.
> +  */
> + if (clock > 60)
> + return MODE_CLOCK_HIGH;
> +
>   ycbcr_420_only = drm_mode_is_420_only(>display_info,
> mode);
> 
>   status = intel_hdmi_mode_clock_valid(connector, clock, has_hdmi_sink,
> ycbcr_420_only);
> --
> 2.25.1



[Intel-gfx] [PATCH v2] drm/i915/hdmi: Prune modes that require HDMI2.1 FRL

2022-07-21 Thread Ankit Nautiyal
HDMI2.1 requires some higher resolution video modes to be enumerated
only if HDMI2.1 Fixed Rate Link (FRL) is supported.
Current platforms do not support FRL transmission so prune modes that
require HDMI2.1 FRL.

v2: Fixed the condition to check for dotclock > 600.
Return MODE_CLOCK_HIGH as mode status.

Signed-off-by: Ankit Nautiyal 
Reviewed-by: Arun R Murthy  (v1)
---
 drivers/gpu/drm/i915/display/intel_hdmi.c | 9 +
 1 file changed, 9 insertions(+)

diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c 
b/drivers/gpu/drm/i915/display/intel_hdmi.c
index ebd91aa69dd2..a88f589351fa 100644
--- a/drivers/gpu/drm/i915/display/intel_hdmi.c
+++ b/drivers/gpu/drm/i915/display/intel_hdmi.c
@@ -2001,6 +2001,15 @@ intel_hdmi_mode_valid(struct drm_connector *connector,
clock *= 2;
}
 
+   /*
+* HDMI2.1 requires higher resolution modes like 8k60, 4K120 to be
+* enumerated only if FRL is supported. Current platforms do not support
+* FRL so prune the higher resolution modes that require doctclock more
+* than 600MHz.
+*/
+   if (clock > 60)
+   return MODE_CLOCK_HIGH;
+
ycbcr_420_only = drm_mode_is_420_only(>display_info, mode);
 
status = intel_hdmi_mode_clock_valid(connector, clock, has_hdmi_sink, 
ycbcr_420_only);
-- 
2.25.1