Fix warnings introduced in prior commit

* #define CONFIG_RTC APPLICATION  (in sim builds)
 * #define CONFIG_STORAGE 0        (in PCTOOL builds)

Change-Id: Iabb77bd0a29ca06a13d1075f1e8cedb00f785c64

firmware/drivers/button.c

@@ -33,11 +33,14 @@
 #include "serial.h"
 #include "power.h"
 #include "powermgmt.h"
-#if defined(HAVE_SDL) && (SDL_MAJOR_VERSION > 1)
+#if defined(HAVE_SDL)
+#include <SDL.h>
+#if (SDL_MAJOR_VERSION > 1)
 #include "button-sdl.h"
 #else
 #include "button-target.h"
 #endif
+#endif /* HAVE_SDL */
 #ifdef HAVE_REMOTE_LCD
 #include "lcd-remote.h"
 #endif

firmware/export/config.h

@@ -340,6 +340,7 @@ Lyre prototype 1 */
 #define NAND_IMX233  6
 
 /* CONFIG_RTC */
+#define RTC_HOSTED   1 /* Generic hosted */
 #define RTC_PCF50605 2 /* iPod 3G, 4G & Mini */
 #define RTC_PCF50606 3 /* iriver H300 */
 #define RTC_S3C2440  4
@@ -623,7 +624,7 @@ Lyre prototype 1 */
 #elif defined(RG_NANO)
 #include "config/rgnano.h"
 #else
-//#error "unknown hwardware platform!"
+#error "unknown hardware platform!"
 #endif
 
 #ifndef CONFIG_CPU
@@ -642,6 +643,7 @@ Lyre prototype 1 */
 #undef HAVE_MULTIDRIVE
 #undef CONFIG_STORAGE_MULTI
 #undef CONFIG_STORAGE
+#define CONFIG_STORAGE 0
 #endif
 
 #ifndef CONFIG_BUFLIB_BACKEND
@@ -649,8 +651,10 @@ Lyre prototype 1 */
 #endif
 
 #ifdef APPLICATION
+#ifndef CONFIG_CPU
 #define CONFIG_CPU 0
 #endif
+#endif
 
 /* keep this include after the target configs */
 #ifdef SIMULATOR

firmware/export/config/android.h

@@ -52,7 +52,7 @@
 #define HAVE_BUTTON_DATA
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/fiiom3klinux.h

@@ -66,7 +66,7 @@
 #define CONFIG_BACKLIGHT_FADING BACKLIGHT_FADING_SW_SETTING
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/hibylinux.h

@@ -60,7 +60,7 @@
 #define CONFIG_BACKLIGHT_FADING BACKLIGHT_FADING_SW_SETTING
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 #ifndef SIMULATOR
 /* We have usb power and can detect usb but it is handled by Linux */

firmware/export/config/nokian8xx.h

@@ -44,7 +44,7 @@
 #define HAVE_BUTTON_DATA
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/nokian900.h

@@ -43,7 +43,7 @@
 #define HAVE_BUTTON_DATA
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/pandora.h

@@ -44,7 +44,7 @@
 #define HAVE_BUTTON_DATA
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/rgnano.h

@@ -48,7 +48,7 @@
 #define HAVE_VOLUME_IN_LIST
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x200000

firmware/export/config/samsungypr1.h

@@ -112,7 +112,7 @@
 /* R1 has a standard linux RTC driver on /dev/rtc1 (->/dev/rtc)
  * The RTC is S35392 A
  */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 #define BATTERY_CAPACITY_DEFAULT 600 /* default battery capacity */
 #define BATTERY_CAPACITY_MIN 600  /* min. capacity selectable */

firmware/export/config/sdlapp.h

@@ -53,7 +53,7 @@
 #define HAVE_BUTTON_DATA
 
 /* define this if you have a real-time clock */
-#define CONFIG_RTC APPLICATION
+#define CONFIG_RTC RTC_HOSTED
 
 /* The number of bytes reserved for loadable codecs */
 #define CODEC_SIZE 0x100000

firmware/export/config/sim.h

@@ -2,6 +2,7 @@
    target specific configs */
 
 #undef CONFIG_CPU
+#define CONFIG_CPU 0
 
 #undef HAVE_FMADC
 
@@ -17,6 +18,8 @@
 #undef HAVE_ATA_POWER_OFF
 
 #undef CONFIG_LCD
+#define CONFIG_LCD 0
+
 #undef LCD_DPI /* likely to be too different on a PC */
 
 #undef CONFIG_LED
@@ -86,7 +89,8 @@
 
 #undef BUTTON_DRIVER_CLOSE
 
