[PATCH 14/14] drm/i915/dp_mst: Use link.{min/max}_bpp_x16

[Kandpal, Suraj]

> -----Original Message-----
> From: Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>
> Sent: Tuesday, December 17, 2024 3:03 PM
> To: intel-gfx at lists.freedesktop.org
> Cc: intel-xe at lists.freedesktop.org; Kandpal, Suraj <suraj.kandpal at intel.com>;
> jani.nikula at linux.intel.com; Deak, Imre <imre.deak at intel.com>
> Subject: [PATCH 14/14] drm/i915/dp_mst: Use link.{min/max}_bpp_x16
> 
> The link.{min/max}_bpp_x16 is already set in crtc_state, use that while
> computing link config for MST.
> 
> Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>

LGTM,
Reviewed-by: Suraj Kandpal <suraj.kandpal at intel.com>

> ---
>  drivers/gpu/drm/i915/display/intel_dp_mst.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> index 689fbd6bcf9b..a9d9d7694acb 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> @@ -428,15 +428,8 @@ static int
> mst_stream_dsc_compute_link_config(struct intel_dp *intel_dp,
> 
>  	crtc_state->pipe_bpp = max_bpp;
> 
> -	max_compressed_bpp =
> intel_dp_dsc_sink_max_compressed_bpp(connector,
> -								  crtc_state,
> -								  max_bpp /
> 3);
> -	max_compressed_bpp = min(max_compressed_bpp,
> -				 fxp_q4_to_int(limits->link.max_bpp_x16));
> -
> -	min_compressed_bpp =
> intel_dp_dsc_sink_min_compressed_bpp(crtc_state);
> -	min_compressed_bpp = max(min_compressed_bpp,
> -				 fxp_q4_to_int_roundup(limits-
> >link.min_bpp_x16));
> +	max_compressed_bpp = fxp_q4_to_int(limits->link.max_bpp_x16);
> +	min_compressed_bpp = fxp_q4_to_int_roundup(limits-
> >link.min_bpp_x16);
> 
>  	drm_dbg_kms(display->drm, "DSC Sink supported compressed min
> bpp %d compressed max bpp %d\n",
>  		    min_compressed_bpp, max_compressed_bpp);
> --
> 2.45.2