[RFC PATCH] ALSA: hda: call ext hda codec link up/down if available

Takashi Iwai tiwai at suse.de
Thu Feb 4 10:14:28 CET 2021


On Thu, 04 Feb 2021 09:07:09 +0100,
Kai Vehmanen wrote:
> 
> Hi,
> 
> On Thu, 4 Feb 2021, Kai Vehmanen wrote:
> > To support more fine-grained control of link power, this patch
> > implements new helper functions for codec drivers to turn codec links
> > up and down. The HDA common suspend/resume code is modified to use
> > the new functions. This allows to fully reuse the driver code both
> > for plain and extended HDA controllers.
> 
> Takashi and others, this patch is a follow-up to the earlier thread about 
> hdac link management:
> https://mailman.alsa-project.org/pipermail/alsa-devel/2021-January/179715.html
> 
> We have absolutely no calls from sound/pci/hda/ to hdaudio_ext.h before
> this patch, so I'm wondering if this is the right approach to take. 

An obvious drawback by this patch is that it'll make the hda-ext code
always loaded whenever it's enabled, no matter whether it's really
used or not.

Maybe adding a new callback (link) in hdac_bus_ops can help?

> It does seem to be the natural path forward and mimics what was done in 
> e.g. hdac_hdmi. But we want to keep reusing driver code for all HDA 
> implementations, so adding hdac-ext support to codec drivers seem like the 
> best path.

Yes, I find we're heading to the right direction.


thanks,

Takashi


More information about the Alsa-devel mailing list