On Mon, Mar 13, 2017 at 9:33 AM, Daniel Vetter daniel@ffwll.ch wrote:
On Tue, Jan 31, 2017 at 10:36:42PM +0100, Takashi Iwai wrote:
Hi,
the following patches enable DisplayPort Audio on Cherrytrail machines when applied on top of my topic/intel-lpe-audio branch. Tests of DP audio were run on Dell Wyse 3040. The regression test were performed on Baytrail (Compute Stick) and Cherrytrail (Zotac PI330) in HDMI mode. On Cherrytrail there were no issues changing between HDMI and DP connectors dynamically.
Could you i915 people review and give ACK if they are OK? The changes in drm/i915 side are fairly trivial, so I'd like to take them through sound git tree once after I receive your ACKs.
Changes since RFC:
- reordered and squashed patches
- clean-up of register definitions and offsets (based on feedback from Jani/Ville)
- unmute amp for both HDMI and DP unconditionally
- mute amp on invalid ELD (unplug)
- remove test for chicken bit which seems to have no effect in hardware
- cosmetic edits to make checkpatch happy
- change i915 notification argument to pass the plataform device instead
Most of hard work in this patchset has been done by Pierre, so all credits go to him.
Please build the htmldocs and fix the new fallout these patches create:
$ make DOCBOOKS="" htmldocs
0day should be reporting these (if it scans your mailing list), but there's been a hiccup recently.
Good docs matter and all that ...
Apparently this is still not fixed, I applied a fixup patch now from someone else. Tsk. -Daniel