On Wed, Apr 18, 2012 at 11:36:08PM +0200, Daniel Vetter wrote: > On Fri, Apr 13, 2012 at 06:24:38PM +0100, Chris Wilson wrote: > > From: Jesse Barnes <jbarnes at virtuousgeek.org> > > > > PCH PLLs aren't required for outputs on the CPU, so we shouldn't just > > treat them as part of the pipe. > > > > So split the code out and manage PCH PLLs separately, allocating them > > when needed or trying to re-use existing PCH PLL setups when the timings > > match. > > > > v2: add num_pch_pll field to dev_priv (Daniel) > > don't NULL the pch_pll pointer in disable or DPMS will fail (Jesse) > > put register offsets in pll struct (Chris) > > > > v3: Decouple enable/disable of PLLs from get/put. > > > > Fixes https://bugs.freedesktop.org/show_bug.cgi?id=44309 > > > > Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org> > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk> > > Patch applied to dinq with Jesse's irc reviewed-by and tested-by, thanks > for the patch. QA reported that this blows up: https://bugs.freedesktop.org/process_bug.cgi I've taken this patch out of dinq again. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
