[alsa-devel] [PATCH] ASoC: cAVS: add device_link to HDMI audio

[Pierre-Louis Bossart]

>> 2. if this is really a generic issue then shouldn't it be fixed for all users of the
>> iDISP link? Why stop at the HDAudio machine driver?
> 
> I will submit other patches for other machine drivers. Which HDAudio machine
> driver do you mean?

if you plan on updating other drivers, then by all means let's use a 
common set of helper functions. I see you have follow-up patches on 
github already, let's avoid copy/paste if possible if the same 
functionality is needed in multiple places.

>>
>> 3. we already have the component model to deal with interaction between
>> i915 and audio, now we are adding a second layer. That looks clunky.
> 
> The component model between i915 and audio is used to communicate
> between display driver and audio driver. This patch is used to setup the
> consumer and supplier relationship.

consumer of what? it's very vague for people who haven't really looked 
into the details.