Don't include the .bss and .ibss sections in the binary image for codecs and plugins. Saves quite some disk space (and buffer space in case of codec changes during playback).

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@8308 a1c6a512-1295-4272-9138-f99709370657

13 files changed, 70 insertions, 24 deletions

diff --git a/apps/codecs/a52.c b/apps/codecs/a52.c
index ee80f66..dd504e8 100644
--- a/apps/codecs/a52.c
+++ b/apps/codecs/a52.c
@@ -117,6 +117,8 @@ void a52_decode_data(uint8_t *start, uint8_t *end)
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* this is the codec entry point */
@@ -132,6 +134,7 @@ enum codec_status codec_start(struct codec_api *api)
 
     #ifdef USE_IRAM 
     ci->memcpy(iramstart, iramcopy, iramend - iramstart);
+    ci->memset(iedata, 0, iend - iedata);
     #endif
 
     ci->configure(CODEC_DSP_ENABLE, (bool *)true);
diff --git a/apps/codecs/alac.c b/apps/codecs/alac.c
index 58ab111..30c1ada 100644
--- a/apps/codecs/alac.c
+++ b/apps/codecs/alac.c
@@ -21,10 +21,12 @@
 #include "libm4a/m4a.h"
 #include "libalac/decomp.h"
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 int32_t outputbuffer[ALAC_MAX_CHANNELS][ALAC_BLOCKSIZE] IBSS_ATTR;
@@ -53,8 +55,9 @@ enum codec_status codec_start(struct codec_api* api)
   rb = api;
   ci = (struct codec_api*)api;
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
   rb->memcpy(iramstart, iramcopy, iramend-iramstart);
+  rb->memset(iedata, 0, iend - iedata);
 #endif
 
   ci->configure(CODEC_SET_FILEBUF_WATERMARK, (int *)(1024*512));
diff --git a/apps/codecs/flac.c b/apps/codecs/flac.c
index 7639b9a..ae05b0f 100644
--- a/apps/codecs/flac.c
+++ b/apps/codecs/flac.c
@@ -20,10 +20,12 @@
 #include "codeclib.h"
 #include <codecs/libffmpegFLAC/decoder.h>
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 struct codec_api* rb;
@@ -229,8 +231,9 @@ enum codec_status codec_start(struct codec_api* api)
     rb = api;
     ci = (struct codec_api*)api;
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
     ci->memcpy(iramstart, iramcopy, iramend-iramstart);
+    ci->memset(iedata, 0, iend - iedata);
 #endif
 
     ci->configure(CODEC_SET_FILEBUF_WATERMARK, (int *)(1024*512));
diff --git a/apps/codecs/mpa.c b/apps/codecs/mpa.c
index 3e6a045..3d824d9 100644
--- a/apps/codecs/mpa.c
+++ b/apps/codecs/mpa.c
@@ -43,6 +43,8 @@ int mpeg_latency[3] = { 0, 481, 529 };
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 struct codec_api *ci;
@@ -83,6 +85,7 @@ enum codec_status codec_start(struct codec_api *api)
 
 #ifdef USE_IRAM
     ci->memcpy(iramstart, iramcopy, iramend - iramstart);
+    ci->memset(iedata, 0, iend - iedata);
 #endif
 
     if (codec_init(api))
diff --git a/apps/codecs/mpc.c b/apps/codecs/mpc.c
index db25015..207a63c 100644
--- a/apps/codecs/mpc.c
+++ b/apps/codecs/mpc.c
@@ -68,6 +68,8 @@ MPC_SAMPLE_FORMAT sample_buffer[MPC_FRAME_LENGTH*2] IBSS_ATTR;
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* this is the codec entry point */
@@ -83,6 +85,7 @@ enum codec_status codec_start(struct codec_api *api)
     TEST_CODEC_API(api);
     #ifdef USE_IRAM 
     ci->memcpy(iramstart, iramcopy, iramend - iramstart);
+    ci->memset(iedata, 0, iend - iedata);
     #endif
     
     ci->configure(CODEC_DSP_ENABLE, (bool *)true);
diff --git a/apps/codecs/shorten.c b/apps/codecs/shorten.c
index c58b009..9e679f5 100644
--- a/apps/codecs/shorten.c
+++ b/apps/codecs/shorten.c
@@ -20,10 +20,12 @@
 #include "codeclib.h"
 #include <codecs/libffmpegFLAC/shndec.h>
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 struct codec_api* rb;
@@ -52,8 +54,9 @@ enum codec_status codec_start(struct codec_api* api)
     rb = api;
     ci = (struct codec_api*)api;
 
-#ifndef SIMULATOR
+#ifdef USE_IRAM
     ci->memcpy(iramstart, iramcopy, iramend-iramstart);
+    ci->memset(iedata, 0, iend - iedata);
 #endif
 
     ci->configure(CODEC_SET_FILEBUF_WATERMARK, (int *)(1024*512));
diff --git a/apps/codecs/vorbis.c b/apps/codecs/vorbis.c
index 69eba24..795e689 100644
--- a/apps/codecs/vorbis.c
+++ b/apps/codecs/vorbis.c
@@ -101,6 +101,8 @@ bool vorbis_set_codec_parameters(OggVorbis_File *vf)
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* this is the codec entry point */
@@ -125,6 +127,7 @@ enum codec_status codec_start(struct codec_api *api)
 
     #ifdef USE_IRAM
     rb->memcpy(iramstart, iramcopy, iramend - iramstart);
