Skip to content

Commit c00d82c

Browse files
author
Jyri Sarha
committed
ipc4: Replace interim_heap_bytes and lifetime_heap_bytes with heap_bytes
Simplify the IPC4 memory data structs by replacing separate interim_heap_bytes, lifetime_heap_bytes, and shared_bytes fields with a single heap_bytes field, matching the Linux driver side changes. Affects both ipc4_module_init_ext_obj_dp_data and ipc4_pipeline_ext_obj_mem_data structs, and their corresponding log messages. Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
1 parent ab33679 commit c00d82c

4 files changed

Lines changed: 6 additions & 12 deletions

File tree

src/audio/module_adapter/module_adapter_ipc4.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,9 @@ int module_ext_init_decode(const struct comp_driver *drv, struct module_ext_init
8181
}
8282
ext_data->dp_data = dp_data;
8383
comp_cl_info(drv,
84-
"init_ext_obj_dp_data domain %u stack %u interim %u lifetime %u shared %u",
84+
"init_ext_obj_dp_data domain %u stack %u heap %u",
8585
dp_data->domain_id, dp_data->stack_bytes,
86-
dp_data->interim_heap_bytes, dp_data->lifetime_heap_bytes,
87-
dp_data->shared_bytes);
86+
dp_data->heap_bytes);
8887
break;
8988
}
9089
case IPC4_MOD_INIT_DATA_ID_MODULE_DATA:

src/include/ipc4/module.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,7 @@ struct ipc4_module_init_ext_object {
9393
struct ipc4_module_init_ext_obj_dp_data {
9494
uint32_t domain_id; /* userspace domain ID */
9595
uint32_t stack_bytes; /* required stack size in bytes */
96-
uint32_t interim_heap_bytes; /* required interim heap size in bytes */
97-
uint32_t lifetime_heap_bytes; /* required lifetime heap size in bytes */
98-
uint32_t shared_bytes; /* required shared memory size in bytes */
96+
uint32_t heap_bytes; /* required heap size in bytes */
9997
} __attribute__((packed, aligned(4)));
10098

10199
/*

src/include/ipc4/pipeline.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,7 @@ struct ipc4_pipeline_ext_object {
8888
struct ipc4_pipeline_ext_obj_mem_data {
8989
uint32_t domain_id; /* userspace domain ID */
9090
uint32_t stack_bytes; /* required stack size in bytes */
91-
uint32_t interim_heap_bytes; /* required interim heap size in bytes */
92-
uint32_t lifetime_heap_bytes; /* required lifetime heap size in bytes */
93-
uint32_t shared_bytes; /* required shared memory in bytes */
91+
uint32_t heap_bytes; /* required heap size in bytes */
9492
} __packed __aligned(4);
9593

9694
/*

src/ipc/ipc4/helper.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -317,10 +317,9 @@ __cold static int ipc4_create_pipeline_payload_decode(char *data,
317317
}
318318
pparams->mem_data = mem_data;
319319
tr_info(&ipc_tr,
320-
"init_ext_obj_mem_data domain %u stack %u interim %u lifetime %u shared %u",
320+
"init_ext_obj_mem_data domain %u stack %u heap %u",
321321
mem_data->domain_id, mem_data->stack_bytes,
322-
mem_data->interim_heap_bytes, mem_data->lifetime_heap_bytes,
323-
mem_data->shared_bytes);
322+
mem_data->heap_bytes);
324323
break;
325324
}
326325
default:

0 commit comments

Comments
 (0)