May 2025 - Alsa-devel - mailman.alsa-project.org

No `appl_ptr` update or `.ack` callback call on driver side when using dmix plugin. Works fine using `hw:`
by GitHub issues - edited 02 May '25

02 May '25

alsa-project/alsa-lib issue #454 was edited from dbrignoli: This issue affects an out of tree `snd-asihpi` driver which uses a different h/w pointer update implementation compared to the in-tree `snd-asihpi` driver. In particular, this version of the driver uses the `.ack` callback to manage h/w pointer update. Driver source is available for download here <https://www.audioscience.com/internet/download/drivers/released/v4/20/47/hp…>. OS: Ubuntu 22.04.5 LTS libasound2: 1.2.6.1-1ubuntu1 LInux: 6.8.0-58-generic Using the following `.asoundrc` (with or without `slowptr true`): ``` pcm.testPlugDmix { type plug slave.pcm "dmixer" } pcm.dmixer { type dmix ipc_key 1024 slave { pcm "hw:1,0" channels 2 format S32_LE period_time 0 period_size 1024 buffer_size 4096 rate 44100 } bindings { 0 0 1 1 } # slowptr true } ``` and running aplay with `-D testPlugDmix` the trace looks like this: ``` 1) | snd_pcm_ioctl [snd_pcm]() { 1) | snd_pcm_common_ioctl [snd_pcm]() { 1) | snd_pcm_sync_ptr [snd_pcm]() { 1) 0.388 us | pcm_lib_apply_appl_ptr [snd_pcm](); /* = 0x0 */ 1) 1.155 us | } /* snd_pcm_sync_ptr [snd_pcm] = 0x0 */ 1) 1.697 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 1) 2.055 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` Running aplay with `-D hw:1` instead the trace looks like this: ``` 0) | snd_pcm_ioctl [snd_pcm]() { 0) | snd_pcm_common_ioctl [snd_pcm]() { 0) | __snd_pcm_lib_xfer [snd_pcm]() { 0) 0.333 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) 0.302 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) | interleaved_copy [snd_pcm]() { 0) | do_transfer [snd_pcm]() { 0) 4.608 us | default_write_copy [snd_pcm](); /* = 0x0 */ 0) 5.249 us | } /* do_transfer [snd_pcm] = 0x0 */ 0) 8.835 us | } /* interleaved_copy [snd_pcm] = 0x0 */ 0) 0.311 us | snd_dma_buffer_sync [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) | pcm_lib_apply_appl_ptr [snd_pcm]() { 0) | snd_card_asihpi_playback_ack [snd_asihpi]() { 0) | /* asihpi_stream_xfer: pcmC1D0p/sub0 : 4096 */ 0) | /* asihpi_stream_state: pcmC1D0p/sub0 in .ack: ALSA: aptr 4096, hptr 0, avail 12288; HPI size 16384, host 4096 (4096), dsp 0 (0, 0), fifo 0, avail 12288, xfer 0 */ 0) 1.376 us | } /* snd_card_asihpi_playback_ack [snd_asihpi] = 0x0 */ 0) 1.999 us | } /* pcm_lib_apply_appl_ptr [snd_pcm] = 0x0 */ 0) 0.364 us | snd_pcm_update_state [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) + 15.415 us | } /* __snd_pcm_lib_xfer [snd_pcm] = 0x1000 */ 0) + 16.252 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 0) + 16.920 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` With dmix the driver never sees changes to `appl_ptr` and the `.ack` callback is never called. The driver sets the `SNDRV_PCM_INFO_SYNC_APPLPTR` flag and and I have tried adding `SNDRV_PCM_INFO_EXPLICIT_SYNC` but it makes no difference. Using `dsnoop` on the capture side shows the same symptoms. Is this expected with dmix? Thank you. Issue URL : https://github.com/alsa-project/alsa-lib/issues/454 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

No `appl_ptr` update or `.ack` callback call on driver side when using dmix plugin. Works fine using `hw:`
by GitHub issues - edited 02 May '25

02 May '25

