Re: [PATCH] ASoC: cAVS: add device_link to HDMI audio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux