audio: base_fw: add platform layer to hw_config data

Some of the HW_CONFIG fields are platform specific and cannot be filled with generic means in generic code. Handle this data via a simple platform abstraction to hook platform specific code to HW_CONFIG handling. Move out existing code for Intel cAVS and ACE platforms. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
thesofproject · Apr 8, 2024 · b896576 · b896576
1 parent f77bdfb
commit b896576
Show file tree

Hide file tree

Showing 4 changed files with 69 additions and 12 deletions.
diff --git a/src/audio/base_fw.c b/src/audio/base_fw.c
@@ -146,21 +146,12 @@ static int basefw_hw_config(uint32_t *data_offset, char *data)
 	value = SOF_DIV_ROUND_UP(EBB_BANKS_IN_SEGMENT * SRAM_BANK_SIZE, HOST_PAGE_SIZE);
 	tlv_value_uint32_set(tuple, IPC4_TOTAL_PHYS_MEM_PAGES_HW_CFG, value);
 
-	tuple = tlv_next(tuple);
-	tlv_value_uint32_set(tuple, IPC4_HP_EBB_COUNT_HW_CFG, PLATFORM_HPSRAM_EBB_COUNT);
-
-	tuple = tlv_next(tuple);
-	/* 2 DMIC dais */
-	value =  DAI_NUM_SSP_BASE + DAI_NUM_HDA_IN + DAI_NUM_HDA_OUT +
-			DAI_NUM_ALH_BI_DIR_LINKS + 2;
-	tlv_value_uint32_set(tuple, IPC4_GATEWAY_COUNT_HW_CFG, value);
-
-	tuple = tlv_next(tuple);
-	tlv_value_uint32_set(tuple, IPC4_LP_EBB_COUNT_HW_CFG, PLATFORM_LPSRAM_EBB_COUNT);
-
 	tuple = tlv_next(tuple);
 	*data_offset = (int)((char *)tuple - data);
 
+	/* add platform specific tuples */
+	platform_basefw_hw_config(data_offset, (char *)tuple);
+
 	return 0;
 }
 

diff --git a/src/include/sof/platform.h b/src/include/sof/platform.h
@@ -52,6 +52,16 @@ int platform_init(struct sof *sof);
  */
 int platform_context_save(struct sof *sof);
 
+/**
+ * \brief Platform specific routine to add data tuples to HW_CONFIG
+ * structure sent to host via IPC.
+ * \param[out] data_offset data offset after tuples added
+ * \parma[in] data pointer where to add new entries
+ * \return 0 if successful, error code otherwise.
+ */
+int platform_basefw_hw_config(uint32_t *data_offset, char *data);
+
+
 /** @}*/
 
 #endif

diff --git a/src/platform/intel/ace/platform.c b/src/platform/intel/ace/platform.c
@@ -19,13 +19,18 @@
 #include <sof/schedule/dp_schedule.h>
 #include <sof/schedule/ll_schedule.h>
 #include <sof/schedule/ll_schedule_domain.h>
+#include <sof/tlv.h>
 #include <sof/trace/trace.h>
 #include <ipc/header.h>
 #include <ipc/info.h>
 #include <kernel/abi.h>
 #include <rtos/clk.h>
 #include <sof/lib/cpu.h>
 
+#if defined(CONFIG_IPC_MAJOR_4)
+#include <ipc4/base_fw.h>
+#endif
+
 #include <sof_versions.h>
 #include <stdint.h>
 
@@ -140,3 +145,26 @@ int platform_context_save(struct sof *sof)
 {
 	return 0;
 }
+
+int platform_basefw_hw_config(uint32_t *data_offset, char *data)
+{
+	struct sof_tlv *tuple = (struct sof_tlv *)data;
+	uint32_t value;
+
+	tuple = tlv_next(tuple);
+	tlv_value_uint32_set(tuple, IPC4_HP_EBB_COUNT_HW_CFG, PLATFORM_HPSRAM_EBB_COUNT);
+
+	tuple = tlv_next(tuple);
+	/* 2 DMIC dais */
+	value =  DAI_NUM_SSP_BASE + DAI_NUM_HDA_IN + DAI_NUM_HDA_OUT +
+			DAI_NUM_ALH_BI_DIR_LINKS + 2;
+	tlv_value_uint32_set(tuple, IPC4_GATEWAY_COUNT_HW_CFG, value);
+
+	tuple = tlv_next(tuple);
+	tlv_value_uint32_set(tuple, IPC4_LP_EBB_COUNT_HW_CFG, PLATFORM_LPSRAM_EBB_COUNT);
+
+	tuple = tlv_next(tuple);
+	*data_offset = (int)((char *)tuple - data);
+
+	return 0;
+}
diff --git a/src/platform/intel/cavs/platform.c b/src/platform/intel/cavs/platform.c
@@ -33,11 +33,16 @@
 #include <sof/schedule/ll_schedule.h>
 #include <sof/schedule/ll_schedule_domain.h>
 #include <sof/trace/trace.h>
+#include <sof/tlv.h>
 #include <ipc/header.h>
 #include <ipc/info.h>
 #include <kernel/abi.h>
 #include <kernel/ext_manifest.h>
 
+#if defined(CONFIG_IPC_MAJOR_4)
+#include <ipc4/base_fw.h>
+#endif
+
 #include <sof_versions.h>
 #include <errno.h>
 #include <stdint.h>
@@ -239,3 +244,26 @@ int platform_context_save(struct sof *sof)
 #endif
 	return 0;
 }
+
+int platform_basefw_hw_config(uint32_t *data_offset, char *data)
+{
+	struct sof_tlv *tuple = (struct sof_tlv *)data;
+	uint32_t value;
+
+	tuple = tlv_next(tuple);
+	tlv_value_uint32_set(tuple, IPC4_HP_EBB_COUNT_HW_CFG, PLATFORM_HPSRAM_EBB_COUNT);
+
+	tuple = tlv_next(tuple);
+	/* 2 DMIC dais */
+	value =  DAI_NUM_SSP_BASE + DAI_NUM_HDA_IN + DAI_NUM_HDA_OUT +
+			DAI_NUM_ALH_BI_DIR_LINKS + 2;
+	tlv_value_uint32_set(tuple, IPC4_GATEWAY_COUNT_HW_CFG, value);
+
+	tuple = tlv_next(tuple);
+	tlv_value_uint32_set(tuple, IPC4_LP_EBB_COUNT_HW_CFG, PLATFORM_LPSRAM_EBB_COUNT);
+
+	tuple = tlv_next(tuple);
+	*data_offset = (int)((char *)tuple - data);
+
+	return 0;
+}