alsa-project/alsa-lib issue #454 was edited from dbrignoli: This issue affects an out of tree `snd-asihpi` driver which uses a different h/w pointer update implementation compared to the in-tree `snd-asihpi` driver. In particular, this version of the driver uses the `.ack` callback to manage h/w pointer update. Driver source is available for download here <https://www.audioscience.com/internet/download/drivers/released/v4/20/47/hp…>. Using the following `.asoundrc` (with or without `slowptr true`): ``` pcm.testPlugDmix { type plug slave.pcm "dmixer" } pcm.dmixer { type dmix ipc_key 1024 slave { pcm "hw:1,0" channels 2 format S32_LE period_time 0 period_size 1024 buffer_size 4096 rate 44100 } bindings { 0 0 1 1 } # slowptr true } ``` and running aplay with `-D testPlugDmix` the trace looks like this: ``` 1) | snd_pcm_ioctl [snd_pcm]() { 1) | snd_pcm_common_ioctl [snd_pcm]() { 1) | snd_pcm_sync_ptr [snd_pcm]() { 1) 0.388 us | pcm_lib_apply_appl_ptr [snd_pcm](); /* = 0x0 */ 1) 1.155 us | } /* snd_pcm_sync_ptr [snd_pcm] = 0x0 */ 1) 1.697 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 1) 2.055 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` Running aplay with `-D hw:1` instead the trace looks like this: ``` 0) | snd_pcm_ioctl [snd_pcm]() { 0) | snd_pcm_common_ioctl [snd_pcm]() { 0) | __snd_pcm_lib_xfer [snd_pcm]() { 0) 0.333 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) 0.302 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) | interleaved_copy [snd_pcm]() { 0) | do_transfer [snd_pcm]() { 0) 4.608 us | default_write_copy [snd_pcm](); /* = 0x0 */ 0) 5.249 us | } /* do_transfer [snd_pcm] = 0x0 */ 0) 8.835 us | } /* interleaved_copy [snd_pcm] = 0x0 */ 0) 0.311 us | snd_dma_buffer_sync [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) | pcm_lib_apply_appl_ptr [snd_pcm]() { 0) | snd_card_asihpi_playback_ack [snd_asihpi]() { 0) | /* asihpi_stream_xfer: pcmC1D0p/sub0 : 4096 */ 0) | /* asihpi_stream_state: pcmC1D0p/sub0 in .ack: ALSA: aptr 4096, hptr 0, avail 12288; HPI size 16384, host 4096 (4096), dsp 0 (0, 0), fifo 0, avail 12288, xfer 0 */ 0) 1.376 us | } /* snd_card_asihpi_playback_ack [snd_asihpi] = 0x0 */ 0) 1.999 us | } /* pcm_lib_apply_appl_ptr [snd_pcm] = 0x0 */ 0) 0.364 us | snd_pcm_update_state [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) + 15.415 us | } /* __snd_pcm_lib_xfer [snd_pcm] = 0x1000 */ 0) + 16.252 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 0) + 16.920 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` With dmix the driver never sees changes to `appl_ptr` and the `.ack` callback is never called. The driver sets the `SNDRV_PCM_INFO_SYNC_APPLPTR` flag and and I have tried adding `SNDRV_PCM_INFO_EXPLICIT_SYNC` but it makes no difference. Using `dsnoop` on the capture side shows the same symptoms. Is this expected with dmix? Thank you. Issue URL : https://github.com/alsa-project/alsa-lib/issues/454 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

No `appl_ptr` update or `.ack` callback call on driver side when using dmix plugin. Works fine using `hw:`
by GitHub issues - opened 02 May '25

02 May '25

