ASoC: SOF: Intel: hda-mlink: improve hda_bus_ml_free() helper

Use list_for_each_entry_safe() instead of open-coding.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Takashi Iwai <tiwai@suse.de>
Link: https://lore.kernel.org/r/20230404104127.5629-4-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/sof/intel/hda-mlink.c b/sound/soc/sof/intel/hda-mlink.c
index 76ab9a2..ac9bf47 100644 (file)
--- a/sound/soc/sof/intel/hda-mlink.c
+++ b/sound/soc/sof/intel/hda-mlink.c
@@ -36,13 +36,12 @@ void hda_bus_ml_get_capabilities(struct hdac_bus *bus)
 
 void hda_bus_ml_free(struct hdac_bus *bus)
 {
-       struct hdac_ext_link *hlink;
+       struct hdac_ext_link *hlink, *_h;
 
        if (!bus->mlcap)
                return;
 
-       while (!list_empty(&bus->hlink_list)) {
-               hlink = list_first_entry(&bus->hlink_list, struct hdac_ext_link, list);
+       list_for_each_entry_safe(hlink, _h, &bus->hlink_list, list) {
                list_del(&hlink->list);
                kfree(hlink);
        }