On Thu, 21 Mar 2024 15:07:57 +0200, Peter Ujfalusi wrote: > The current version of delay reporting code can report incorrect > values when paired with a firmware which enables this feature. > > Unfortunately there are several smaller issues that needed to be addressed > to correct the behavior: > > Wrong information was used for the host side of counter > For MTL/LNL used incorrect (in a sense that it was verified only on MTL) > link side counter function. > The link side counter needs compensation logic if pause/resume is used. > The offset values were not refreshed from firmware. > Finally, not strictly connected, but the ALSA buffer size needs to be > constrained to avoid constant xrun from media players (like mpv) > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [01/17] ASoC: SOF: Add dsp_max_burst_size_in_ms member to snd_sof_pcm_stream commit: fb9f8125ed9d9b8e11f309a7dbfbe7b40de48fba [02/17] ASoC: SOF: ipc4-topology: Save the DMA maximum burst size for PCMs commit: 842bb8b62cc6f3546d61eb63115b32ebc6dd4a87 [03/17] ASoC: SOF: Intel: hda-pcm: Use dsp_max_burst_size_in_ms to place constraint commit: fe76d2e75a6da97edd2b4ec5cfb9efd541be087a [04/17] ASoC: SOF: Intel: hda: Implement get_stream_position (Linear Link Position) commit: 67b182bea08a8d1092b91b57aefdfe420fce1634 [05/17] ASoC: SOF: Intel: mtl/lnl: Use the generic get_stream_position callback commit: 4374f698d7d9f849b66f3fa8f7a64f0bc1a53d7f [06/17] ASoC: SOF: Introduce a new callback pair to be used for PCM delay reporting commit: ce2faa9a180c1984225689b6b1cb26045f8b7470 [07/17] ASoC: SOF: Intel: Set the dai/host get frame/byte counter callbacks commit: fd6f6a0632bc891673490bf4a92304172251825c [08/17] ASoC: SOF: ipc4-pcm: Use the snd_sof_pcm_get_dai_frame_counter() for pcm_delay commit: 37679a1bd372c8308a3faccf3438c9df642565b3 [09/17] ASoC: SOF: Intel: hda-common-ops: Do not set the get_stream_position callback commit: 4ab6c38c664442c1fc9911eb3c5c6953d3dbcca5 [10/17] ASoC: SOF: Remove the get_stream_position callback commit: 07007b8ac42cffc23043d00e56b0f67a75dc4b22 [11/17] ASoC: SOF: ipc4-pcm: Move struct sof_ipc4_timestamp_info definition locally commit: 31d2874d083ba6cc2a4f4b26dab73c3be1c92658 [12/17] ASoC: SOF: ipc4-pcm: Combine the SOF_IPC4_PIPE_PAUSED cases in pcm_trigger commit: 55ca6ca227bfc5a8d0a0c2c5d6e239777226a604 [13/17] ASoC: SOF: ipc4-pcm: Invalidate the stream_start_offset in PAUSED state commit: 3ce3bc36d91510389955b47e36ea4c4e94fcbdd3 [14/17] ASoC: SOF: sof-pcm: Add pointer callback to sof_ipc_pcm_ops commit: 77165bd955d55114028b06787a530b8f9220e4b0 [15/17] ASoC: SOF: ipc4-pcm: Correct the delay calculation commit: 0ea06680dfcb4464ac6c05968433d060efb44345 [16/17] ALSA: hda: Add pplcllpl/u members to hdac_ext_stream commit: f9eeb6bb13fb5d7af1ea5b74a10b1f8ead962540 [17/17] ASoC: SOF: Intel: hda: Compensate LLP in case it is not reset commit: 1abc2642588e06f6180b3fbb21968cf5d0ba9e5f 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
