Remove get_viewport_default_colour()

The function isn't used except for the skin engine's %Vf/%Vb tags,
so inline it there and remove the separate function.

Change-Id: Ia207321877234bc3679457c820a4292ae53f3520

apps/gui/skin_engine/skin_parser.c

@@ -712,8 +712,35 @@ static int parse_viewportcolour(struct skin_element *element,
         return -1;
     if (isdefault(param))
     {
-        colour->colour = get_viewport_default_colour(curr_screen,
+        unsigned int fg_color;
-                                   token->type == SKIN_TOKEN_VIEWPORT_FGCOLOUR);
+        unsigned int bg_color;
+
+        switch (curr_screen)
+        {
+#if defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1
+        case SCREEN_REMOTE:
+            fg_color = LCD_REMOTE_DEFAULT_FG;
+            bg_color = LCD_REMOTE_DEFAULT_BG;
+            break;
+#endif
+        default:
+#if defined(HAVE_LCD_COLOR)
+            fg_color = global_settings.fg_color;
+            bg_color = global_settings.bg_color;
+#elif LCD_DEPTH > 1
+            fg_color = LCD_DEFAULT_FG;
+            bg_color = LCD_DEFAULT_BG;
+#else
+            fg_color = 0;
+            bg_color = 0;
+#endif
+            break;
+        }
+
+        if (token->type == SKIN_TOKEN_VIEWPORT_FGCOLOUR)
+            colour->colour = fg_color;
+        else
+            colour->colour = bg_color;
     }
     else
     {

apps/gui/viewport.c

@@ -348,41 +348,3 @@ void viewport_set_defaults(struct viewport *vp,
 #endif /* !__PCTOOL__ */
         viewport_set_fullscreen(vp, screen);
 }
-
-
-int get_viewport_default_colour(enum screen_type screen, bool fgcolour)
-{
-    (void)screen; (void)fgcolour;
-#if (LCD_DEPTH > 1) || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)
-    int colour;
-    if (fgcolour)
-    {
-#if (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)
-        if (screen == SCREEN_REMOTE)
-            colour = REMOTE_FG_FALLBACK;
-        else
-#endif
-#if defined(HAVE_LCD_COLOR)
-            colour = global_settings.fg_color;
-#else
-            colour = FG_FALLBACK;
-#endif
-    }
-    else
-    {
-#if (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)
-        if (screen == SCREEN_REMOTE)
-            colour = REMOTE_BG_FALLBACK;
-        else
-#endif
-#if defined(HAVE_LCD_COLOR)
-            colour = global_settings.bg_color;
-#else
-            colour = BG_FALLBACK;
-#endif
-    }
-    return colour;
-#else
-    return 0;
-#endif /* LCD_DEPTH > 1 || LCD_REMOTE_DEPTH > 1 */
-}

apps/gui/viewport.h

@@ -43,7 +43,6 @@ void viewport_set_defaults(struct viewport *vp,
                             const enum screen_type screen);
 void viewport_set_fullscreen(struct viewport *vp,
                               const enum screen_type screen);
-int get_viewport_default_colour(enum screen_type screen, bool fgcolour);
 
 #ifndef __PCTOOL__