From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> snd_pcm_running() itself is checking parameter now. Let's remove duplicate check Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> --- sound/soc/dwc/dwc-pcm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/dwc/dwc-pcm.c b/sound/soc/dwc/dwc-pcm.c index 2cc9632..1bab46c 100644 --- a/sound/soc/dwc/dwc-pcm.c +++ b/sound/soc/dwc/dwc-pcm.c @@ -102,7 +102,7 @@ static void dw_pcm_transfer(struct dw_i2s_dev *dev, bool push) substream = rcu_dereference(dev->tx_substream); else substream = rcu_dereference(dev->rx_substream); - active = substream && snd_pcm_running(substream); + active = snd_pcm_running(substream); if (active) { unsigned int ptr; unsigned int new_ptr; -- 1.9.1 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel