[Intel-gfx] [PATCH 3/3] drm/i915: Fix checking of pixel multiplier on SDVOX

[Chris Wilson]

On Sun, Jun 09, 2013 at 09:48:11PM +0200, Daniel Vetter wrote:
> On Sun, Jun 9, 2013 at 7:23 PM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> > The pixel multiplier is only stored in SDVOX on i915g/i915gm, so we need
> > to use the value retreived from the card for pipe_config and restrict
> > the cross check to i915g/i915gm.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> 
> Failing this cross check was a fallout from the incosistent ordering
> of the get_pipe_config vs. encoder->get_config calls in setup_hw_state
> vs. modeset_check_state. It should be fixed in latest dinq, at least
> it seemed to on my g33 and i915g.

It is not fixed as of:

commit 22e407d749a418b4bb4cc93ef76e0429a9f83c82
Author: Ville Syrjälä <ville.syrjala at linux.intel.com>
Date:   Fri Jun 7 18:52:24 2013 +0300

    drm/i915: Make g4x_fixup_plane() operational again
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre