mbox series

[v2,0/3] ALSA: hda: add link_power op to hdac_bus_ops

Message ID 20210205184630.1938761-1-kai.vehmanen@linux.intel.com (mailing list archive)
Headers show
Series ALSA: hda: add link_power op to hdac_bus_ops | expand

Message

Kai Vehmanen Feb. 5, 2021, 6:46 p.m. UTC 
Here's the 2nd iteration of the patch to improve link management
granularity between controllers using HDA extended bus and HDA
codec drivers.

Based on feedback to V1, this now adds a new link_power op.
This allows to decouple HDA codec drivers from the hdac_ext core.

I also included two patches to SOF to show how this interface is
used in SOF.

Kai Vehmanen (3):
  ALSA: hda: add link_power op to hdac_bus_ops
  ASoC: SOF: Intel: hda: use hdac_ext fine-grained link management
  ASoC: SOF: Intel: hda: release display power at link_power

 include/sound/hdaudio.h             | 14 ++++-------
 include/sound/hdaudio_ext.h         |  2 ++
 sound/hda/ext/hdac_ext_controller.c | 37 +++++++++++++++++++++++++++++
 sound/hda/hdac_bus.c                | 23 ++++++++++++++++++
 sound/hda/hdac_controller.c         | 14 +++++++++++
 sound/soc/sof/intel/hda-bus.c       | 33 ++++++++++++++++++++++++-
 sound/soc/sof/intel/hda.h           |  3 ++-
 7 files changed, 115 insertions(+), 11 deletions(-)


base-commit: 387740a5e9532db13dfb577bd4b8d1aa496487ab

Comments

Takashi Iwai Feb. 6, 2021, 12:24 p.m. UTC | #1 
On Fri, 05 Feb 2021 19:46:27 +0100,
Kai Vehmanen wrote:
> 
> Here's the 2nd iteration of the patch to improve link management
> granularity between controllers using HDA extended bus and HDA
> codec drivers.
> 
> Based on feedback to V1, this now adds a new link_power op.
> This allows to decouple HDA codec drivers from the hdac_ext core.
> 
> I also included two patches to SOF to show how this interface is
> used in SOF.
> 
> Kai Vehmanen (3):
>   ALSA: hda: add link_power op to hdac_bus_ops
>   ASoC: SOF: Intel: hda: use hdac_ext fine-grained link management
>   ASoC: SOF: Intel: hda: release display power at link_power

The patch set looks good to me.  If Mark can give an ack, I'll merge
them through my tree later.


thanks,

Takashi
Mark Brown Feb. 8, 2021, 11:11 a.m. UTC | #2 
On Sat, Feb 06, 2021 at 01:24:38PM +0100, Takashi Iwai wrote:
> Kai Vehmanen wrote:

> > Kai Vehmanen (3):
> >   ALSA: hda: add link_power op to hdac_bus_ops
> >   ASoC: SOF: Intel: hda: use hdac_ext fine-grained link management
> >   ASoC: SOF: Intel: hda: release display power at link_power

> The patch set looks good to me.  If Mark can give an ack, I'll merge
> them through my tree later.

Yeah, looks good to me too

Acked-by: Mark Brown <broonie@kernel.org>
Takashi Iwai Feb. 8, 2021, 2:58 p.m. UTC | #3 
On Mon, 08 Feb 2021 12:11:20 +0100,
Mark Brown wrote:
> 
> On Sat, Feb 06, 2021 at 01:24:38PM +0100, Takashi Iwai wrote:
> > Kai Vehmanen wrote:
> 
> > > Kai Vehmanen (3):
> > >   ALSA: hda: add link_power op to hdac_bus_ops
> > >   ASoC: SOF: Intel: hda: use hdac_ext fine-grained link management
> > >   ASoC: SOF: Intel: hda: release display power at link_power
> 
> > The patch set looks good to me.  If Mark can give an ack, I'll merge
> > them through my tree later.
> 
> Yeah, looks good to me too
> 
> Acked-by: Mark Brown <broonie@kernel.org>

OK, thanks.  Now I applied all three patches to for-next branch.


Takashi