alsa-project/alsa-lib issue #454 was opened from dbrignoli: This issue affects an out of tree `snd-asihpi` driver which uses a different h/w pointer update implementation compared to the in-tree `snd-asihpi` driver. In particular, this version of the driver uses the `.ack` callback to manage h/w pointer update. Driver source is available for download here <https://www.audioscience.com/internet/download/drivers/released/v4/20/47/hp…>. Using the following `.asoundrc` (with or without `slowptr true`): ``` pcm.testPlugDmix { type plug slave.pcm "dmixer" } pcm.dmixer { type dmix ipc_key 1024 slave { pcm "hw:1,0" channels 2 format S32_LE period_time 0 period_size 1024 buffer_size 4096 rate 44100 } bindings { 0 0 1 1 } # slowptr true } ``` and running aplay with `-D testPlugDmix` the trace looks like this: ``` 1) | snd_pcm_ioctl [snd_pcm]() { 1) | snd_pcm_common_ioctl [snd_pcm]() { 1) | snd_pcm_sync_ptr [snd_pcm]() { 1) 0.388 us | pcm_lib_apply_appl_ptr [snd_pcm](); /* = 0x0 */ 1) 1.155 us | } /* snd_pcm_sync_ptr [snd_pcm] = 0x0 */ 1) 1.697 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 1) 2.055 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` Running aplay with `-D hw:1` instead the trace looks like this: ``` 0) | snd_pcm_ioctl [snd_pcm]() { 0) | snd_pcm_common_ioctl [snd_pcm]() { 0) | __snd_pcm_lib_xfer [snd_pcm]() { 0) 0.333 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) 0.302 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) | interleaved_copy [snd_pcm]() { 0) | do_transfer [snd_pcm]() { 0) 4.608 us | default_write_copy [snd_pcm](); /* = 0x0 */ 0) 5.249 us | } /* do_transfer [snd_pcm] = 0x0 */ 0) 8.835 us | } /* interleaved_copy [snd_pcm] = 0x0 */ 0) 0.311 us | snd_dma_buffer_sync [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_lock_irq [snd_pcm](); /* = 0x0 */ 0) | pcm_lib_apply_appl_ptr [snd_pcm]() { 0) | snd_card_asihpi_playback_ack [snd_asihpi]() { 0) | /* asihpi_stream_xfer: pcmC1D0p/sub0 : 4096 */ 0) | /* asihpi_stream_state: pcmC1D0p/sub0 in .ack: ALSA: aptr 4096, hptr 0, avail 12288; HPI size 16384, host 4096 (4096), dsp 0 (0, 0), fifo 0, avail 12288, xfer 0 */ 0) 1.376 us | } /* snd_card_asihpi_playback_ack [snd_asihpi] = 0x0 */ 0) 1.999 us | } /* pcm_lib_apply_appl_ptr [snd_pcm] = 0x0 */ 0) 0.364 us | snd_pcm_update_state [snd_pcm](); /* = 0x0 */ 0) 0.299 us | snd_pcm_stream_unlock_irq [snd_pcm](); /* = 0x0 */ 0) + 15.415 us | } /* __snd_pcm_lib_xfer [snd_pcm] = 0x1000 */ 0) + 16.252 us | } /* snd_pcm_common_ioctl [snd_pcm] = 0x0 */ 0) + 16.920 us | } /* snd_pcm_ioctl [snd_pcm] = 0x0 */ ``` With dmix the driver never sees changes to `appl_ptr` and the `.ack` callback is never called. The driver sets the `SNDRV_PCM_INFO_SYNC_APPLPTR` flag and and I have tried adding `SNDRV_PCM_INFO_EXPLICIT_SYNC` but it makes no difference. Using `dsnoop` on the capture side shows the same symptoms. Is this expected with dmix? Thank you. Issue URL : https://github.com/alsa-project/alsa-lib/issues/454 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH 1/3] ASoC: amd: acp: Fix NULL pointer deref on acp resume path
by Venkata Prasad Potturu 02 May '25

02 May '25

