Re: [Intel-gfx] [PATCH 09/81] drm/i915/lvds: ditch ->prepare special case

2012-07-22 Thread Daniel Vetter
On Sun, Jul 22, 2012 at 6:32 PM, Takashi Iwai wrote: > At Sun, 22 Jul 2012 16:52:54 +0200, > Daniel Vetter wrote: >> >> On Wed, Jul 11, 2012 at 04:27:52PM +0200, Daniel Vetter wrote: >> > LVDS is the first output where dpms on/off and prepare/commit don't >> > perfectly match. Now the idea behind

Re: [Intel-gfx] [PATCH 09/81] drm/i915/lvds: ditch ->prepare special case

2012-07-22 Thread Takashi Iwai
At Sun, 22 Jul 2012 16:52:54 +0200, Daniel Vetter wrote: > > On Wed, Jul 11, 2012 at 04:27:52PM +0200, Daniel Vetter wrote: > > LVDS is the first output where dpms on/off and prepare/commit don't > > perfectly match. Now the idea behind this special case seems to be > > that for simple resolution

Re: [Intel-gfx] [PATCH 09/81] drm/i915/lvds: ditch ->prepare special case

2012-07-22 Thread Daniel Vetter
On Wed, Jul 11, 2012 at 04:27:52PM +0200, Daniel Vetter wrote: > LVDS is the first output where dpms on/off and prepare/commit don't > perfectly match. Now the idea behind this special case seems to be > that for simple resolution changes on the LVDS we don't need to stop > the pipe, because (at le

[Intel-gfx] [PATCH 09/81] drm/i915/lvds: ditch ->prepare special case

2012-07-11 Thread Daniel Vetter
LVDS is the first output where dpms on/off and prepare/commit don't perfectly match. Now the idea behind this special case seems to be that for simple resolution changes on the LVDS we don't need to stop the pipe, because (at least on newer chips) we can adjust the panel fitter on the fly. There a