<div dir="ltr">Hi Stephen,<div><br></div><div>I have a whole changelist in the cover letter (PATCH 0/2). Please take a look at it.</div><div><br></div><div>And about the codec dummy things.. I do that only because the codec driver's doc say so.</div>
<div><br></div><div>Thank you,</div><div>Nicolin<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 21, 2013 at 12:17 AM, Stephen Warren <span dir="ltr"><<a href="mailto:swarren@wwwdotorg.org" target="_blank">swarren@wwwdotorg.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 08/19/2013 10:32 PM, Nicolin Chen wrote:<br>
> This patch implements a device-tree-only machine driver for Freescale<br>
> i.MX series Soc. It works with spdif_transmitter/spdif_receiver and<br>
> fsl_spdif.c drivers.<br>
><br>
> Signed-off-by: Nicolin Chen <<a href="mailto:b42378@freescale.com">b42378@freescale.com</a>><br>
> ---<br>
>  .../devicetree/bindings/sound/imx-audio-spdif.txt  |   29 +++++<br>
<br>
</div>What changed between v8 and v9?<br>
<br>
> diff --git a/Documentation/devicetree/bindings/sound/imx-audio-spdif.txt b/Documentation/devicetree/bindings/sound/imx-audio-spdif.txt<br>
<div class="im"><br>
> +Optional properties:<br>
> +<br>
> +  - spdif-transmitter : The phandle of the spdif-transmitter dummy codec<br>
> +<br>
> +  - spdif-receiver : The phandle of the spdif-receiver dummy codec<br>
> +<br>
> +* Note: At least one of these two properties should be set in the DT binding.<br>
<br>
</div>I still don't think "dummy CODEC" is an appropriate thing to have in DT.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
Alsa-devel mailing list<br>
<a href="mailto:Alsa-devel@alsa-project.org">Alsa-devel@alsa-project.org</a><br>
<a href="http://mailman.alsa-project.org/mailman/listinfo/alsa-devel" target="_blank">http://mailman.alsa-project.org/mailman/listinfo/alsa-devel</a><br>
</div></div></blockquote></div><br></div></div></div>