drm/i915: correctly order the ring init sequence
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 7 Aug 2012 07:54:14 +0000 (09:54 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 26 Aug 2012 22:00:40 +0000 (15:00 -0700)
commit57ecc93ce680b1ace1f9e79d588dabe32353202c
tree610d327eb665b5df956ed78f9d9e0373d422600e
parent1a475b7af63febbef784d0d2ccb6bdbb119d0245
drm/i915: correctly order the ring init sequence

commit 0d8957c8a90bbb5d34fab9a304459448a5131e06 upstream.

We may only start to set up the new register values after having
confirmed that the ring is truely off. Otherwise the hw might lose the
newly written register values. This is caught later on in the init
sequence, when we check whether the register writes have stuck.

Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50522
Tested-by: Yang Guang <guang.a.yang@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/gpu/drm/i915/intel_ringbuffer.c