From b6bab28a5e04656ef3aa705d9d6799750b5eb5c5 Mon Sep 17 00:00:00 2001 From: DineshKumar Kalva Date: Tue, 11 Aug 2026 16:37:29 +0530 Subject: [PATCH 1/2] platform: amd: acp_7_x: move IPC interrupt to EXT_LEVEL5 Move the host to DSP software (IPC) interrupt from EXT_LEVEL3 to EXT_LEVEL5 so the ACP DMA log backend can own EXT_LEVEL3 for its DMA completion interrupt without colliding with IPC. Update the IPC handler registration and the PLATFORM_IPC_INTERRUPT definition accordingly, and fix a comment typo. Signed-off-by: DineshKumar Kalva --- src/drivers/amd/acp_7_x/ipc.c | 13 ++++++------- .../amd/acp_7_x/include/platform/platform.h | 2 +- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/src/drivers/amd/acp_7_x/ipc.c b/src/drivers/amd/acp_7_x/ipc.c index 777ad3e08494..35f8f6b0bb08 100644 --- a/src/drivers/amd/acp_7_x/ipc.c +++ b/src/drivers/amd/acp_7_x/ipc.c @@ -42,8 +42,7 @@ LOG_MODULE_REGISTER(ipc_handler_file, LOG_LEVEL_DBG); #define HOST_TO_DSP_INTR 1 #define INTERRUPT_DISABLE 0 -#define IRQ_NUM_EXT_LEVEL3 3 -#define IRQ_EXT_IPC_LEVEL_3 3 +#define IRQ_EXT_IPC_LEVEL_5 5 /* This function triggers a host interrupt from ACP DSP */ void acp_dsp_to_host_intr_trig(void) @@ -123,11 +122,11 @@ int platform_ipc_init(struct ipc *ipc) /* schedule */ schedule_task_init_edf(&ipc->ipc_task, SOF_UUID(ipc_task_amd_uuid), &ipc_task_ops, ipc, 0, 0); - interrupt_disable(IRQ_EXT_IPC_LEVEL_3, ipc); - interrupt_unregister(IRQ_EXT_IPC_LEVEL_3, ipc); - interrupt_register(IRQ_EXT_IPC_LEVEL_3, amd_irq_handler, ipc); - /* Enabling software interuppts */ - interrupt_enable(IRQ_EXT_IPC_LEVEL_3, ipc); + interrupt_disable(IRQ_EXT_IPC_LEVEL_5, ipc); + interrupt_unregister(IRQ_EXT_IPC_LEVEL_5, ipc); + interrupt_register(IRQ_EXT_IPC_LEVEL_5, amd_irq_handler, ipc); + /* Enabling software interrupts */ + interrupt_enable(IRQ_EXT_IPC_LEVEL_5, ipc); return 0; } diff --git a/src/platform/amd/acp_7_x/include/platform/platform.h b/src/platform/amd/acp_7_x/include/platform/platform.h index 2a6b0147446d..a44ffab57c59 100644 --- a/src/platform/amd/acp_7_x/include/platform/platform.h +++ b/src/platform/amd/acp_7_x/include/platform/platform.h @@ -27,7 +27,7 @@ struct timer; #define PLATFORM_DEFAULT_CLOCK CLK_CPU(0) /* IPC Interrupt */ -#define PLATFORM_IPC_INTERRUPT IRQ_EXT_IPC_LEVEL_3 +#define PLATFORM_IPC_INTERRUPT 5 /* EXT_LEVEL5 */ #define PLATFORM_IPC_INTERRUPT_NAME NULL /* Host page size */ From 4831ebec87614307b035564436fb6f6a1eba6f38 Mon Sep 17 00:00:00 2001 From: DineshKumar Kalva Date: Tue, 11 Aug 2026 16:37:36 +0530 Subject: [PATCH 2/2] acp_7_x: board: enable ACP DMA log backend Enable CONFIG_LOG_BACKEND_ACP on the ACP 7.X ADSP board and the firmware logs are streamed to the host over ACP DMA. Set the SOF log level to INF explicitly, as the SOF log level configuration must be selected explicitly. Signed-off-by: DineshKumar Kalva --- app/boards/acp_7_x_adsp.conf | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/app/boards/acp_7_x_adsp.conf b/app/boards/acp_7_x_adsp.conf index 04c63271c9ac..9c4d3df561b7 100644 --- a/app/boards/acp_7_x_adsp.conf +++ b/app/boards/acp_7_x_adsp.conf @@ -5,8 +5,6 @@ CONFIG_DCACHE_LINE_SIZE=128 CONFIG_DYNAMIC_INTERRUPTS=y CONFIG_SHARED_INTERRUPTS=n CONFIG_ZEPHYR_LOG=y -CONFIG_LOG_MODE_DEFERRED=n -CONFIG_LOG_MODE_MINIMAL=y CONFIG_DMA=y CONFIG_DMA_DOMAIN=n CONFIG_ZEPHYR_NATIVE_DRIVERS=y @@ -55,3 +53,12 @@ CONFIG_CORE_COUNT=1 CONFIG_FORMAT_CONVERT_HIFI3=n CONFIG_PCM_CONVERTER_FORMAT_S24_4LE_MSB=y CONFIG_XTENSA_INTERRUPT_NONPREEMPTABLE=y +CONFIG_LOG_BACKEND_ACP=y +# Set INF log level: SOF log level configs must be set explicitly +CONFIG_LOG_DEFAULT_LEVEL=3 +CONFIG_LOG_MAX_LEVEL=3 +CONFIG_SOF_LOG_LEVEL_OFF=n +CONFIG_SOF_LOG_LEVEL_ERR=n +CONFIG_SOF_LOG_LEVEL_WRN=n +CONFIG_SOF_LOG_LEVEL_INF=y +CONFIG_SOF_LOG_LEVEL_DBG=n