Re: [PATCH 0/7] DisplayPort audio support on Cherrytrail

From: Takashi Iwai <tiwai@suse.de>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	intel-gfx <intel-gfx@lists.freedesktop.org>,
	Rakesh A Ughreja <rakesh.a.ughreja@intel.com>
Subject: Re: [PATCH 0/7] DisplayPort audio support on Cherrytrail
Date: Fri, 31 Mar 2017 08:40:03 +0200	[thread overview]
Message-ID: <s5hr31egda4.wl-tiwai@suse.de> (raw)
In-Reply-To: <CAKMK7uF9ovHW7EcThmcw9CZbA-Nzn++K19vqJ8rq7a0zdz7vcA@mail.gmail.com>

On Fri, 31 Mar 2017 08:29:55 +0200,
Daniel Vetter wrote:
> 
> 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.

Ah, sorry, this felt into crack during my vacation.
Good that it was already fixed.  Thanks.

Takashi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx