On Wed, Sep 13, 2017 at 03:41:46PM +0100, Chris Wilson wrote: > Quoting Ville Syrjala (2017-09-13 15:08:56) > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > Pass the crtc state to i9xx_enable_pll() and use it rather than > > crtc->config. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > (mutters something about consistency in crtc_state vs pipe_config) Yeah, that's a PITA. I'm thinking we'll try to take care of that with coccinelle or sed once the non-trivial cases are sorted. Also looks like I missed i9xx_set_pll_dividers(). I'll have to fix that one up as well to get the gmch DPLL code cleaned up. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
