OrgACRN
/
sof
mirror of https://github.com/thesofproject/sof.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Audio: SRC: Make stage buffer_start to be aligned at 8-byte boundary 

sbuf length is empirically found and used to compute buffer_start.

buffer_start address is used by some instructions that require
8-byte aligned addresses:

See: src/audio/src/src_hifi4.c:
220	/* Load two data samples from two channels */
221	AE_L32X2F24_XC(d0, dp, inc); /* r0, l0 */

buffer_start = cd->delay_lines + cd->param.sbuf_length

Then we make sure cd->delay_lines is aligned to 8 and we only add
multiple by 8 lenghts.

Suggested-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
This commit is contained in:
Daniel Baluta 2023-01-26 19:19:38 +02:00 committed by Daniel Baluta
parent bcc14074cd
commit f4be6ecc64
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/audio/src/src.c
View File

@ -141,7 +141,7 @@ static int src_buffer_lengths(struct comp_dev *dev, struct comp_data *cd,
struct src_param *a; struct src_param *a;
int fs_in, fs_out; int fs_in, fs_out;
int source_frames; int source_frames;
int r1; int r1, n;
a = &cd->param; a = &cd->param;
fs_in = cd->source_rate; fs_in = cd->source_rate;
@ -206,8 +206,8 @@ static int src_buffer_lengths(struct comp_dev *dev, struct comp_data *cd,
* variable number of blocks to process per each stage * variable number of blocks to process per each stage
* there is no equation known for minimum size. * there is no equation known for minimum size.
*/ */
a->sbuf_length = 2 * nch * stage1->blk_out * r1; n = 2 * stage1->blk_out * r1;
a->sbuf_length += a->sbuf_length >> 3; a->sbuf_length = nch * (n + (n >> 3));
} }
a->src_multich = a->fir_s1 + a->fir_s2 + a->out_s1 + a->out_s2; a->src_multich = a->fir_s1 + a->fir_s2 + a->out_s1 + a->out_s2;
@ -771,7 +771,12 @@ static int src_params_general(struct comp_dev *dev, struct comp_data *cd,
goto out; goto out;
} }
delay_lines_size = sizeof(int32_t) * cd->param.total; /*
* delay_lines_size is used to compute buffer_start which needs to
* be aligned to 8 bytes as required by some Xtensa
* instructions (e.g AE_L32X2F24_XC)
*/
delay_lines_size = ALIGN_UP(sizeof(int32_t) * cd->param.total, 8);
if (delay_lines_size == 0) { if (delay_lines_size == 0) {
comp_err(dev, "src_params(): delay_lines_size = 0"); comp_err(dev, "src_params(): delay_lines_size = 0");
@ -792,7 +797,7 @@ static int src_params_general(struct comp_dev *dev, struct comp_data *cd,
/* Clear all delay lines here */ /* Clear all delay lines here */
memset(cd->delay_lines, 0, delay_lines_size); memset(cd->delay_lines, 0, delay_lines_size);
buffer_start = cd->delay_lines + cd->param.sbuf_length; buffer_start = cd->delay_lines + ALIGN_UP(cd->param.sbuf_length, 2);
/* Initialize SRC for actual sample rate */ /* Initialize SRC for actual sample rate */
n = src_polyphase_init(&cd->src, &cd->param, buffer_start); n = src_polyphase_init(&cd->src, &cd->param, buffer_start);