Re: [PATCH v3 6/6] ASoC: zx: add zx296702 hdmi codec

From: Mark Brown <broonie@kernel.org>
To: Jun Nie <jun.nie@linaro.org>
Cc: alsa-devel@alsa-project.org, Lars-Peter Clausen <lars@metafoo.de>,
	"wan.zhijun" <wan.zhijun@zte.com.cn>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Shawn Guo <shawn.guo@linaro.org>
Subject: Re: [PATCH v3 6/6] ASoC: zx: add zx296702 hdmi codec
Date: Tue, 2 Jun 2015 19:13:26 +0100	[thread overview]
Message-ID: <20150602181326.GB14071@sirena.org.uk> (raw)
In-Reply-To: <CABymUCNvM=k4tWJutpzoK-VFvKhBC4EGjh_OzWYMMRppUqvSOA@mail.gmail.com>

[-- Attachment #1.1: Type: text/plain, Size: 963 bytes --]

On Mon, Jun 01, 2015 at 09:42:34AM +0800, Jun Nie wrote:

> So I need implement dai link in machine audio card driver, while not
> use simple card and device tree initialization. In this way, I surely
> can embed HDMI audio codec in HDMI driver. Is that right?

I'm sorry, I don't entirely follow what you're saying here.

> I had try to initialize audio all from dts, so need a HDMI codec dt
> node to connect with DAI. However, I cannot find a way to create an
> independent HDMI CODEC dt node because it is brought up from HDMI
> driver. If my above understanding is correct, a machine level DAI link
> shall resolve this issue with dropping simple card and DAI link in
> dts. Thanks for your comments!

If the HDMI encoder is simple I'd expect it to be possible to use it
with simple-card.  If it isn't then we can look at why.

> BTW: HDMI driver is far from mature, so you did not see HDMI function
> implementation in this patch serial.

Sure, no problem.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]