* [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation
@ 2016-03-04 14:55 Vinod Koul
2016-03-04 14:55 ` [PATCH 2/2] ALSA: compress: fix some typos Vinod Koul
2016-03-04 16:50 ` [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Takashi Iwai
0 siblings, 2 replies; 3+ messages in thread
From: Vinod Koul @ 2016-03-04 14:55 UTC (permalink / raw)
To: alsa-devel; +Cc: liam.r.girdwood, tiwai, broonie, Vinod Koul, patches.audio
Stream states were explained in the code comments but
SNDRV_PCM_STATE_PREPARED was missed so add it
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
---
sound/core/compress_offload.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c
index 7fac3cae8abd..51a8eb495f54 100644
--- a/sound/core/compress_offload.c
+++ b/sound/core/compress_offload.c
@@ -74,6 +74,9 @@ struct snd_compr_file {
* SNDRV_PCM_STATE_SETUP: When stream has been initialized. This is done by
* calling SNDRV_COMPRESS_SET_PARAMS. running streams will come to this
* state at stop by calling SNDRV_COMPRESS_STOP, or at end of drain.
+ * SNDRV_PCM_STATE_PREPARED: When a stream has been written to (for
+ * playback only). User after setting up stream writes the data buffer
+ * before starting the stream.
* SNDRV_PCM_STATE_RUNNING: When stream has been started and is
* decoding/encoding and rendering/capturing data.
* SNDRV_PCM_STATE_DRAINING: When stream is draining current data. This is done
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] ALSA: compress: fix some typos
2016-03-04 14:55 [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Vinod Koul
@ 2016-03-04 14:55 ` Vinod Koul
2016-03-04 16:50 ` [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Takashi Iwai
1 sibling, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2016-03-04 14:55 UTC (permalink / raw)
To: alsa-devel; +Cc: liam.r.girdwood, tiwai, broonie, Vinod Koul, patches.audio
Found few typos while looking at code, so fix them
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
---
sound/core/compress_offload.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c
index 51a8eb495f54..c4b14b5f1b35 100644
--- a/sound/core/compress_offload.c
+++ b/sound/core/compress_offload.c
@@ -69,10 +69,10 @@ struct snd_compr_file {
/*
* a note on stream states used:
- * we use follwing states in the compressed core
+ * we use following states in the compressed core
* SNDRV_PCM_STATE_OPEN: When stream has been opened.
* SNDRV_PCM_STATE_SETUP: When stream has been initialized. This is done by
- * calling SNDRV_COMPRESS_SET_PARAMS. running streams will come to this
+ * calling SNDRV_COMPRESS_SET_PARAMS. Running streams will come to this
* state at stop by calling SNDRV_COMPRESS_STOP, or at end of drain.
* SNDRV_PCM_STATE_PREPARED: When a stream has been written to (for
* playback only). User after setting up stream writes the data buffer
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation
2016-03-04 14:55 [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Vinod Koul
2016-03-04 14:55 ` [PATCH 2/2] ALSA: compress: fix some typos Vinod Koul
@ 2016-03-04 16:50 ` Takashi Iwai
1 sibling, 0 replies; 3+ messages in thread
From: Takashi Iwai @ 2016-03-04 16:50 UTC (permalink / raw)
To: Vinod Koul; +Cc: liam.r.girdwood, patches.audio, alsa-devel, broonie
On Fri, 04 Mar 2016 15:55:29 +0100,
Vinod Koul wrote:
>
> Stream states were explained in the code comments but
> SNDRV_PCM_STATE_PREPARED was missed so add it
>
> Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Applied both patches. Thanks.
Takashi
> ---
> sound/core/compress_offload.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c
> index 7fac3cae8abd..51a8eb495f54 100644
> --- a/sound/core/compress_offload.c
> +++ b/sound/core/compress_offload.c
> @@ -74,6 +74,9 @@ struct snd_compr_file {
> * SNDRV_PCM_STATE_SETUP: When stream has been initialized. This is done by
> * calling SNDRV_COMPRESS_SET_PARAMS. running streams will come to this
> * state at stop by calling SNDRV_COMPRESS_STOP, or at end of drain.
> + * SNDRV_PCM_STATE_PREPARED: When a stream has been written to (for
> + * playback only). User after setting up stream writes the data buffer
> + * before starting the stream.
> * SNDRV_PCM_STATE_RUNNING: When stream has been started and is
> * decoding/encoding and rendering/capturing data.
> * SNDRV_PCM_STATE_DRAINING: When stream is draining current data. This is done
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-03-04 16:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-04 14:55 [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Vinod Koul
2016-03-04 14:55 ` [PATCH 2/2] ALSA: compress: fix some typos Vinod Koul
2016-03-04 16:50 ` [PATCH 1/2] ALSA: compress: Add SNDRV_PCM_STATE_PREPARED state explanation Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).