Skip to content

Commit d847de3

Browse files
jxsteltersoftwarecki
authored andcommitted
iadk: Fix GetConfiguration API function
The GetConfiguration function was incorrectly defined for IADK module adapter. This patch fixes the definition. Signed-off-by: Jaroslaw Stelter <Jaroslaw.Stelter@intel.com> Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
1 parent efbe395 commit d847de3

3 files changed

Lines changed: 8 additions & 8 deletions

File tree

src/audio/module_adapter/iadk/iadk_module_adapter.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,9 @@ IadkModuleAdapter::IadkModuleAdapter_SetConfiguration(uint32_t config_id,
121121
AdspErrorCode
122122
IadkModuleAdapter::IadkModuleAdapter_GetConfiguration(uint32_t config_id,
123123
enum module_cfg_fragment_position pos,
124-
uint32_t data_offset_size,
124+
uint32_t &data_offset_size,
125125
uint8_t *fragment_buffer,
126-
size_t fragment_size)
126+
size_t &fragment_size)
127127
{
128128
intel_adsp::ConfigurationFragmentPosition fragment_position =
129129
(intel_adsp::ConfigurationFragmentPosition::Enum) pos;
@@ -219,12 +219,12 @@ int iadk_wrapper_set_configuration(void *md, uint32_t config_id,
219219

220220
int iadk_wrapper_get_configuration(void *md, uint32_t config_id,
221221
enum module_cfg_fragment_position pos,
222-
uint32_t data_offset_size,
222+
uint32_t *data_offset_size,
223223
uint8_t *fragment, size_t fragment_size)
224224
{
225225
struct IadkModuleAdapter *mod_adp = (struct IadkModuleAdapter *) md;
226226
return mod_adp->IadkModuleAdapter_GetConfiguration(config_id, pos,
227-
data_offset_size,
227+
*data_offset_size,
228228
fragment,
229229
fragment_size);
230230
}

src/audio/module_adapter/module/modules.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ static int modules_get_configuration(struct processing_module *mod, uint32_t con
191191
size_t fragment_size)
192192
{
193193
return iadk_wrapper_get_configuration(module_get_private_data(mod), config_id,
194-
MODULE_CFG_FRAGMENT_SINGLE, *data_offset_size,
194+
MODULE_CFG_FRAGMENT_SINGLE, data_offset_size,
195195
fragment, fragment_size);
196196
}
197197

src/include/sof/audio/module_adapter/iadk/iadk_module_adapter.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ namespace dsp_fw
9191
AdspErrorCode
9292
IadkModuleAdapter_GetConfiguration(uint32_t config_id,
9393
enum module_cfg_fragment_position fragment_position,
94-
uint32_t data_offset_size,
94+
uint32_t &data_offset_size,
9595
uint8_t *fragment_buffer,
96-
size_t fragment_size);
96+
size_t &fragment_size);
9797
/**
9898
* Module specific reset procedure, called as part of codec_adapter component
9999
* reset in .reset(). This should reset all parameters to their initial stage
@@ -146,7 +146,7 @@ int iadk_wrapper_set_configuration(void *md, uint32_t config_id,
146146

147147
int iadk_wrapper_get_configuration(void *md, uint32_t config_id,
148148
enum module_cfg_fragment_position pos,
149-
uint32_t data_offset_size,
149+
uint32_t *data_offset_size,
150150
uint8_t *fragment, size_t fragment_size);
151151

152152
int iadk_wrapper_process(void *md,

0 commit comments

Comments
 (0)