>> >diff --git a/include/sound/pcm.h b/include/sound/pcm.h index
>> >04e97564949c..2c30c1ad1b0d 100644
>> >--- a/include/sound/pcm.h
>> >+++ b/include/sound/pcm.h
>> >@@ -582,13 +582,8 @@ int snd_pcm_stop(struct snd_pcm_substream
>> >*substream, snd_pcm_state_t status); int snd_pcm_drain_done(struct
>> >snd_pcm_substream *substream); int snd_pcm_stop_xrun(struct
>> >snd_pcm_substream *substream); #ifdef CONFIG_PM -int
>> >snd_pcm_suspend(struct snd_pcm_substream *substream); int
>> >snd_pcm_suspend_all(struct snd_pcm *pcm); #else -static inline int
>> >snd_pcm_suspend(struct snd_pcm_substream *substream) -{
>> >- return 0;
>> >-}
>> > static inline int snd_pcm_suspend_all(struct snd_pcm *pcm) {
>> > return 0;
>> >diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index
>> >818dff1de545..26afb6b0889a 100644
>> >--- a/sound/core/pcm_native.c
>> >+++ b/sound/core/pcm_native.c
>> >@@ -1460,29 +1460,24 @@ static const struct action_ops
>> >snd_pcm_action_suspend = {
>> > .post_action = snd_pcm_post_suspend };
>> >
>> >-/**
>> >+/*
>> > * snd_pcm_suspend - trigger SUSPEND to all linked streams
>> > * @substream: the PCM substream
>> > *
>> > * After this call, all streams are changed to SUSPENDED state.
>> > *
>> >- * Return: Zero if successful (or @substream is %NULL), or a
>> >negative error
>> >- * code.
>> >+ * Return: Zero if successful, or a negative error code.
>> > */
>> >-int snd_pcm_suspend(struct snd_pcm_substream *substream)
>> >+static int snd_pcm_suspend(struct snd_pcm_substream *substream)
>>
>> If some drivers may use snd_pcm_suspend() later for corner cases, is
>> it OK to make it be external again?
>
>Yes. But it means that you're doing something special and often wrong. We
>can catch such a case more easily by this action :)
You are right. I checked SOF (it use snd_pcm_suspend()) and believe we
don't have to call snd_pcm_suspend() directly. :-)
Regards,
Libin
>
>
>thanks,
>
>Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
