> -----Original Message-----
> From: Nautiyal, Ankit K <ankit.k.nautiyal@xxxxxxxxx>
> Sent: Tuesday, December 17, 2024 3:03 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: intel-xe@xxxxxxxxxxxxxxxxxxxxx; Kandpal, Suraj <suraj.kandpal@xxxxxxxxx>;
> jani.nikula@xxxxxxxxxxxxxxx; Deak, Imre <imre.deak@xxxxxxxxx>
> 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@xxxxxxxxx>
LGTM,
Reviewed-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx>
> ---
> 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
