ASoC: Intel: avs: Account for libraries when booting basefw
authorCezary Rojewski <cezary.rojewski@intel.com>
Mon, 16 May 2022 10:11:02 +0000 (12:11 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 17 May 2022 10:57:52 +0000 (11:57 +0100)
Not all modules are part of base firmware. Some are part of loadable
libraries. These need to be loaded after base firmware reports ready
status through FW_READY notification.

Their loading process is similar to the base firmware's one. Request the
binary file, verify and strip the manifest and load the actual code into
DSP memory with help of CLDMA or HD-Audio render stream, depending on
audio device generation.

List of libraries needed for loading is obtained through the topology -
vendor sections specifying the name of firmware files to request.

Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://lore.kernel.org/r/20220516101116.190192-2-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/avs/avs.h
sound/soc/intel/avs/loader.c

index c57a07a18d8e8e5cd83643644d0d3d12770469f7..14b4a780a91c25f25f2a7cde6ab94c345b94122a 100644 (file)
@@ -19,6 +19,8 @@
 
 struct avs_dev;
 struct avs_tplg;
+struct avs_tplg_library;
+struct avs_soc_component;
 
 /*
  * struct avs_dsp_ops - Platform-specific DSP operations
@@ -241,6 +243,7 @@ void avs_hda_clock_gating_enable(struct avs_dev *adev, bool enable);
 void avs_hda_power_gating_enable(struct avs_dev *adev, bool enable);
 void avs_hda_l1sen_enable(struct avs_dev *adev, bool enable);
 
+int avs_dsp_load_libraries(struct avs_dev *adev, struct avs_tplg_library *libs, u32 num_libs);
 int avs_dsp_boot_firmware(struct avs_dev *adev, bool purge);
 int avs_dsp_first_boot_firmware(struct avs_dev *adev);
 
index c47f85161d951571e175c538b692c2053719efb6..8aa658c5232513406d3f24be52eec5afbcb0c0af 100644 (file)
@@ -15,6 +15,7 @@
 #include "cldma.h"
 #include "messages.h"
 #include "registers.h"
+#include "topology.h"
 
 #define AVS_ROM_STS_MASK               0xFF
 #define AVS_ROM_INIT_DONE              0x1
@@ -466,6 +467,71 @@ int avs_hda_transfer_modules(struct avs_dev *adev, bool load,
        return 0;
 }
 
+int avs_dsp_load_libraries(struct avs_dev *adev, struct avs_tplg_library *libs, u32 num_libs)
+{
+       int start, id, i = 0;
+       int ret;
+
+       /* Calculate the id to assign for the next lib. */
+       for (id = 0; id < adev->fw_cfg.max_libs_count; id++)
+               if (adev->lib_names[id][0] == '\0')
+                       break;
+       if (id + num_libs >= adev->fw_cfg.max_libs_count)
+               return -EINVAL;
+
+       start = id;
+       while (i < num_libs) {
+               struct avs_fw_manifest *man;
+               const struct firmware *fw;
+               struct firmware stripped_fw;
+               char *filename;
+               int j;
+
+               filename = kasprintf(GFP_KERNEL, "%s/%s/%s", AVS_ROOT_DIR, adev->spec->name,
+                                    libs[i].name);
+               if (!filename)
+                       return -ENOMEM;
+
+               /*
+                * If any call after this one fails, requested firmware is not released with
+                * avs_release_last_firmware() as failing to load code results in need for reload
+                * of entire driver module. And then avs_release_firmwares() is in place already.
+                */
+               ret = avs_request_firmware(adev, &fw, filename);
+               kfree(filename);
+               if (ret < 0)
+                       return ret;
+
+               stripped_fw = *fw;
+               ret = avs_fw_manifest_strip_verify(adev, &stripped_fw, NULL);
+               if (ret) {
+                       dev_err(adev->dev, "invalid library data: %d\n", ret);
+                       return ret;
+               }
+
+               ret = avs_fw_manifest_offset(&stripped_fw);
+               if (ret < 0)
+                       return ret;
+               man = (struct avs_fw_manifest *)(stripped_fw.data + ret);
+
+               /* Don't load anything that's already in DSP memory. */
+               for (j = 0; j < id; j++)
+                       if (!strncmp(adev->lib_names[j], man->name, AVS_LIB_NAME_SIZE))
+                               goto next_lib;
+
+               ret = avs_dsp_op(adev, load_lib, &stripped_fw, id);
+               if (ret)
+                       return ret;
+
+               strncpy(adev->lib_names[id], man->name, AVS_LIB_NAME_SIZE);
+               id++;
+next_lib:
+               i++;
+       }
+
+       return start == id ? 1 : 0;
+}
+
 static int avs_dsp_load_basefw(struct avs_dev *adev)
 {
        const struct avs_fw_version *min_req;
@@ -519,6 +585,7 @@ release_fw:
 
 int avs_dsp_boot_firmware(struct avs_dev *adev, bool purge)
 {
+       struct avs_soc_component *acomp;
        int ret, i;
 
        /* Forgo full boot if flash from IMR succeeds. */
@@ -538,7 +605,20 @@ int avs_dsp_boot_firmware(struct avs_dev *adev, bool purge)
        avs_hda_l1sen_enable(adev, false);
 
        ret = avs_dsp_load_basefw(adev);
+       if (ret)
+               goto reenable_gating;
+
+       mutex_lock(&adev->comp_list_mutex);
+       list_for_each_entry(acomp, &adev->comp_list, node) {
+               struct avs_tplg *tplg = acomp->tplg;
+
+               ret = avs_dsp_load_libraries(adev, tplg->libs, tplg->num_libs);
+               if (ret < 0)
+                       break;
+       }
+       mutex_unlock(&adev->comp_list_mutex);
 
+reenable_gating:
        avs_hda_l1sen_enable(adev, true);
        avs_hda_clock_gating_enable(adev, true);