On Wed, May 30, 2012 at 01:56:01PM +0100, Chris Wilson wrote: > On Wed, 30 May 2012 14:52:26 +0200, Daniel Vetter <daniel.vetter at ffwll.ch> wrote: > > Jesse extracted this nice helper in his i9xx_crtc_mode_set refactor, > > but we have the identical code in ironlake_ccrtc_mode_set. And that > > function is huge, so extracting some code full of magic numbers is > > always nice. > > > > Noticed while trying to get a handle on our dp clock code. > > > > Signed-Off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> Queued for -next, thanks for the review. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