-#if     CONFIG_BACKLIGHT_FADING == BACKLIGHT_FADING_SW_HW_REG
+#if defined(CONFIG_BACKLIGHT_FADING)
+#if     (CONFIG_BACKLIGHT_FADING == BACKLIGHT_FADING_SW_HW_REG)
 #undef  CONFIG_BACKLIGHT_FADING
         /* simulate SW_SETTING, as we handle sdl very similary */
 #define CONFIG_BACKLIGHT_FADING BACKLIGHT_FADING_SW_SETTING
@@ -95,7 +99,8 @@
         CONFIG_BACKLIGHT_FADING == BACKLIGHT_FADING_TARGET)
         /* we don't simulate that yet */
 #undef  CONFIG_BACKLIGHT_FADING
-#endif
+#endif /* CONFIG_BACKLIGHT_FADING == BACKLIGHT_FADING_SW_HW_REG */
+#endif /* defined(CONFIG_BACKLIGHT_FADING) */
 
 #ifdef  HAVE_BACKLIGHT_BRIGHTNESS
 #undef  DEFAULT_BRIGHTNESS_SETTING

tools/configure

@@ -262,7 +262,7 @@ simcc () {
  app_type=$1
  winbuild=""
  macbuild=""
- GCCOPTS=`echo $CCOPTS | sed -e s/\ -ffreestanding// -e s/\ -nostdlib// -e s/\ -Wundef//`
+ GCCOPTS=`echo $CCOPTS | sed -e s/\ -ffreestanding// -e s/\ -nostdlib//`
 
  if [ "yes" = "$use_debug" ]; then
     GCCOPTS=`echo $GCCOPTS | sed -e s/\ -Os/\ -Og/`
@@ -638,7 +638,7 @@ maemocc () {
  # Scratchbox sets up "gcc" based on the active target
  prefixtools ""
 
- GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib// -e s/-Wundef//`
+ GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib//`
  GCCOPTS="$GCCOPTS -fno-builtin -g -I\$(SIMDIR)"
  GCCOPTIMIZE=''
  LDOPTS="-lm -ldl $LDOPTS"
@@ -733,7 +733,7 @@ pandoracc () {
  LDOPTS="-L$PNDSDK/arm-angstrom-linux-gnueabi/usr/lib -Wl,-rpath,$PNDSDK/arm-angstrom-linux-gnueabi/usr/lib $LDOPTS"
  PKG_CONFIG="pkg-config"
 
- GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib// -e s/-Wundef//`
+ GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib//`
  GCCOPTS="$GCCOPTS -fno-builtin -g -I\$(SIMDIR)"
  GCCOPTIMIZE=''
  LDOPTS="-lm -ldl $LDOPTS"
@@ -813,7 +813,7 @@ androidcc () {
 
     # the prebuilt android NDK only supports x86_64 architecture anyway, so we can take shortcuts
     buildhost=$(uname | tr "[:upper:]" "[:lower:]")-x86_64
-    GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib// -e s/-Wundef//`
+    GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib//`
     LDOPTS="$LDOPTS -ldl -llog"
     if [ "$modelname" != "ibassodx50" ] && [ "$modelname" != "ibassodx90" ]; then
         LDOPTS="$LDOPTS -Wl,-soname,librockbox.so -shared"
@@ -903,7 +903,7 @@ androidndkcc()
     # the prebuilt android NDK only supports x86_64 architecture anyway, so we can take shortcuts
     buildhost=$(uname -s | tr "[:upper:]" "[:lower:]")-x86_64
 
-    GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib// -e s/-Wundef//`
+    GCCOPTS=`echo $CCOPTS | sed -e s/-ffreestanding// -e s/-nostdlib//`
     LDOPTS="$LDOPTS -ldl -llog"
     SHARED_LDFLAGS="-shared"
     SHARED_CFLAGS=''
@@ -988,7 +988,7 @@ rgnanocc () {
     DLLWRAP=$FUNKEY_SDK_PATH/bin/arm-funkey-linux-musleabihf-dllwrap
     RANLIB=$FUNKEY_SDK_PATH/bin/arm-funkey-linux-musleabihf-gcc-ranlib
 
-    GCCOPTS=`echo $CCOPTS | sed -e s/\ -ffreestanding// -e s/\ -nostdlib// -e s/\ -Wundef//`
+    GCCOPTS=`echo $CCOPTS | sed -e s/\ -ffreestanding// -e s/\ -nostdlib//`
 
     if [ "yes" = "$use_debug" ]; then
         GCCOPTS=`echo $GCCOPTS | sed -e s/\ -Os/\ -Og/`