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 <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) >
Reviewed-by: Vinod Govindapillai <[email protected]> > 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; >
