alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
* [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).