ASoC: amd: acp: remove unnecessary NULL checks
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 19 Aug 2022 05:23:09 +0000 (08:23 +0300)
committerMark Brown <broonie@kernel.org>
Fri, 19 Aug 2022 12:20:00 +0000 (13:20 +0100)
The list iterator can never be NULL.  Delete the bogus NULL checks.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/Yv8ePUuBfzaRu6xV@kili
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/amd/acp/acp-platform.c

index beee53aedeaf55271922834fb6c4c3318dcc7387..85a81add4ef9fd5eb7dd5a5006c19e8e987dc416 100644 (file)
@@ -106,14 +106,14 @@ static irqreturn_t i2s_irq_handler(int irq, void *data)
 
        spin_lock(&adata->acp_lock);
        list_for_each_entry(stream, &adata->stream_list, list) {
-               if (stream && (ext_intr_stat & stream->irq_bit)) {
+               if (ext_intr_stat & stream->irq_bit) {
                        writel(stream->irq_bit,
                               ACP_EXTERNAL_INTR_STAT(adata, rsrc->irqp_used));
                        snd_pcm_period_elapsed(stream->substream);
                        i2s_flag = 1;
                }
                if (adata->rsrc->no_of_ctrls == 2) {
-                       if (stream && (ext_intr_stat1 & stream->irq_bit)) {
+                       if (ext_intr_stat1 & stream->irq_bit) {
                                writel(stream->irq_bit, ACP_EXTERNAL_INTR_STAT(adata,
                                       (rsrc->irqp_used - 1)));
                                snd_pcm_period_elapsed(stream->substream);