Re: [RFC 0/4] ASoC: Add HDA HDMI codec driver

["Koul, Vinod" <vinod.koul@intel.com>]

On Fri, 2015-10-09 at 13:28 +0100, Vinod Koul wrote:
> Yes, this patch series attempts to add yet another HDMI driver to
> ASoC. This codec appears as HDA codec over HDA link. Although
> the codec reside in display we have a HDA link from audio block
> to this codec. The communication to codec is over HDA link
> 
> Thanks to i915 component infrastructure, we do not need to worry
> about keeping the codec on, this is done by i915 for us.
> 
> Based on discussion with Mark here at ELCE and other attempts by
> various folks on HDMI, I wanted to show on list the stuff we have
> done and discuss and try to see how we converge various attempts
> 
> The driver here only supports stereo and doesn't do multichannel
> just yet, will add later once we converge here. The support for
> using i915 component notification by David will be added later
> on.

Apart from comments from Russell I didn't get any other comments.

So as a solution for many HDMI approaches in ASoC should we converge on usage of
component framework for talking to display and not have hard dependency to display
code. That sounds okay on X86 but I am not sure if other arch's can do this...

Also as discussed with Russell, we should use drivers/video/hdmi.c for HDMI
infoframes and sound/core/pcm_drm_eld.c for eld parsing. No open coding these :)

Lastly, sink properties should be the properties of the connected device as read from
ELD. That seems okay to me, other please chime in

I will send updated patchset based on above.

Thanks
-- 
~Vinod