ASoC: amd: acp: delete unnecessary NULL check
authorDan Carpenter <dan.carpenter@linaro.org>
Tue, 18 Jul 2023 07:04:18 +0000 (10:04 +0300)
committerMark Brown <broonie@kernel.org>
Tue, 18 Jul 2023 14:14:35 +0000 (15:14 +0100)
The list iterator can't be NULL.  Delete the check and pull the code
in one tab.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/b0c5b0ca-68da-47e6-a8b0-e0714f0de119@moroto.mountain
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/amd/acp/acp-rembrandt.c

index 21e67ed956d18e4230d65dfd55c5dd844bdd1d6b..cc8284f417c0e915949c985985877eb2ee84a737 100644 (file)
@@ -266,18 +266,16 @@ static int __maybe_unused rmb_pcm_resume(struct device *dev)
        acp6x_master_clock_generate(dev);
        spin_lock(&adata->acp_lock);
        list_for_each_entry(stream, &adata->stream_list, list) {
-               if (stream) {
-                       substream = stream->substream;
-                       if (substream && substream->runtime) {
-                               buf_in_frames = (substream->runtime->buffer_size);
-                               buf_size = frames_to_bytes(substream->runtime, buf_in_frames);
-                               config_pte_for_stream(adata, stream);
-                               config_acp_dma(adata, stream, buf_size);
-                               if (stream->dai_id)
-                                       restore_acp_i2s_params(substream, adata, stream);
-                               else
-                                       restore_acp_pdm_params(substream, adata);
-                       }
+               substream = stream->substream;
+               if (substream && substream->runtime) {
+                       buf_in_frames = (substream->runtime->buffer_size);
+                       buf_size = frames_to_bytes(substream->runtime, buf_in_frames);
+                       config_pte_for_stream(adata, stream);
+                       config_acp_dma(adata, stream, buf_size);
+                       if (stream->dai_id)
+                               restore_acp_i2s_params(substream, adata, stream);
+                       else
+                               restore_acp_pdm_params(substream, adata);
                }
        }
        spin_unlock(&adata->acp_lock);