mbox series

[00/17] ASoC: SOF: Intel: improve SoundWire support for LunarLake

Message ID 20240402151828.175002-1-pierre-louis.bossart@linux.intel.com (mailing list archive)
Headers show
Series ASoC: SOF: Intel: improve SoundWire support for LunarLake | expand

Message

Pierre-Louis Bossart April 2, 2024, 3:18 p.m. UTC
This patchset contains important updates for SoundWire support.

We initially implemented support for multiple amplifiers on different
links using a single HDaudio DMA transfer. To align with the other OS,
the 'aggregation' is now supported by the firmware. This change in
directions has kernel impacts, since we now have multiple HDaudio DMAs
to program and start, but since there are no platforms released so far
there's no end-user impact.

In addition, the behavior in case of xruns is improved by clearing the
PCM states and better handling of the hw_free case.

Note that the hw_free support will compile but will only be functional
with the companion patch "soundwire: intel: add intel_free_stream() back"
already applied in the SoundWire tree.

Bard Liao (5):
  Revert "ASoC: SOF: Intel: hda-dai-ops: reset device count for
    SoundWire DAIs"
  Revert "ASoC: SOF: Intel: hda-dai-ops: only allocate/release streams
    for first CPU DAI"
  ASoC: SOF: make dma_config_tlv be an array
  ASoC: SOF: Intel: hda-dai: set lowest N bits in ch_mask
  ASoC: SOF: Intel: hda-dai: set dma_stream_channel_map device

Pierre-Louis Bossart (10):
  ASoC: SOF: Intel: hda-dai: fix channel map configuration for
    aggregated dailink
  ASoC: SOF: Intel: hda-dai: add helpers to set dai config
  ASoC: SOF: Intel: set the DMA TLV device as dai_index
  ASoC: SOF: Intel: hda: extend signature of sdw_hda_dai_hw_params()
  ASoC: SOF: IPC4: extend dai_data with node_id
  ASoC: SOF: Intel: hda: move helper to static inline
  ASoC: SOF: sof-audio: revisit sof_pcm_stream_free() error handling and
    logs
  ASoC: SOF: pcm: simplify sof_pcm_hw_free() with helper
  ASoC: SOF: pcm: add pending_stop state variable
  ASoC: SOF: pcm: reset all PCM sources in case of xruns

Ranjani Sridharan (2):
  ASoC: SOF: Intel: hda: Clear Soundwire node ID during BE DAI hw_free
  ASoC: SOF: ipc4-topology: Save the ALH DAI index during hw_params

 sound/soc/sof/intel/hda-dai-ops.c |  41 +---------
 sound/soc/sof/intel/hda-dai.c     | 124 +++++++++++++++++++++++-------
 sound/soc/sof/intel/hda.c         |  29 ++++++-
 sound/soc/sof/intel/hda.h         |  11 ++-
 sound/soc/sof/ipc4-topology.c     |  62 ++++++++++-----
 sound/soc/sof/ipc4-topology.h     |   3 +-
 sound/soc/sof/pcm.c               |  58 +++++++-------
 sound/soc/sof/sof-audio.c         |  29 +++++--
 sound/soc/sof/sof-audio.h         |   2 +
 9 files changed, 232 insertions(+), 127 deletions(-)

Comments

Mark Brown April 2, 2024, 11:29 p.m. UTC | #1
On Tue, 02 Apr 2024 10:18:11 -0500, Pierre-Louis Bossart wrote:
> This patchset contains important updates for SoundWire support.
> 
> We initially implemented support for multiple amplifiers on different
> links using a single HDaudio DMA transfer. To align with the other OS,
> the 'aggregation' is now supported by the firmware. This change in
> directions has kernel impacts, since we now have multiple HDaudio DMAs
> to program and start, but since there are no platforms released so far
> there's no end-user impact.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[01/17] ASoC: SOF: Intel: hda-dai: fix channel map configuration for aggregated dailink
        commit: 831045513c8a2ef14c3cf39b33d1ccedf588c4a8
[02/17] Revert "ASoC: SOF: Intel: hda-dai-ops: reset device count for SoundWire DAIs"
        commit: 24b1f93df400e1ab1731e7bcb320e693a6a73792
[03/17] Revert "ASoC: SOF: Intel: hda-dai-ops: only allocate/release streams for first CPU DAI"
        commit: 1d0fb3d0c30749779cdd88be98761b17ebfe5590
[04/17] ASoC: SOF: make dma_config_tlv be an array
        commit: e9c6b118de1afc1d32a4eb3bc9f3d114d4fe0f1a
[05/17] ASoC: SOF: Intel: hda-dai: set lowest N bits in ch_mask
        commit: 8fa10a243600ca8bd92fdc871100deb308fca5f1
[06/17] ASoC: SOF: Intel: hda-dai: set dma_stream_channel_map device
        commit: 17386cb1b48b0d85f69b21ae13d5408d67180e30
[07/17] ASoC: SOF: Intel: hda-dai: add helpers to set dai config
        commit: 58f32cb7011a8a15b18f35e4c0ee044aa98e365b
[08/17] ASoC: SOF: Intel: set the DMA TLV device as dai_index
        commit: 219271481e8965e80ee425cdc2db85230a333a97
[09/17] ASoC: SOF: Intel: hda: extend signature of sdw_hda_dai_hw_params()
        commit: 8bc3b56cac748f6ef6a4b96c906007a546e7fb5a
[10/17] ASoC: SOF: IPC4: extend dai_data with node_id
        commit: a936456d4bce27edc1a18dab270c657e9c07590c
[11/17] ASoC: SOF: Intel: hda: move helper to static inline
        commit: 2ac9e09ba0e874deeba13c3259dc18f22b622311
[12/17] ASoC: SOF: Intel: hda: Clear Soundwire node ID during BE DAI hw_free
        commit: bfe9225455c032c9dd5637047760cf59562e599f
[13/17] ASoC: SOF: sof-audio: revisit sof_pcm_stream_free() error handling and logs
        commit: a6f2b279d22894e81b23464620d03da6429d9ab5
[14/17] ASoC: SOF: pcm: simplify sof_pcm_hw_free() with helper
        commit: bb83ae04d9158276d17640f50c2a1e049100acb6
[15/17] ASoC: SOF: pcm: add pending_stop state variable
        commit: dbc78bce74f5f9057ba02bdc8d1549d24c573900
[16/17] ASoC: SOF: pcm: reset all PCM sources in case of xruns
        commit: ebd3b3014eebdd490f2c509d79e719fbcc680963
[17/17] ASoC: SOF: ipc4-topology: Save the ALH DAI index during hw_params
        commit: f0caa4fc244ca739ce6d12168aa588c412c81190

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark