diff options
| author | Srinivas Kandagatla <srinivas.kandagatla@linaro.org> | 2025-03-14 20:47:58 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2025-03-31 16:59:38 +0300 |
| commit | a93dad6f4e6a04a5943f6ee5686585f24abf7063 (patch) | |
| tree | 8dbf3a27667240ee32439581b959a9c0207d4ffb /lib/string_helpers.c | |
| parent | 5d01ed9b9939b4c726be74db291a982bc984c584 (diff) | |
| download | linux-a93dad6f4e6a04a5943f6ee5686585f24abf7063.tar.xz | |
ASoC: q6apm-dai: make use of q6apm_get_hw_pointer
With the existing code, the buffer position is only reset in pointer
callback, which leaves the possiblity of it going over the size of
buffer size and reporting incorrect position to userspace.
Without this patch, its possible to see errors like:
snd-x1e80100 sound: invalid position: pcmC0D0p:0, pos = 12288, buffer size = 12288, period size = 1536
snd-x1e80100 sound: invalid position: pcmC0D0p:0, pos = 12288, buffer size = 12288, period size = 1536
Fixes: 9b4fe0f1cd791 ("ASoC: qdsp6: audioreach: add q6apm-dai support")
Cc: stable@vger.kernel.org
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Tested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Tested-by: Johan Hovold <johan+linaro@kernel.org>
Link: https://patch.msgid.link/20250314174800.10142-4-srinivas.kandagatla@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions