Re: [Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-14 Thread Jani Nikula
On Wed, 14 Jan 2015, Ander Conselvan de Oliveira wrote: > On 01/14/2015 12:47 PM, Jani Nikula wrote: >> On Tue, 13 Jan 2015, Ander Conselvan de Oliveira >> wrote: >>> If QUIRK_PIPEA_FORCE is necessary, intel_sanitize_crtc() might trigger >>> a mode set. In that case, if pipe A is disabled and pi

Re: [Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-14 Thread Chris Wilson
On Wed, Jan 14, 2015 at 12:50:46PM +0200, Ander Conselvan de Oliveira wrote: > On 01/14/2015 12:47 PM, Jani Nikula wrote: > >On Tue, 13 Jan 2015, Ander Conselvan de Oliveira > > wrote: > >>If QUIRK_PIPEA_FORCE is necessary, intel_sanitize_crtc() might trigger > >>a mode set. In that case, if pipe

Re: [Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-14 Thread Ander Conselvan de Oliveira
On 01/14/2015 12:47 PM, Jani Nikula wrote: On Tue, 13 Jan 2015, Ander Conselvan de Oliveira wrote: If QUIRK_PIPEA_FORCE is necessary, intel_sanitize_crtc() might trigger a mode set. In that case, if pipe A is disabled and pipe B is mapped to plane B, that mode set happens before the mapping is

Re: [Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-14 Thread Jani Nikula
On Tue, 13 Jan 2015, Ander Conselvan de Oliveira wrote: > If QUIRK_PIPEA_FORCE is necessary, intel_sanitize_crtc() might trigger > a mode set. In that case, if pipe A is disabled and pipe B is mapped to > plane B, that mode set happens before the mapping is fixed. Due to the > wrong state, the ca

Re: [Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-13 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) -Summary- Platform Delta drm-intel-nightly Series Applied PNV 354/354

[Intel-gfx] [PATCH] drm/i915: Fix all pipe->plane mappings before sanitizing crtc

2015-01-13 Thread Ander Conselvan de Oliveira
If QUIRK_PIPEA_FORCE is necessary, intel_sanitize_crtc() might trigger a mode set. In that case, if pipe A is disabled and pipe B is mapped to plane B, that mode set happens before the mapping is fixed. Due to the wrong state, the call to disable pipe B disables plane A (which is already disabled)