[Intel-gfx] [PATCH 10/47] drm/i915: pipe and planes should be disabled on haswell_crtc_mode_set

Lespiau, Damien damien.lespiau at intel.com
Fri Oct 5 15:15:38 CEST 2012


On Fri, Oct 5, 2012 at 1:53 PM, Paulo Zanoni <przanoni at gmail.com> wrote:
> 2012/10/4 Lespiau, Damien <damien.lespiau at intel.com>:
>> On Tue, Oct 2, 2012 at 9:51 PM, Paulo Zanoni <przanoni at gmail.com> wrote:
>>> @@ -5356,8 +5361,6 @@ static int haswell_crtc_mode_set(struct drm_crtc *crtc,
>>>
>>>         haswell_set_pipeconf(crtc, adjusted_mode, dither);
>>>
>>> -       intel_wait_for_vblank(dev, pipe);
>>> -
>>
>> I guess this hunk could go into a separate commit.
>
> My way of thinking is that since the pipe is disabled there's no
> reason to wait for vblank here.

Oh, sure I was just saying that it could deserve a separate commit
from the one adding the warning, but bikeshedding really.

-- 
Damien



More information about the Intel-gfx mailing list