diff options
| author | Cástor Muñoz <cmvidal@gmail.com> | 2017-01-13 20:51:07 +0100 |
|---|---|---|
| committer | Franklin Wei <git@fwei.tk> | 2018-07-30 18:50:27 -0400 |
| commit | 744274220833e017f3cc3f1b61abd0fc7c3885ec (patch) | |
| tree | 7b6fcdd7bc9e53bee6f62ed683ddf1db4c0c13ad /lib | |
| parent | 8f38f85fbde2ae78ba50cf9b9f54fbf9d25b4a45 (diff) | |
| download | rockbox-744274220833e017f3cc3f1b61abd0fc7c3885ec.zip rockbox-744274220833e017f3cc3f1b61abd0fc7c3885ec.tar.gz rockbox-744274220833e017f3cc3f1b61abd0fc7c3885ec.tar.bz2 rockbox-744274220833e017f3cc3f1b61abd0fc7c3885ec.tar.xz | |
iPod Classic: disable IRAM1
On Classic, IRAM1 (second 128Kb of a total of 256KB available IRAM) is
slower than DRAM. Codecs that actually are using regions of IRAM1 runs
faster when DRAM is used, so IRAM1 is disabled and only IRAM0 remains
enabled: 48KB for core and 80KB for codecs/plugins.
The next test_codec results shows how decode time is decreased:
file boosted unboosted
*.ra ~1.5% ~0.5%
*.mpc ~21% ~4.5%
*.ogg ~0.5% ~0%
nero_he*.m4a ~8% ~1%
nero*.m4a ~25% ~7%
wmapro*.wma ~4.5% ~0%
wma*.wma ~25% ~7%
In addition there is a small power save when IRAM1 HW is disabled.
Change-Id: I102adee11458e82037f23076d5d5956e23235de8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/rbcodec/codecs/libcook/cook.h | 7 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libfaad/common.h | 4 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libffmpegFLAC/bitstream.h | 5 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libmusepack/mpcdec.h | 4 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libspc/spc_codec.h | 4 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libwma/wmadec.h | 5 | ||||
| -rw-r--r-- | lib/rbcodec/codecs/libwmapro/wmaprodec.h | 4 |
7 files changed, 18 insertions, 15 deletions
diff --git a/lib/rbcodec/codecs/libcook/cook.h b/lib/rbcodec/codecs/libcook/cook.h index fcb437a..c2a5357 100644 --- a/lib/rbcodec/codecs/libcook/cook.h +++ b/lib/rbcodec/codecs/libcook/cook.h @@ -29,15 +29,16 @@ #include "codeclib.h" -#if (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == MCF5250) -/* PP5022/24, MCF5250 have large IRAM */ +#if (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || \ + (CONFIG_CPU == MCF5250) || (CONFIG_CPU == S5L8702) +/* PP5022/24, S5L8702, MCF5250 have large IRAM */ #define IBSS_ATTR_COOK_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_COOK_LARGE_IRAM ICODE_ATTR #define ICONST_ATTR_COOK_LARGE_IRAM ICONST_ATTR #define IBSS_ATTR_COOK_VLCBUF #define ICODE_ATTR_COOK_DECODE -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) /* S5L870X have even larger IRAM and it is faster to use ICODE_ATTR. */ #define IBSS_ATTR_COOK_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_COOK_LARGE_IRAM ICODE_ATTR diff --git a/lib/rbcodec/codecs/libfaad/common.h b/lib/rbcodec/codecs/libfaad/common.h index 720042b..c5eaa0a 100644 --- a/lib/rbcodec/codecs/libfaad/common.h +++ b/lib/rbcodec/codecs/libfaad/common.h @@ -58,14 +58,14 @@ extern struct codec_api* ci; #define ICONST_ATTR_FAAD_LARGE_IRAM ICONST_ATTR #define IBSS_ATTR_FAAD_XLR -#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) +#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == S5L8702) /* Enough IRAM to move additional data and code to it. */ #define IBSS_ATTR_FAAD_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_FAAD_LARGE_IRAM ICODE_ATTR #define ICONST_ATTR_FAAD_LARGE_IRAM ICONST_ATTR #define IBSS_ATTR_FAAD_XLR -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) /* Very large IRAM. Move even more data to it. */ #define IBSS_ATTR_FAAD_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_FAAD_LARGE_IRAM ICODE_ATTR diff --git a/lib/rbcodec/codecs/libffmpegFLAC/bitstream.h b/lib/rbcodec/codecs/libffmpegFLAC/bitstream.h index 2ec54c4..cd9589d 100644 --- a/lib/rbcodec/codecs/libffmpegFLAC/bitstream.h +++ b/lib/rbcodec/codecs/libffmpegFLAC/bitstream.h @@ -18,14 +18,15 @@ #define ICODE_ATTR #endif -#if (CONFIG_CPU == MCF5250) || (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) +#if (CONFIG_CPU == MCF5250) || (CONFIG_CPU == PP5022) || \ + (CONFIG_CPU == PP5024) || (CONFIG_CPU == S5L8702) #define ICODE_ATTR_FLAC ICODE_ATTR #define IBSS_ATTR_FLAC IBSS_ATTR /* Enough IRAM to move additional data to it. */ #define IBSS_ATTR_FLAC_LARGE_IRAM IBSS_ATTR #define IBSS_ATTR_FLAC_XLARGE_IRAM -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) #define ICODE_ATTR_FLAC ICODE_ATTR #define IBSS_ATTR_FLAC IBSS_ATTR /* Enough IRAM to move even more additional data to it. */ diff --git a/lib/rbcodec/codecs/libmusepack/mpcdec.h b/lib/rbcodec/codecs/libmusepack/mpcdec.h index 02003cb..24c06bc 100644 --- a/lib/rbcodec/codecs/libmusepack/mpcdec.h +++ b/lib/rbcodec/codecs/libmusepack/mpcdec.h @@ -58,7 +58,7 @@ extern "C" { /* Keep the data arrays of bitsreadr.c in IRAM. */ #define ICONST_ATTR_MPC_BITSREADER ICONST_ATTR -#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) +#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == S5L8702) /* Enough IRAM to move additional data and code to it. */ #define IBSS_ATTR_MPC_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_MPC_LARGE_IRAM ICODE_ATTR @@ -70,7 +70,7 @@ extern "C" { * sv7/sv8 bitstream demuxing into IRAM. This config is faster. */ #define ICONST_ATTR_MPC_BITSREADER -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) /* Enough IRAM to move additional data and code to it. */ #define IBSS_ATTR_MPC_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_MPC_LARGE_IRAM ICODE_ATTR diff --git a/lib/rbcodec/codecs/libspc/spc_codec.h b/lib/rbcodec/codecs/libspc/spc_codec.h index 446690f..75c3444 100644 --- a/lib/rbcodec/codecs/libspc/spc_codec.h +++ b/lib/rbcodec/codecs/libspc/spc_codec.h @@ -95,7 +95,7 @@ /* Not enough IRAM available to move further data to it. */ #define IBSS_ATTR_SPC_LARGE_IRAM -#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) +#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == S5L8702) #define IBSS_ATTR_SPC IBSS_ATTR #define ICODE_ATTR_SPC ICODE_ATTR #define ICONST_ATTR_SPC ICONST_ATTR @@ -103,7 +103,7 @@ /* Not enough IRAM available to move further data to it. */ #define IBSS_ATTR_SPC_LARGE_IRAM -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) #define IBSS_ATTR_SPC IBSS_ATTR #define ICODE_ATTR_SPC ICODE_ATTR #define ICONST_ATTR_SPC ICONST_ATTR diff --git a/lib/rbcodec/codecs/libwma/wmadec.h b/lib/rbcodec/codecs/libwma/wmadec.h index 76429de..b3bca4f 100644 --- a/lib/rbcodec/codecs/libwma/wmadec.h +++ b/lib/rbcodec/codecs/libwma/wmadec.h @@ -52,13 +52,14 @@ #define LSP_POW_BITS 7 -#if (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == MCF5250) +#if (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || \ + (CONFIG_CPU == MCF5250) || (CONFIG_CPU == S5L8702) /* PP5022/24 and MCF5250 have 128KB of IRAM. 80KB are allocated for codecs */ #define IBSS_ATTR_WMA_LARGE_IRAM IBSS_ATTR #define IBSS_ATTR_WMA_XL_IRAM #define ICONST_ATTR_WMA_XL_IRAM -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) /* S5L870x has even more IRAM. Use it. */ #define IBSS_ATTR_WMA_LARGE_IRAM IBSS_ATTR #define IBSS_ATTR_WMA_XL_IRAM IBSS_ATTR diff --git a/lib/rbcodec/codecs/libwmapro/wmaprodec.h b/lib/rbcodec/codecs/libwmapro/wmaprodec.h index d8398fc..46b4770 100644 --- a/lib/rbcodec/codecs/libwmapro/wmaprodec.h +++ b/lib/rbcodec/codecs/libwmapro/wmaprodec.h @@ -10,7 +10,7 @@ #define IBSS_ATTR_WMAPRO_VLC_TABLES #define ICONST_ATTR_WMAPRO_WIN_VS_TMP -#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) +#elif (CONFIG_CPU == PP5022) || (CONFIG_CPU == PP5024) || (CONFIG_CPU == S5L8702) /* Enough IRAM to move additional data and code to it. */ #define IBSS_ATTR_WMAPRO_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_WMAPRO_LARGE_IRAM ICODE_ATTR @@ -18,7 +18,7 @@ #define IBSS_ATTR_WMAPRO_VLC_TABLES #define ICONST_ATTR_WMAPRO_WIN_VS_TMP -#elif defined(CPU_S5L870X) +#elif (CONFIG_CPU == S5L8700) || (CONFIG_CPU == S5L8701) /* Enough IRAM to move additional data and code to it. */ #define IBSS_ATTR_WMAPRO_LARGE_IRAM IBSS_ATTR #define ICODE_ATTR_WMAPRO_LARGE_IRAM ICODE_ATTR |