Re: [Intel-gfx] [PATCH] drm/i915: don't chnage the original mode in dp_mode_fixup

2012-05-30 Thread Daniel Vetter
On Wed, May 30, 2012 at 11:58:43AM +0100, Chris Wilson wrote: > On Wed, 30 May 2012 12:28:04 +0200, Daniel Vetter > wrote: > > We should only frob adjusted_mode. This is in preparation of > > a massive patch by Laurent Pinchart to make the mode argument > > const. > > > > The only thing we actua

Re: [Intel-gfx] [PATCH] drm/i915: don't chnage the original mode in dp_mode_fixup

2012-05-30 Thread Chris Wilson
On Wed, 30 May 2012 12:28:04 +0200, Daniel Vetter wrote: > We should only frob adjusted_mode. This is in preparation of > a massive patch by Laurent Pinchart to make the mode argument > const. > > The only thing we actually touch is mode->clock, but only if > it's a panel. And in that case we al

[Intel-gfx] [PATCH] drm/i915: don't chnage the original mode in dp_mode_fixup

2012-05-30 Thread Daniel Vetter
We should only frob adjusted_mode. This is in preparation of a massive patch by Laurent Pinchart to make the mode argument const. The only thing we actually touch is mode->clock, but only if it's a panel. And in that case we also set adjusted_mode->clock to the same value. All the generic code alr