len0rd/rockbox
1
0
Fork 1
mirror of https://github.com/Rockbox/rockbox.git synced 2025-12-08 20:55:17 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

rbcodec dsp: Remove INIT_ATTR from the DSP library

Browse source 
All of these are technically unsafe cross-section references but
most aren't reported by the linker, probably due to inlining. In
practice there was no problem because the affected code was only
run at init time anyway.

For now, remove INIT_ATTR until the init code can be refactored
to avoid the problematic references. This should also save code
size by moving more code to the init section.

dsp_init() gets to keep its attribute because it's already OK.

Change-Id: Idc9ac0e02cb07f31d186686e0382275c02a85dbb
This commit is contained in:
Aidan MacDonald 2022-12-03 15:04:06 +00:00
parent b650e774a1
commit 8165a6c245
8 changed files with 15 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

8
lib/rbcodec/dsp/dsp_sample_input.c
View file

@ -257,8 +257,8 @@ static void dsp_sample_input_flush(struct sample_io_data *this)
this->sample_buf.remcount = 0;
}
static void INIT_ATTR dsp_sample_input_init(struct sample_io_data *this,
enum dsp_ids dsp_id)
static void dsp_sample_input_init(struct sample_io_data *this,
enum dsp_ids dsp_id)
{
int32_t *lbuf, *rbuf;
@ -283,8 +283,8 @@ static void INIT_ATTR dsp_sample_input_init(struct sample_io_data *this,
this->sample_buf_p[1] = rbuf;
}
static void INIT_ATTR dsp_sample_io_init(struct sample_io_data *this,
enum dsp_ids dsp_id)
static void dsp_sample_io_init(struct sample_io_data *this,
enum dsp_ids dsp_id)
{
this->output_sampr = DSP_OUT_DEFAULT_HZ;
dsp_sample_input_init(this, dsp_id);