[alsa-devel] [PATCH tinycompress] cplay: Stop playback stream at the end

Daniel Baluta daniel.baluta at nxp.com
Wed Jun 27 09:48:11 CEST 2018


Each compress_start should have a corresponding compress_stop
at the end.

Also, this gives user a change to cleanup resources that might
have been allocated in compress_start.

Signed-off-by: Daniel Baluta <daniel.baluta at nxp.com>
---
I'm implementing codec library dynamic loading at runtime and I'm exploring
using:
	* SNDRV_COMPRESS_START -> to load the library
	* SNDRV_COMPRESS_STOP  -> to unload the library

This might not be the best idea becasue for example loading
the library might introduce some delays, anyhow i still think
this that compress_stop is correct thing to add at the end of
playback.


 src/utils/cplay.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/utils/cplay.c b/src/utils/cplay.c
index 87863a3..3c48d15 100644
--- a/src/utils/cplay.c
+++ b/src/utils/cplay.c
@@ -403,6 +403,7 @@ void play_samples(char *name, unsigned int card, unsigned int device,
 		printf("%s: exit success\n", __func__);
 	/* issue drain if it supports */
 	compress_drain(compress);
+	compress_stop(compress);
 	free(buffer);
 	fclose(file);
 	compress_close(compress);
-- 
2.7.4



More information about the Alsa-devel mailing list