On Tue, Aug 27, 2013 at 12:24:09PM +0300, Imre Deak wrote:
> 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>
This regression has been introduced in
commit 21d8a4756af5fdf4a42e79a77cf3b6f52678d443
Author: Daniel Vetter <daniel.vetter@xxxxxxxx>
Date: Fri Jul 12 08:07:30 2013 +0200
drm/i915: fix pfit regression for non-autoscaled resolutions
> ---
> 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
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
