Re: [PATCH 24/50] drm/i915/dp: Pass mode clock to dsc_throughput_quirk_max_bpp_x16()

["Govindapillai, Vinod" <vinod.govindapillai@intel.com>]

On Thu, 2025-11-27 at 19:49 +0200, Imre Deak wrote:
> Prepare for follow-up changes using
> dsc_throughput_quirk_max_bpp_x16()
> without an intel_crtc_state pointer.
> 
> Signed-off-by: Imre Deak <imre.deak@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 

Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>

> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 54a037fcf5111..193d9c2079347 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -2616,11 +2616,8 @@ int intel_dp_dsc_compute_config(struct
> intel_dp *intel_dp,
>  
>  static int
>  dsc_throughput_quirk_max_bpp_x16(const struct intel_connector
> *connector,
> -				 const struct intel_crtc_state
> *crtc_state)
> +				 int mode_clock)
>  {
> -	const struct drm_display_mode *adjusted_mode =
> -		&crtc_state->hw.adjusted_mode;
> -
>  	if (!connector->dp.dsc_throughput_quirk)
>  		return INT_MAX;
>  
> @@ -2640,7 +2637,7 @@ dsc_throughput_quirk_max_bpp_x16(const struct
> intel_connector *connector,
>  	 * smaller than the YUV422/420 value, but let's not depend
> on this
>  	 * assumption.
>  	 */
> -	if (adjusted_mode->crtc_clock <
> +	if (mode_clock <
>  	    min(connector-
> >dp.dsc_branch_caps.overall_throughput.rgb_yuv444,
>  		connector-
> >dp.dsc_branch_caps.overall_throughput.yuv422_420) / 2)
>  		return INT_MAX;
> @@ -2704,7 +2701,8 @@ intel_dp_compute_config_link_bpp_limits(struct
> intel_connector *connector,
>  
>  		max_link_bpp_x16 = min(max_link_bpp_x16,
> fxp_q4_from_int(dsc_max_bpp));
>  
> -		throughput_max_bpp_x16 =
> dsc_throughput_quirk_max_bpp_x16(connector, crtc_state);
> +		throughput_max_bpp_x16 =
> +			dsc_throughput_quirk_max_bpp_x16(connector,
> adjusted_mode->crtc_clock);
>  		if (throughput_max_bpp_x16 < max_link_bpp_x16) {
>  			max_link_bpp_x16 = throughput_max_bpp_x16;
>