Do not include kernel.h in system.h.

system.h doesn't need it on its own and this change makes it less
dependant on Rockbox internals.

Change-Id: I4e1e4108a52a7b599627a829204eb82b392fc6d6

apps/codec_thread.c

@@ -20,6 +20,7 @@
  * KIND, either express or implied.
  *
  ****************************************************************************/
+
 #include "config.h"
 #include "system.h"
 #include "kernel.h"

apps/rbcodecconfig.h

@@ -18,6 +18,9 @@
  * {,U}INT{8,16,32,64}_{MIN,MAX} */
 #include "system.h"
 
+/* HZ, TIME_AFTER */
+#include "kernel.h"
+
 /* Structure to record some info during processing call */
 struct dsp_loop_context
 {

apps/recorder/bmp.c

@@ -38,6 +38,7 @@
    ahead by whole lines, or read the next chunk of the current line
 */
 
+#define ROCKBOX_DEBUG_BMP_LOADER
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -53,7 +54,7 @@
 #include "lcd-remote.h"
 #endif
 #ifdef ROCKBOX_DEBUG_BMP_LOADER
-#define BDEBUGF DEBUGF
+#define BDEBUGF printf
 #else
 #define BDEBUGF(...)
 #endif
@@ -774,6 +775,7 @@ int read_bmp_fd(int fd,
 #endif
         .alpha_detected = false, .first_alpha_byte = 0x80,
         .order = order,
+        .buf = NULL,
     };
 
 #if (LCD_DEPTH > 1 || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)) && \

apps/recorder/resize.c

@@ -31,12 +31,14 @@
  *
  ****************************************************************************/
 
-#include <system.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <general.h>
+#include <stdint.h>
-#include "inttypes.h"
+
+#include "general.h"
+#include "kernel.h"
+#include "system.h"
 #ifndef PLUGIN
 #include "debug.h"
 #endif

apps/voice_thread.c

@@ -18,8 +18,10 @@
  * KIND, either express or implied.
  *
  ****************************************************************************/
+
 #include "config.h"
 #include "system.h"
+#include "kernel.h"
 #include "core_alloc.h"
 #include "thread.h"
 #include "appevents.h"

firmware/buflib.c

@@ -27,6 +27,7 @@
 
 #include <stdlib.h> /* for abs() */
 #include <stdio.h> /* for snprintf() */
+#include <stddef.h> /* for ptrdiff_t */
 #include "buflib.h"
 #include "string-extra.h" /* strlcpy() */
 #include "debug.h"

firmware/drivers/audio/as3514.c

@@ -21,9 +21,11 @@
  * KIND, either express or implied.
  *
  ****************************************************************************/
+
 #include "cpu.h"
 #include "debug.h"
 #include "system.h"
+#include "kernel.h"
 #include "audio.h"
 #include "sound.h"
 

firmware/export/system.h

@@ -22,9 +22,10 @@
 #ifndef __SYSTEM_H__
 #define __SYSTEM_H__
 
+#include <stdbool.h>
+#include <stdint.h>
+
 #include "cpu.h"
-#include "stdbool.h"
-#include "kernel.h"
 #include "gcc_extensions.h" /* for LIKELY/UNLIKELY */
 
 extern void system_reboot (void);

firmware/font.c

@@ -24,14 +24,16 @@
  * loaded at startup, as well as their mapping into
  * the FONT_SYSFIXED, FONT_UI and FONT_MP3 ids.
  */
-#include "config.h"
-
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
-#include "inttypes.h"
+#include <stdint.h>
-#include "lcd.h"
+#include <stddef.h>
+
+#include "config.h"
 #include "system.h"
+#include "kernel.h"
+#include "lcd.h"
 #include "font.h"
 #include "file.h"
 #include "core_alloc.h"

firmware/general.c

@@ -19,8 +19,10 @@
  *
  ****************************************************************************/
 
-#include "config.h"
 #include <stdio.h>
+#include "config.h"
+#include "system.h"
+#include "kernel.h"
 #include "general.h"
 #include "file.h"
 #include "dir.h"
@@ -28,7 +30,6 @@
 #include "limits.h"
 #include "stdlib.h"
 #include "string-extra.h"
-#include "system.h"
 #include "time.h"
 #include "timefuncs.h"
 

firmware/target/arm/as3525/debug-as3525.c

@@ -20,10 +20,11 @@
  ****************************************************************************/
 
 #include <stdbool.h>
+#include "system.h"
+#include "kernel.h"
 #include "button.h"
 #include "lcd.h"
 #include "font.h"
-#include "system.h"
 #include "cpu.h"
 #include "pl180.h"
 #include "ascodec.h"

firmware/target/arm/as3525/lcd-fuze.c

@@ -19,11 +19,12 @@
  * KIND, either express or implied.
  *
  ****************************************************************************/
-#include "config.h"
 
+#include "config.h"
 #include "cpu.h"
-#include "lcd.h"
 #include "system.h"
+#include "kernel.h"
+#include "lcd.h"
 #include "file.h"
 #include "clock-target.h"
 #include "dbop-as3525.h"

firmware/target/arm/as3525/sansa-fuzev2/button-fuzev2.c

@@ -21,6 +21,7 @@
 
 #include "config.h"
 #include "system.h"
+#include "kernel.h"
 #include "button.h"
 #include "backlight.h"
 

firmware/target/arm/pp/system-target.h

@@ -22,9 +22,10 @@
 #ifndef SYSTEM_TARGET_H
 #define SYSTEM_TARGET_H
 
+#include <stdbool.h>
 #include "config.h"
 #include "system-arm.h"
-#include <stdbool.h>
+#include "kernel.h"
 
 /* TODO: This header could be split in 2 */
 

firmware/target/hosted/android/pcm-android.c

@@ -22,8 +22,9 @@
 #include <jni.h>
 #include <stdbool.h>
 #define _SYSTEM_WITH_JNI /* for getJavaEnvironment */
-#include <system.h>
 #include <pthread.h>
+#include "system.h"
+#include "kernel.h"
 #include "debug.h"
 #include "pcm.h"
 #include "pcm-internal.h"

firmware/target/hosted/kernel-unix.c

@@ -26,8 +26,10 @@
 #include <errno.h>
 #include <unistd.h>
 #include <pthread.h>
+
 #include "config.h"
 #include "system.h"
+#include "kernel.h"
 #include "button.h"
 #include "audio.h"
 #include "panic.h"
@@ -165,4 +167,3 @@ void timer_unregister(void)
     timer_delete(timer_tid);
     timer_prio = -1;
 }
-