On Mon, 16 Jan 2012 21:26:18 +0100, Daniel Vetter wrote: > Keith, does this address your concern and this patch is r-b: Keith or do > we want an > > } else { > I915_WRITE(aud_config, 0); > } > > for paranoia? I think we want this added, just to be sure we set the configuration correctly in all cases; it's easy to imagine moving from DP to HDMI and leaving this bit set. -- keith.packard@intel.com