ASoC: SOF: topology: don't use list_for_each_entry_reverse()
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Tue, 23 Nov 2021 17:16:05 +0000 (19:16 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 24 Nov 2021 12:57:24 +0000 (12:57 +0000)
It's not clear why we would walk the list backwards. That makes no
difference.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Link: https://lore.kernel.org/r/20211123171606.129350-2-kai.vehmanen@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/topology.c

index 63948bb..b3ad3a6 100644 (file)
@@ -3529,7 +3529,7 @@ static int sof_complete(struct snd_soc_component *scomp)
                         * Apply the dynamic_pipeline_widget flag and set the pipe_widget field
                         * for all widgets that have the same pipeline ID as the scheduler widget
                         */
-                       list_for_each_entry_reverse(comp_swidget, &sdev->widget_list, list)
+                       list_for_each_entry(comp_swidget, &sdev->widget_list, list)
                                if (comp_swidget->pipeline_id == swidget->pipeline_id) {
                                        ret = sof_set_pipe_widget(sdev, swidget, comp_swidget);
                                        if (ret < 0)