ASoC: Intel: Skylake: Fix passing loadable flag for module

skl_get_module_info() tries to set mconfig->module->loadable before
mconfig->module has been assigned thus flag was always set to false
and driver did not try to load module binaries.

Signed-off-by: Gustaw Lewandowski <gustaw.lewandowski@linux.intel.com>
Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Tested-by: Lukasz Majczak <lma@semihalf.com>
Link: https://lore.kernel.org/r/20210818075742.1515155-7-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/intel/skylake/skl-pcm.c b/sound/soc/intel/skylake/skl-pcm.c
index c604200de80e47129969954128fb75057e5291aa..9ecaf6a1e8475de4743f551784c667b1b5e7873e 100644 (file)
--- a/sound/soc/intel/skylake/skl-pcm.c
+++ b/sound/soc/intel/skylake/skl-pcm.c
@@ -1310,21 +1310,6 @@ static int skl_get_module_info(struct skl_dev *skl,
                return -EIO;
        }
 
-       list_for_each_entry(module, &skl->uuid_list, list) {
-               if (guid_equal(uuid_mod, &module->uuid)) {
-                       mconfig->id.module_id = module->id;
-                       if (mconfig->module)
-                               mconfig->module->loadable = module->is_loadable;
-                       ret = 0;
-                       break;
-               }
-       }
-
-       if (ret)
-               return ret;
-
-       uuid_mod = &module->uuid;
-       ret = -EIO;
        for (i = 0; i < skl->nr_modules; i++) {
                skl_module = skl->modules[i];
                uuid_tplg = &skl_module->uuid;
@@ -1334,10 +1319,18 @@ static int skl_get_module_info(struct skl_dev *skl,
                        break;
                }
        }
+
        if (skl->nr_modules && ret)
                return ret;
 
+       ret = -EIO;
        list_for_each_entry(module, &skl->uuid_list, list) {
+               if (guid_equal(uuid_mod, &module->uuid)) {
+                       mconfig->id.module_id = module->id;
+                       mconfig->module->loadable = module->is_loadable;
+                       ret = 0;
+               }
+
                for (i = 0; i < MAX_IN_QUEUE; i++) {
                        pin_id = &mconfig->m_in_pin[i].id;
                        if (guid_equal(&pin_id->mod_uuid, &module->uuid))
@@ -1351,7 +1344,7 @@ static int skl_get_module_info(struct skl_dev *skl,
                }
        }
 
-       return 0;
+       return ret;
 }
 
 static int skl_populate_modules(struct skl_dev *skl)