+    rb->memset(iedata, 0, iend - iedata);
     #endif
 
     rb->configure(CODEC_DSP_ENABLE, (bool *)true);
diff --git a/apps/codecs/wav.c b/apps/codecs/wav.c
index 85e2412..e86b559 100644
--- a/apps/codecs/wav.c
+++ b/apps/codecs/wav.c
@@ -94,6 +94,8 @@ enum
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* Those are lookup tables, so they should be in the idata section
@@ -235,6 +237,7 @@ enum codec_status codec_start(struct codec_api* api)
 
 #ifdef USE_IRAM 
   ci->memcpy(iramstart, iramcopy, iramend-iramstart);
+  ci->memset(iedata, 0, iend - iedata);
 #endif
 
   ci->configure(CODEC_SET_FILEBUF_WATERMARK, (int *)(1024*512));
diff --git a/apps/codecs/wavpack.c b/apps/codecs/wavpack.c
index 21c1947..ff890ba 100644
--- a/apps/codecs/wavpack.c
+++ b/apps/codecs/wavpack.c
@@ -40,6 +40,8 @@ static long read_callback (void *buffer, long bytes)
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* this is the codec entry point */
@@ -56,6 +58,7 @@ enum codec_status codec_start(struct codec_api* api)
 
 #ifdef USE_IRAM
     ci->memcpy(iramstart, iramcopy, iramend-iramstart);
+    ci->memset(iedata, 0, iend - iedata);
 #endif
 
     ci->configure(CODEC_SET_FILEBUF_WATERMARK, (int *)(1024*512));
diff --git a/apps/plugins/plugin.lds b/apps/plugins/plugin.lds
index 83825b9..fc9d4d4 100644
--- a/apps/plugins/plugin.lds
+++ b/apps/plugins/plugin.lds
@@ -60,7 +60,8 @@ MEMORY
 
 SECTIONS
 {
-    .text : {
+    .text : 
+    {
         KEEP(*(.entry))
         *(.text*)
 #if CONFIG_CPU==PP5020
@@ -69,26 +70,25 @@ SECTIONS
 #endif
     } > PLUGIN_RAM
 
-    .data : {
-        *(.data*)
-    } > PLUGIN_RAM
-
-    /DISCARD/ : {
-        *(.eh_frame)
-    }
-
-    .bss : {
-        *(.bss*)
-    } > PLUGIN_RAM
-
-    .rodata : {
+    .rodata : 
+    {
         *(.rodata*)
         . = ALIGN(0x4);
+    } > PLUGIN_RAM
+
+    .data : 
+    {
+        *(.data*)
 #if defined(ARCH_IRIVER) || defined(ARCH_IPOD)
         iramcopy = .;
 #endif
     } > PLUGIN_RAM
 
+    /DISCARD/ : 
+    {
+        *(.eh_frame)
+    }
+
 #if defined(ARCH_IRIVER) || defined(ARCH_IPOD)
     .iram IRAMORIG : AT ( iramcopy)
     {
@@ -96,13 +96,29 @@ SECTIONS
         *(.icode)
         *(.irodata)
         *(.idata)
-        *(.ibss)
         iramend = .;
     } > PLUGIN_IRAM
+
+    .ibss (NOLOAD) :
+    {
+        iedata = .;
+        *(.ibss)
+        . = ALIGN(0x4);
+        iend = .;
+    } > PLUGIN_IRAM
+
+    .bss ADDR(.data) + SIZEOF(.data) + SIZEOF(.iram):
+#else
+    .bss :
 #endif
+    {
+        *(.bss*)
+    } > PLUGIN_RAM
+
     /* Special trick to avoid a linker error when no other sections are
        left after garbage collection (plugin not for this platform) */
-    .comment 0 : {
+    .comment 0 : 
+    {
         KEEP(*(.comment))
     }
 }
diff --git a/apps/plugins/rockboy/cpu.c b/apps/plugins/rockboy/cpu.c
index 6118633..5c4abc5 100644
--- a/apps/plugins/rockboy/cpu.c
+++ b/apps/plugins/rockboy/cpu.c
@@ -17,7 +17,7 @@
 #endif
 
 
-struct cpu cpu IDATA_ATTR;
+struct cpu cpu IBSS_ATTR;
 
 
 
diff --git a/apps/plugins/rockboy/lcd.c b/apps/plugins/rockboy/lcd.c
index a8f8c69..883c495 100644
--- a/apps/plugins/rockboy/lcd.c
+++ b/apps/plugins/rockboy/lcd.c
@@ -15,7 +15,7 @@
 
 struct lcd lcd;
 
-struct scan scan IDATA_ATTR;
+struct scan scan IBSS_ATTR;
 
 #define BG (scan.bg)
 #define WND (scan.wnd)
diff --git a/apps/plugins/rockboy/rockboy.c b/apps/plugins/rockboy/rockboy.c
index 90452e5..5677387 100644
--- a/apps/plugins/rockboy/rockboy.c
+++ b/apps/plugins/rockboy/rockboy.c
@@ -42,6 +42,8 @@ const struct {
 extern char iramcopy[];
 extern char iramstart[];
 extern char iramend[];
+extern char iedata[];
+extern char iend[];
 #endif
 
 /* here is a global api struct pointer. while not strictly necessary,
@@ -116,6 +118,7 @@ enum plugin_status plugin_start(struct plugin_api* api, void* parameter)
 
 #ifdef USE_IRAM
     memcpy(iramstart, iramcopy, iramend-iramstart);
+    memset(iedata, 0, iend - iedata);
 #endif
     shut=0;
     cleanshut=0;