Imre Deak <imre.deak@xxxxxxxxx> writes: > If need to enable the panel fitter, the crtc timings have to be > programmed according to the panel's native (fixed) mode. This isn't the > case atm, since after the encoder changes adjusted_mode to fixed > mode the crtc_* timing fields of adjusted_mode will stay at their original > non-native values that the user passed in. This results in a corrupted > output. > > One exception is when we have a second pass of computing encoder configs > due to bandwidth limitation, since then we'll set adjusted_mode.crtc_* > fields to the fixed mode values set in the first pass; so in this case > things will work out. > > Fix this by updating the adjusted_mode.crtc_* fields when we set the > fixed panel mode. > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> Daniel, the commit that regressed seem to be 135c81b8c3c9a70d7b55758c9c2a247a4abb7b64 instead of 21d8a4756af5fdf4a42e79a77cf3b6f52678d443. Reviewed-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_panel.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c > index a43c33b..913cb9d 100644 > --- a/drivers/gpu/drm/i915/intel_panel.c > +++ b/drivers/gpu/drm/i915/intel_panel.c > @@ -50,6 +50,8 @@ intel_fixed_panel_mode(struct drm_display_mode *fixed_mode, > adjusted_mode->vtotal = fixed_mode->vtotal; > > adjusted_mode->clock = fixed_mode->clock; > + > + drm_mode_set_crtcinfo(adjusted_mode, 0); > } > > /* adjusted_mode has been preset to be the panel's fixed mode */ > -- > 1.8.3.2 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
