mirror of https://github.com/thesofproject/sof.git
various components: Remove default/stub init_alignment_constants usage
Traditionally audio_stream has failed to initialize its computed alignment fields, forcing components to do this themselves even when they don't actually have special alignment requirements. Remove all the code that was merely setting default values, leaving only a handful of spots with specialr equirements (e.g. eq/area need to treat pairs of samples, a few others have HiFi-optimized variants which need SIMD alignment). Signed-off-by: Andy Ross <andyross@google.com>
This commit is contained in:
parent
4175ac7ad0
commit
fec9e990e5
|
@ -162,9 +162,7 @@ static void aria_set_stream_params(struct comp_buffer *buffer,
|
|||
const struct ipc4_audio_format *audio_fmt = &mod->priv.cfg.base_cfg.audio_fmt;
|
||||
|
||||
ipc4_update_buffer_format(buffer, audio_fmt);
|
||||
#ifdef ARIA_GENERIC
|
||||
audio_stream_init_alignment_constants(1, 1, &buffer->stream);
|
||||
#else
|
||||
#ifndef ARIA_GENERIC
|
||||
audio_stream_init_alignment_constants(8, 1, &buffer->stream);
|
||||
#endif
|
||||
}
|
||||
|
|
|
@ -550,9 +550,6 @@ static int asrc_prepare(struct processing_module *mod,
|
|||
sinkb = list_first_item(&dev->bsink_list,
|
||||
struct comp_buffer, source_list);
|
||||
|
||||
audio_stream_init_alignment_constants(1, 1, &sourceb->stream);
|
||||
audio_stream_init_alignment_constants(1, 1, &sinkb->stream);
|
||||
|
||||
/* get source data format and period bytes */
|
||||
cd->source_format = audio_stream_get_frm_fmt(&sourceb->stream);
|
||||
source_period_bytes = audio_stream_period_bytes(&sourceb->stream,
|
||||
|
|
|
@ -546,14 +546,11 @@ static int crossover_prepare(struct processing_module *mod,
|
|||
/* Get source data format */
|
||||
cd->source_format = audio_stream_get_frm_fmt(&source->stream);
|
||||
channels = audio_stream_get_channels(&source->stream);
|
||||
audio_stream_init_alignment_constants(1, 1, &source->stream);
|
||||
|
||||
/* Validate frame format and buffer size of sinks */
|
||||
list_for_item(sink_list, &dev->bsink_list) {
|
||||
sink = container_of(sink_list, struct comp_buffer, source_list);
|
||||
if (cd->source_format == audio_stream_get_frm_fmt(&sink->stream)) {
|
||||
audio_stream_init_alignment_constants(1, 1, &sink->stream);
|
||||
} else {
|
||||
if (cd->source_format != audio_stream_get_frm_fmt(&sink->stream)) {
|
||||
comp_err(dev, "crossover_prepare(): Source fmt %d and sink fmt %d are different.",
|
||||
cd->source_format, audio_stream_get_frm_fmt(&sink->stream));
|
||||
ret = -EINVAL;
|
||||
|
|
|
@ -183,17 +183,6 @@ static int dcblock_process(struct processing_module *mod,
|
|||
return 0;
|
||||
}
|
||||
|
||||
/* init and calculate the aligned setting for available frames and free frames retrieve*/
|
||||
static inline void dcblock_set_frame_alignment(struct audio_stream *source,
|
||||
struct audio_stream *sink)
|
||||
{
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, source);
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, sink);
|
||||
}
|
||||
|
||||
/**
|
||||
* \brief Prepares DC Blocking Filter component for processing.
|
||||
* \param[in,out] dev DC Blocking Filter base component device.
|
||||
|
@ -221,8 +210,6 @@ static int dcblock_prepare(struct processing_module *mod,
|
|||
/* get sink data format and period bytes */
|
||||
cd->sink_format = audio_stream_get_frm_fmt(&sinkb->stream);
|
||||
|
||||
dcblock_set_frame_alignment(&sourceb->stream, &sinkb->stream);
|
||||
|
||||
dcblock_init_state(cd);
|
||||
cd->dcblock_func = dcblock_find_func(cd->source_format);
|
||||
if (!cd->dcblock_func) {
|
||||
|
|
|
@ -260,14 +260,6 @@ static int drc_get_config(struct processing_module *mod,
|
|||
return comp_data_blob_get_cmd(cd->model_handler, cdata, fragment_size);
|
||||
}
|
||||
|
||||
static void drc_set_alignment(struct audio_stream *source,
|
||||
struct audio_stream *sink)
|
||||
{
|
||||
/* Currently no optimizations those would use wider loads and stores */
|
||||
audio_stream_init_alignment_constants(1, 1, source);
|
||||
audio_stream_init_alignment_constants(1, 1, sink);
|
||||
}
|
||||
|
||||
static int drc_process(struct processing_module *mod,
|
||||
struct input_stream_buffer *input_buffers,
|
||||
int num_input_buffers,
|
||||
|
@ -344,7 +336,6 @@ static int drc_prepare(struct processing_module *mod,
|
|||
/* DRC component will only ever have 1 source and 1 sink buffer */
|
||||
sourceb = list_first_item(&dev->bsource_list, struct comp_buffer, sink_list);
|
||||
sinkb = list_first_item(&dev->bsink_list, struct comp_buffer, source_list);
|
||||
drc_set_alignment(&sourceb->stream, &sinkb->stream);
|
||||
|
||||
/* get source data format */
|
||||
cd->source_format = audio_stream_get_frm_fmt(&sourceb->stream);
|
||||
|
|
|
@ -576,7 +576,6 @@ static int google_rtc_audio_processing_prepare(struct processing_module *mod,
|
|||
microphone_stream_channels);
|
||||
}
|
||||
|
||||
audio_stream_init_alignment_constants(1, 1, &source->stream);
|
||||
i++;
|
||||
}
|
||||
|
||||
|
@ -592,7 +591,6 @@ static int google_rtc_audio_processing_prepare(struct processing_module *mod,
|
|||
return -EINVAL;
|
||||
}
|
||||
|
||||
audio_stream_init_alignment_constants(1, 1, &output->stream);
|
||||
frame_fmt = audio_stream_get_frm_fmt(&output->stream);
|
||||
rate = audio_stream_get_rate(&output->stream);
|
||||
output_stream_channels = audio_stream_get_channels(&output->stream);
|
||||
|
|
|
@ -893,16 +893,12 @@ static int kpb_prepare(struct comp_dev *dev)
|
|||
*/
|
||||
if (kpb->ipc4_cfg.base_cfg.ibs != kpb->ipc4_cfg.base_cfg.obs) {
|
||||
struct list_item *sink_list;
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
uint32_t sink_id;
|
||||
|
||||
list_for_item(sink_list, &dev->bsink_list) {
|
||||
struct comp_buffer *sink =
|
||||
container_of(sink_list, struct comp_buffer, source_list);
|
||||
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req,
|
||||
&sink->stream);
|
||||
sink_id = buf_get_id(sink);
|
||||
|
||||
if (sink_id == 0)
|
||||
|
|
|
@ -177,15 +177,6 @@ static int mfcc_process(struct processing_module *mod,
|
|||
return 0;
|
||||
}
|
||||
|
||||
static void mfcc_set_alignment(struct audio_stream *source, struct audio_stream *sink)
|
||||
{
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, source);
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, sink);
|
||||
}
|
||||
|
||||
static int mfcc_prepare(struct processing_module *mod,
|
||||
struct sof_source **sources, int num_of_sources,
|
||||
struct sof_sink **sinks, int num_of_sinks)
|
||||
|
@ -208,9 +199,6 @@ static int mfcc_prepare(struct processing_module *mod,
|
|||
/* get source data format */
|
||||
source_format = audio_stream_get_frm_fmt(&sourceb->stream);
|
||||
|
||||
/* set align requirements */
|
||||
mfcc_set_alignment(&sourceb->stream, &sinkb->stream);
|
||||
|
||||
/* get sink data format and period bytes */
|
||||
sink_format = audio_stream_get_frm_fmt(&sinkb->stream);
|
||||
sink_period_bytes = audio_stream_period_bytes(&sinkb->stream, dev->frames);
|
||||
|
|
|
@ -205,17 +205,8 @@ static inline void mixer_set_frame_alignment(struct audio_stream *source)
|
|||
/*There is no limit for frame number, so set it as 1*/
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
#else
|
||||
|
||||
/* Since the generic version process signal sample by sample, so there is no
|
||||
* limit for it, then set the byte_align and frame_align_req to be 1.
|
||||
*/
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
#endif
|
||||
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, source);
|
||||
#endif
|
||||
}
|
||||
|
||||
static int mixer_prepare(struct processing_module *mod,
|
||||
|
|
|
@ -310,14 +310,6 @@ static int multiband_drc_get_config(struct processing_module *mod,
|
|||
return multiband_drc_get_ipc_config(mod, cdata, fragment_size);
|
||||
}
|
||||
|
||||
static void multiband_drc_set_alignment(struct audio_stream *source,
|
||||
struct audio_stream *sink)
|
||||
{
|
||||
/* Currently no optimizations those would use wider loads and stores */
|
||||
audio_stream_init_alignment_constants(1, 1, source);
|
||||
audio_stream_init_alignment_constants(1, 1, sink);
|
||||
}
|
||||
|
||||
static int multiband_drc_process(struct processing_module *mod,
|
||||
struct input_stream_buffer *input_buffers, int num_input_buffers,
|
||||
struct output_stream_buffer *output_buffers,
|
||||
|
@ -356,7 +348,7 @@ static int multiband_drc_prepare(struct processing_module *mod,
|
|||
{
|
||||
struct multiband_drc_comp_data *cd = module_get_private_data(mod);
|
||||
struct comp_dev *dev = mod->dev;
|
||||
struct comp_buffer *sourceb, *sinkb;
|
||||
struct comp_buffer *sourceb;
|
||||
int channels;
|
||||
int rate;
|
||||
int ret = 0;
|
||||
|
@ -369,9 +361,6 @@ static int multiband_drc_prepare(struct processing_module *mod,
|
|||
|
||||
/* DRC component will only ever have 1 source and 1 sink buffer */
|
||||
sourceb = list_first_item(&dev->bsource_list, struct comp_buffer, sink_list);
|
||||
sinkb = list_first_item(&dev->bsink_list, struct comp_buffer, source_list);
|
||||
|
||||
multiband_drc_set_alignment(&sourceb->stream, &sinkb->stream);
|
||||
|
||||
/* get source data format */
|
||||
cd->source_format = audio_stream_get_frm_fmt(&sourceb->stream);
|
||||
|
|
|
@ -378,12 +378,8 @@ static int mux_prepare(struct processing_module *mod,
|
|||
{
|
||||
struct comp_dev *dev = mod->dev;
|
||||
struct comp_data *cd = module_get_private_data(mod);
|
||||
struct list_item *blist;
|
||||
struct comp_buffer *source;
|
||||
struct comp_buffer *sink;
|
||||
struct sof_mux_config *config;
|
||||
size_t blob_size;
|
||||
int state;
|
||||
int ret;
|
||||
|
||||
comp_dbg(dev, "mux_prepare()");
|
||||
|
@ -410,23 +406,6 @@ static int mux_prepare(struct processing_module *mod,
|
|||
return -EINVAL;
|
||||
}
|
||||
|
||||
/* check each mux source state, set source align to 1 byte, 1 frame */
|
||||
list_for_item(blist, &dev->bsource_list) {
|
||||
source = container_of(blist, struct comp_buffer, sink_list);
|
||||
state = source->source->state;
|
||||
audio_stream_init_alignment_constants(1, 1, &source->stream);
|
||||
|
||||
/* only prepare downstream if we have no active sources */
|
||||
if (state == COMP_STATE_PAUSED || state == COMP_STATE_ACTIVE)
|
||||
return PPL_STATUS_PATH_STOP;
|
||||
}
|
||||
|
||||
/* set sink align to 1 byte, 1 frame */
|
||||
list_for_item(blist, &dev->bsink_list) {
|
||||
sink = container_of(blist, struct comp_buffer, source_list);
|
||||
audio_stream_init_alignment_constants(1, 1, &sink->stream);
|
||||
}
|
||||
|
||||
/* prepare downstream */
|
||||
return 0;
|
||||
}
|
||||
|
|
|
@ -80,8 +80,6 @@ static void set_mux_params(struct processing_module *mod)
|
|||
struct comp_buffer *sink, *source;
|
||||
struct list_item *source_list;
|
||||
int j;
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
params->direction = dev->direction;
|
||||
params->channels = cd->md.base_cfg.audio_fmt.channels_count;
|
||||
|
@ -101,8 +99,6 @@ static void set_mux_params(struct processing_module *mod)
|
|||
/* update sink format */
|
||||
if (!list_is_empty(&dev->bsink_list)) {
|
||||
sink = list_first_item(&dev->bsink_list, struct comp_buffer, source_list);
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req,
|
||||
&sink->stream);
|
||||
|
||||
if (!sink->hw_params_configured) {
|
||||
ipc4_update_buffer_format(sink, &cd->md.output_format);
|
||||
|
@ -117,8 +113,6 @@ static void set_mux_params(struct processing_module *mod)
|
|||
list_for_item(source_list, &dev->bsource_list)
|
||||
{
|
||||
source = container_of(source_list, struct comp_buffer, sink_list);
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req,
|
||||
&source->stream);
|
||||
j = buf_get_id(source);
|
||||
cd->config.streams[j].pipeline_id = source->pipeline_id;
|
||||
if (j == BASE_CFG_QUEUED_ID)
|
||||
|
|
|
@ -623,7 +623,6 @@ static void volume_set_alignment(struct audio_stream *source,
|
|||
struct audio_stream *sink)
|
||||
{
|
||||
#if XCHAL_HAVE_HIFI3 || XCHAL_HAVE_HIFI4
|
||||
|
||||
/* Both source and sink buffer in HiFi 3 or HiFi4 processing version,
|
||||
* xtensa intrinsics ask for 8-byte aligned. 5.1 format SSE audio
|
||||
* requires 16-byte aligned.
|
||||
|
@ -633,18 +632,9 @@ static void volume_set_alignment(struct audio_stream *source,
|
|||
/*There is no limit for frame number, so both source and sink set it to be 1*/
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
#else
|
||||
|
||||
/* Since the generic version process signal sample by sample, so there is no
|
||||
* limit for it, then set the byte_align and frame_align_req to be 1.
|
||||
*/
|
||||
const uint32_t byte_align = 1;
|
||||
const uint32_t frame_align_req = 1;
|
||||
|
||||
#endif
|
||||
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, source);
|
||||
audio_stream_init_alignment_constants(byte_align, frame_align_req, sink);
|
||||
#endif
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
Loading…
Reference in New Issue