In intel_crt.c (VGA displays) and intel_lvds.c (old laptop displays), both files were using a legacy boolean flag crtc_state->bw_constrained
to guess if the display link was bandwidth limited. To fix this, replaced the bw_constrained boolean checks with direct comparisons against the max_link_bpp_x16 fixed-point API. Signed-off-by: George Karagounis <[email protected]> --- drivers/gpu/drm/i915/display/intel_crt.c | 5 ++--- drivers/gpu/drm/i915/display/intel_lvds.c | 6 +++--- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_crt.c b/drivers/gpu/drm/i915/display/intel_crt.c index 5b8968197fbc..3da9547f2c6e 100644 --- a/drivers/gpu/drm/i915/display/intel_crt.c +++ b/drivers/gpu/drm/i915/display/intel_crt.c @@ -459,10 +459,9 @@ static int hsw_crt_compute_config(struct intel_atomic_state *state, /* LPT FDI RX only supports 8bpc. */ if (HAS_PCH_LPT(display)) { - /* TODO: Check crtc_state->max_link_bpp_x16 instead of bw_constrained */ - if (crtc_state->bw_constrained && crtc_state->pipe_bpp < 24) { + if (crtc_state->max_link_bpp_x16 < fxp_q4_from_int(24)) { drm_dbg_kms(display->drm, - "LPT only supports 24bpp\n"); + "LPT only supports 24bpp\n"); return -EINVAL; } diff --git a/drivers/gpu/drm/i915/display/intel_lvds.c b/drivers/gpu/drm/i915/display/intel_lvds.c index 872753478cf2..6bbd13ab5da7 100644 --- a/drivers/gpu/drm/i915/display/intel_lvds.c +++ b/drivers/gpu/drm/i915/display/intel_lvds.c @@ -443,10 +443,10 @@ static int intel_lvds_compute_config(struct intel_atomic_state *state, else lvds_bpp = 6*3; - /* TODO: Check crtc_state->max_link_bpp_x16 instead of bw_constrained */ - if (lvds_bpp != crtc_state->pipe_bpp && !crtc_state->bw_constrained) { + if (lvds_bpp != crtc_state->pipe_bpp && + crtc_state->max_link_bpp_x16 >= fxp_q4_from_int(lvds_bpp)) { drm_dbg_kms(display->drm, - "forcing display bpp (was %d) to LVDS (%d)\n", + "forcing display bpp (was %d) to LVDS (%d)\n", crtc_state->pipe_bpp, lvds_bpp); crtc_state->pipe_bpp = lvds_bpp; } -- 2.55.0