update chip data using dev_get_drvdata(dev->parent) instead of dev_get_platdata(dev). BUG: kernel NULL pointer dereference, address: 0000000000000010 Call Trace: <TASK> ? __pfx_platform_pm_resume+0x10/0x10 platform_pm_resume+0x28/0x60 dpm_run_callback+0x51/0x1a0 device_resume+0x1a6/0x2b0 dpm_resume+0x168/0x230 Fixes: e3933683b25e ("ASoC: amd: acp: Remove redundant acp_dev_data structure") Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu(a)amd.com> --- sound/soc/amd/acp/acp-rembrandt.c | 2 +- sound/soc/amd/acp/acp-renoir.c | 2 +- sound/soc/amd/acp/acp63.c | 2 +- sound/soc/amd/acp/acp70.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/amd/acp/acp-rembrandt.c b/sound/soc/amd/acp/acp-rembrandt.c index 746b6ed72029..cccdd10c345e 100644 --- a/sound/soc/amd/acp/acp-rembrandt.c +++ b/sound/soc/amd/acp/acp-rembrandt.c @@ -199,7 +199,7 @@ static void rembrandt_audio_remove(struct platform_device *pdev) static int rmb_pcm_resume(struct device *dev) { - struct acp_chip_info *chip = dev_get_platdata(dev); + struct acp_chip_info *chip = dev_get_drvdata(dev->parent); struct acp_stream *stream; struct snd_pcm_substream *substream; snd_pcm_uframes_t buf_in_frames; diff --git a/sound/soc/amd/acp/acp-renoir.c b/sound/soc/amd/acp/acp-renoir.c index ebf0106fc737..04f6d70b6a92 100644 --- a/sound/soc/amd/acp/acp-renoir.c +++ b/sound/soc/amd/acp/acp-renoir.c @@ -146,7 +146,7 @@ static void renoir_audio_remove(struct platform_device *pdev) static int rn_pcm_resume(struct device *dev) { - struct acp_chip_info *chip = dev_get_platdata(dev); + struct acp_chip_info *chip = dev_get_drvdata(dev->parent); struct acp_stream *stream; struct snd_pcm_substream *substream; snd_pcm_uframes_t buf_in_frames; diff --git a/sound/soc/amd/acp/acp63.c b/sound/soc/amd/acp/acp63.c index 52d895e624c7..1f15c96a9b94 100644 --- a/sound/soc/amd/acp/acp63.c +++ b/sound/soc/amd/acp/acp63.c @@ -250,7 +250,7 @@ static void acp63_audio_remove(struct platform_device *pdev) static int acp63_pcm_resume(struct device *dev) { - struct acp_chip_info *chip = dev_get_platdata(dev); + struct acp_chip_info *chip = dev_get_drvdata(dev->parent); struct acp_stream *stream; struct snd_pcm_substream *substream; snd_pcm_uframes_t buf_in_frames; diff --git a/sound/soc/amd/acp/acp70.c b/sound/soc/amd/acp/acp70.c index 6d5f5ade075c..217b717e9beb 100644 --- a/sound/soc/amd/acp/acp70.c +++ b/sound/soc/amd/acp/acp70.c @@ -182,7 +182,7 @@ static void acp_acp70_audio_remove(struct platform_device *pdev) static int acp70_pcm_resume(struct device *dev) { - struct acp_chip_info *chip = dev_get_platdata(dev); + struct acp_chip_info *chip = dev_get_drvdata(dev->parent); struct acp_stream *stream; struct snd_pcm_substream *substream; snd_pcm_uframes_t buf_in_frames; -- 2.43.0

5 7

[GIT PULL] ASoC fixes for v6.15-rc4
by Mark Brown 01 May '25

01 May '25

The following changes since commit b4432656b36e5cc1d50a1f2dc15357543add530e: Linux 6.15-rc4 (2025-04-27 15:19:23 -0700) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.15-rc4 for you to fetch changes up to 315c84567001aec7f7b74a0dbd4fce3e1c5cbd87: ASoC: intel/sdw_utils: Add volume limits to CS35L56 (2025-05-01 14:43:52 +0900) ---------------------------------------------------------------- ASoC: Fixes for v6.15 A moderately large batch of fixes for v6.15, many driver specific including cleanups for the enabling of the Cirrus KUnit tests and a fix for a nasty crash on resume on AMD systems. We also have one core fix, for an ordering issue between DAPM and DPCM which could leave things incorrectly unpowered. ---------------------------------------------------------------- Alexander Stein (1): ASoC: simple-card-utils: Fix pointer check in graph_util_parse_link_direction Arnd Bergmann (1): ASoC: Intel: catpt: avoid type mismatch in dev_dbg() format Bard Liao (1): ASoC: soc-acpi-intel-ptl-match: add empty item to ptl_cs42l43_l3[] Charles Keepax (1): ASoC: cs42l43: Disable headphone clamps during type detection Chenyuan Yang (2): ASoC: imx-card: Adjust over allocation of memory in imx_card_parse_of() ASoC: Intel: sof_sdw: Add NULL check in asoc_sdw_rt_dmic_rtd_init() Claudiu Beznea (1): ASoC: renesas: rz-ssi: Use NOIRQ_SYSTEM_SLEEP_PM_OPS() Mark Brown (3): Fix up building KUnit tests for Cirrus Logic modules ASoC: stm32: sai: fix kernel rate configuration ASoC: intel/sdw_utils: Add volume limits to CS35L56 Nico Pache (1): firmware: cs_dsp: tests: Depend on FW_CS_DSP rather then enabling it Olivier Moysan (2): ASoC: stm32: sai: skip useless iterations on kernel rate loop ASoC: stm32: sai: add a check on minimal kernel frequency Richard Fitzgerald (3): kunit: configs: Add some Cirrus Logic modules to all_tests ASoC: cs-amp-lib-test: Don't select SND_SOC_CS_AMP_LIB ASoC: Add Cirrus and Wolfson headers to ASoC section of MAINTAINERS Sheetal (1): ASoC: soc-pcm: Fix hw_params() and DAPM widget sequence Stefan Binding (2): ASoC: intel/sdw_utils: Add volume limit to cs42l43 speakers ASoC: intel/sdw_utils: Add volume limit to cs35l56 speakers Takashi Iwai (1): ASoC: Intel: bytcr_rt5640: Add DMI quirk for Acer Aspire SW3-013 Venkata Prasad Potturu (3): ASoC: amd: acp: Fix NULL pointer deref on acp resume path ASoC: amd: acp: Fix NULL pointer deref in acp_i2s_set_tdm_slot ASoC: amd: acp: Fix devm_snd_soc_register_card(acp-pdm-mach) failure Vijendar Mukunda (1): ASoC: amd: ps: fix for irq handler return status MAINTAINERS | 6 ++++++ drivers/firmware/cirrus/Kconfig | 5 +---- include/sound/soc_sdw_utils.h | 1 + sound/soc/amd/acp/acp-i2s.c | 2 +- sound/soc/amd/acp/acp-legacy-common.c | 2 +- sound/soc/amd/acp/acp-rembrandt.c | 2 +- sound/soc/amd/acp/acp-renoir.c | 2 +- sound/soc/amd/acp/acp63.c | 2 +- sound/soc/amd/acp/acp70.c | 2 +- sound/soc/amd/ps/pci-ps.c | 5 +++-- sound/soc/codecs/Kconfig | 5 ++--- sound/soc/codecs/cs42l43-jack.c | 7 +++++++ sound/soc/fsl/imx-card.c | 2 +- sound/soc/generic/simple-card-utils.c | 4 ++-- sound/soc/intel/boards/bytcr_rt5640.c | 13 ++++++++++++ sound/soc/intel/catpt/dsp.c | 2 +- sound/soc/intel/common/soc-acpi-intel-ptl-match.c | 3 ++- sound/soc/renesas/rz-ssi.c | 2 +- sound/soc/sdw_utils/soc_sdw_bridge_cs35l56.c | 4 ++++ sound/soc/sdw_utils/soc_sdw_cs42l43.c | 10 ++++++++++ sound/soc/sdw_utils/soc_sdw_cs_amp.c | 24 +++++++++++++++++++++++ sound/soc/sdw_utils/soc_sdw_rt_dmic.c | 2 ++ sound/soc/soc-pcm.c | 5 ++++- sound/soc/stm/stm32_sai_sub.c | 16 ++++++++++----- tools/testing/kunit/configs/all_tests.config | 2 ++ 25 files changed, 103 insertions(+), 27 deletions(-)

2 1

[PATCH] ASoC: amd: ps: fix for irq handler return status
by Vijendar Mukunda 01 May '25

01 May '25

If any Soundwire manager interrupt is reported, and wake interrupt is not reported, in this scenario irq_flag will be set to zero, which results in interrupt handler return status as IRQ_NONE. Add new irq flag 'wake_irq_flag' check for SoundWire wake interrupt handling to fix incorrect irq handling return status. Fixes: 3898b189079c8 ("ASoC: amd: ps: add soundwire wake interrupt handling") Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> --- sound/soc/amd/ps/pci-ps.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sound/soc/amd/ps/pci-ps.c b/sound/soc/amd/ps/pci-ps.c index 8e57f31ef7f7..7936b3173632 100644 --- a/sound/soc/amd/ps/pci-ps.c +++ b/sound/soc/amd/ps/pci-ps.c @@ -193,6 +193,7 @@ static irqreturn_t acp63_irq_handler(int irq, void *dev_id) struct amd_sdw_manager *amd_manager; u32 ext_intr_stat, ext_intr_stat1; u16 irq_flag = 0; + u16 wake_irq_flag = 0; u16 sdw_dma_irq_flag = 0; adata = dev_id; @@ -231,7 +232,7 @@ static irqreturn_t acp63_irq_handler(int irq, void *dev_id) } if (adata->acp_rev >= ACP70_PCI_REV) - irq_flag = check_and_handle_acp70_sdw_wake_irq(adata); + wake_irq_flag = check_and_handle_acp70_sdw_wake_irq(adata); if (ext_intr_stat & BIT(PDM_DMA_STAT)) { ps_pdm_data = dev_get_drvdata(&adata->pdm_dev->dev); @@ -245,7 +246,7 @@ static irqreturn_t acp63_irq_handler(int irq, void *dev_id) if (sdw_dma_irq_flag) return IRQ_WAKE_THREAD; - if (irq_flag) + if (irq_flag | wake_irq_flag) return IRQ_HANDLED; else return IRQ_NONE; -- 2.45.2

2 1