On Tue, 2019-10-01 at 18:46 +0300, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > When adding the max plane size checks to the .mode_valid() hooks > I naturally forgot about MST. Take care of that one as well. Reviewed-by: José Roberto de Souza <jose.souza@xxxxxxxxx> > > Cc: Manasi Navare <manasi.d.navare@xxxxxxxxx> > Cc: Sean Paul <sean@xxxxxxxxxx> > Cc: José Roberto de Souza <jose.souza@xxxxxxxxx> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Fixes: 2d20411e25a3 ("drm/i915: Don't advertise modes that exceed the > max plane size") > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_dp_mst.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c > b/drivers/gpu/drm/i915/display/intel_dp_mst.c > index df4f35c10a69..2203be28ea01 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c > +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c > @@ -424,6 +424,7 @@ static enum drm_mode_status > intel_dp_mst_mode_valid(struct drm_connector *connector, > struct drm_display_mode *mode) > { > + struct drm_i915_private *dev_priv = to_i915(connector->dev); > struct intel_connector *intel_connector = > to_intel_connector(connector); > struct intel_dp *intel_dp = intel_connector->mst_port; > int max_dotclk = to_i915(connector->dev)->max_dotclk_freq; > @@ -451,7 +452,7 @@ intel_dp_mst_mode_valid(struct drm_connector > *connector, > if (mode_rate > max_rate || mode->clock > max_dotclk) > return MODE_CLOCK_HIGH; > > - return MODE_OK; > + return intel_mode_valid_max_plane_size(dev_priv, mode); > } > > static struct drm_encoder *intel_mst_atomic_best_encoder(struct > drm_connector *connector, _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx