From 6469a8c407ff23af8b7a6218f759b368eec7c339 Mon Sep 17 00:00:00 2001
From: notaz <notasas@gmail.com>
Date: Sun, 22 Apr 2012 01:56:22 +0300
Subject: [PATCH] frontend: clean up some hacks
---
Makefile | 1 -
frontend/menu.c | 9 ++++----
frontend/omap.h | 6 -----
frontend/plat.h | 6 +++++
frontend/plat_dummy.c | 26 ++++++++++++++-------
frontend/plat_omap.c | 51 ++++++++++++++++++++++++++++++++++--------
frontend/plat_pollux.c | 38 +++++++++++++++----------------
frontend/plugin_lib.c | 50 +++++++++++++----------------------------
frontend/plugin_lib.h | 5 ++++-
maemo/hildon.c | 20 +++++++++--------
10 files changed, 120 insertions(+), 92 deletions(-)
delete mode 100644 frontend/omap.h
diff --git a/Makefile b/Makefile
index cdc1a740..141929a4 100644
--- a/Makefile
+++ b/Makefile
@@ -123,7 +123,6 @@ ifeq "$(PLATFORM)" "generic"
OBJS += frontend/plat_dummy.o
endif
ifeq "$(PLATFORM)" "pandora"
-frontend/%.o: CFLAGS += -DVOUT_FBDEV
OBJS += frontend/linux/fbdev.o
OBJS += frontend/plat_omap.o
OBJS += frontend/plat_pandora.o
diff --git a/frontend/menu.c b/frontend/menu.c
index 27ba8257..989c6ec9 100644
--- a/frontend/menu.c
+++ b/frontend/menu.c
@@ -22,7 +22,6 @@
#include "config.h"
#include "plugin.h"
#include "plugin_lib.h"
-#include "omap.h"
#include "plat.h"
#include "pcnt.h"
#include "common/plat.h"
@@ -1147,7 +1146,7 @@ static int menu_loop_cscaler(int id, int keys)
scaling = SCALE_CUSTOM;
- omap_enable_layer(1);
+ plat_gvideo_open();
for (;;)
{
@@ -1182,11 +1181,12 @@ static int menu_loop_cscaler(int id, int keys)
g_layer_w = 800 - g_layer_x;
if (g_layer_y + g_layer_h > 480)
g_layer_h = 480 - g_layer_y;
- omap_enable_layer(1);
+ // resize the layer
+ plat_gvideo_open();
}
}
- omap_enable_layer(0);
+ plat_gvideo_close();
return 0;
}
@@ -2258,6 +2258,7 @@ void menu_init(void)
#endif
}
+// XXX: should really menu code cotrol the layer size?
void menu_notify_mode_change(int w, int h, int bpp)
{
float mult;
diff --git a/frontend/omap.h b/frontend/omap.h
deleted file mode 100644
index 5cd78eda..00000000
--- a/frontend/omap.h
+++ /dev/null
@@ -1,6 +0,0 @@
-
-extern struct vout_fbdev *layer_fb;
-extern int g_layer_x, g_layer_y;
-extern int g_layer_w, g_layer_h;
-
-int omap_enable_layer(int enabled);
diff --git a/frontend/plat.h b/frontend/plat.h
index 6a22035f..9076cd5c 100644
--- a/frontend/plat.h
+++ b/frontend/plat.h
@@ -5,5 +5,11 @@ int plat_cpu_clock_get(void);
int plat_cpu_clock_apply(int cpu_clock);
int plat_get_bat_capacity(void);
+// indirectly called from GPU plugin
+void plat_gvideo_open(void);
+void *plat_gvideo_set_mode(int *w, int *h, int *bpp);
+void *plat_gvideo_flip(void);
+void plat_gvideo_close(void);
+
// XXX
int plat_pandora_init(void);
diff --git a/frontend/plat_dummy.c b/frontend/plat_dummy.c
index 6249df5f..9a325498 100644
--- a/frontend/plat_dummy.c
+++ b/frontend/plat_dummy.c
@@ -6,20 +6,12 @@
*/
#include "common/input.h"
-#include "linux/fbdev.h"
#include "plat.h"
-struct vout_fbdev *layer_fb;
-int g_layer_x, g_layer_y, g_layer_w, g_layer_h;
struct in_default_bind in_evdev_defbinds[] = {
{ 0, 0, 0 },
};
-int omap_enable_layer(int enabled)
-{
- return 0;
-}
-
void plat_video_menu_enter(int is_rom_loaded)
{
}
@@ -36,6 +28,24 @@ void plat_video_menu_leave(void)
{
}
+void plat_gvideo_open(void)
+{
+}
+
+void *plat_gvideo_set_mode(int *w, int *h, int *bpp)
+{
+ return 0;
+}
+
+void *plat_gvideo_flip(void)
+{
+ return 0;
+}
+
+void plat_gvideo_close(void)
+{
+}
+
void plat_init(void)
{
}
diff --git a/frontend/plat_omap.c b/frontend/plat_omap.c
index 4ab7cf04..96f75cec 100644
--- a/frontend/plat_omap.c
+++ b/frontend/plat_omap.c
@@ -1,5 +1,5 @@
/*
- * (C) notaz, 2010
+ * (C) Gražvydas "notaz" Ignotas, 2010-2012
*
* This work is licensed under the terms of the GNU GPLv2 or later.
* See the COPYING file in the top-level directory.
@@ -19,15 +19,10 @@
#include "linux/xenv.h"
#include "plugin_lib.h"
#include "pl_gun_ts.h"
-#include "omap.h"
#include "plat.h"
+#include "menu.h"
-
-static struct vout_fbdev *main_fb;
-int g_layer_x = 80, g_layer_y = 0;
-int g_layer_w = 640, g_layer_h = 480;
-
-struct vout_fbdev *layer_fb;
+static struct vout_fbdev *main_fb, *layer_fb;
static int omap_setup_layer_(int fd, int enabled, int x, int y, int w, int h)
{
@@ -79,7 +74,7 @@ static int omap_setup_layer_(int fd, int enabled, int x, int y, int w, int h)
return 0;
}
-int omap_enable_layer(int enabled)
+static int omap_enable_layer(int enabled)
{
if (enabled)
pl_set_gun_rect(g_layer_x, g_layer_y, g_layer_w, g_layer_h);
@@ -88,6 +83,36 @@ int omap_enable_layer(int enabled)
g_layer_x, g_layer_y, g_layer_w, g_layer_h);
}
+void plat_gvideo_open(void)
+{
+ omap_enable_layer(1);
+
+ // try to align redraws to vsync
+ vout_fbdev_wait_vsync(layer_fb);
+}
+
+void *plat_gvideo_set_mode(int *w, int *h, int *bpp)
+{
+ void *buf;
+
+ vout_fbdev_clear(layer_fb);
+ buf = vout_fbdev_resize(layer_fb, *w, *h, *bpp, 0, 0, 0, 0, 3);
+
+ omap_enable_layer(1);
+
+ return buf;
+}
+
+void *plat_gvideo_flip(void)
+{
+ return vout_fbdev_flip(layer_fb);
+}
+
+void plat_gvideo_close(void)
+{
+ omap_enable_layer(0);
+}
+
void plat_video_menu_enter(int is_rom_loaded)
{
g_menuscreen_ptr = vout_fbdev_resize(main_fb,
@@ -133,6 +158,11 @@ void plat_trigger_vibrate(int is_strong)
{
}
+void *plat_prepare_screenshot(int *w, int *h, int *bpp)
+{
+ return NULL;
+}
+
void plat_init(void)
{
const char *main_fb_name, *layer_fb_name;
@@ -155,6 +185,9 @@ void plat_init(void)
exit(1);
}
+ g_layer_x = 80, g_layer_y = 0;
+ g_layer_w = 640, g_layer_h = 480;
+
ret = omap_setup_layer_(fd, 0, g_layer_x, g_layer_y, g_layer_w, g_layer_h);
close(fd);
if (ret != 0) {
diff --git a/frontend/plat_pollux.c b/frontend/plat_pollux.c
index c03173ea..012d9eb5 100644
--- a/frontend/plat_pollux.c
+++ b/frontend/plat_pollux.c
@@ -49,18 +49,7 @@ static int psx_step, psx_width, psx_height, psx_bpp;
static int psx_offset_x, psx_offset_y, psx_src_width, psx_src_height;
static int fb_offset_x, fb_offset_y;
-// TODO: get rid of this
-struct vout_fbdev;
-struct vout_fbdev *layer_fb;
-int g_layer_x, g_layer_y, g_layer_w, g_layer_h;
-
-int omap_enable_layer(int enabled)
-{
- return 0;
-}
-
static void caanoo_init(void);
-static void *pl_vout_set_mode(int w, int h, int bpp);
static void *fb_flip(void)
@@ -249,7 +238,6 @@ void plat_video_menu_leave(void)
memset(g_menuscreen_ptr, 0, 320*240 * psx_bpp/8);
g_menuscreen_ptr = fb_flip();
- pl_vout_set_mode(psx_width, psx_height, psx_bpp);
}
void *plat_prepare_screenshot(int *w, int *h, int *bpp)
@@ -335,7 +323,7 @@ static void raw_flip_dma(int x, int y)
if (psx_bpp == 16) {
pl_vout_buf = g_menuscreen_ptr;
- pl_print_hud(320, fb_offset_y + psx_src_height, fb_offset_x);
+ pl_print_hud(fb_offset_x);
}
g_menuscreen_ptr = fb_flip();
@@ -364,7 +352,7 @@ static void name(int x, int y)
\
if (psx_bpp == 16) { \
pl_vout_buf = g_menuscreen_ptr; \
- pl_print_hud(320, fb_offset_y + psx_src_height, fb_offset_x); \
+ pl_print_hud(fb_offset_x); \
} \
\
g_menuscreen_ptr = fb_flip(); \
@@ -378,9 +366,10 @@ make_flip_func(raw_flip_soft_368, blit320_368)
make_flip_func(raw_flip_soft_512, blit320_512)
make_flip_func(raw_flip_soft_640, blit320_640)
-static void *pl_vout_set_mode(int w, int h, int bpp)
+void *plat_gvideo_set_mode(int *w_, int *h_, int *bpp_)
{
int poff_w, poff_h, w_max;
+ int w = *w_, h = *h_, bpp = *bpp_;
if (!w || !h || !bpp)
return NULL;
@@ -446,14 +435,27 @@ static void *pl_vout_set_mode(int w, int h, int bpp)
pl_set_gun_rect(fb_offset_x, fb_offset_y, w > 320 ? 320 : w, h);
+ // adjust for hud
+ *w_ = 320;
+ *h_ = fb_offset_y + psx_src_height;
+
return NULL;
}
-static void *pl_vout_flip(void)
+/* not really used, we do raw_flip */
+void plat_gvideo_open(void)
+{
+}
+
+void *plat_gvideo_flip(void)
{
return NULL;
}
+void plat_gvideo_close(void)
+{
+}
+
static void save_multiple_regs(unsigned int *dest, int base, int count)
{
const volatile unsigned int *regs = memregl + base / 4;
@@ -603,9 +605,7 @@ void plat_init(void)
if (mixerdev == -1)
perror("open(/dev/mixer)");
- pl_rearmed_cbs.pl_vout_flip = pl_vout_flip;
pl_rearmed_cbs.pl_vout_raw_flip = have_warm ? raw_flip_dma : raw_flip_soft;
- pl_rearmed_cbs.pl_vout_set_mode = pl_vout_set_mode;
pl_rearmed_cbs.pl_vout_set_raw_vram = pl_vout_set_raw_vram;
psx_src_width = 320;
@@ -699,7 +699,7 @@ static int haptic_read(const char *fname, struct haptic_data *data)
f = fopen(fname, "r");
if (f == NULL) {
- fprintf("fopen(%s)", fname);
+ fprintf(stderr, "fopen(%s)", fname);
perror("");
return -1;
}
diff --git a/frontend/plugin_lib.c b/frontend/plugin_lib.c
index e79e5232..dbfdb36c 100644
--- a/frontend/plugin_lib.c
+++ b/frontend/plugin_lib.c
@@ -21,7 +21,6 @@
#include "linux/fbdev.h"
#include "common/fonts.h"
#include "common/input.h"
-#include "omap.h"
#include "menu.h"
#include "main.h"
#include "plat.h"
@@ -37,6 +36,7 @@ int in_keystate, in_state_gun;
int in_enable_vibration;
void *tsdev;
void *pl_vout_buf;
+int g_layer_x, g_layer_y, g_layer_w, g_layer_h;
static int pl_vout_w, pl_vout_h, pl_vout_bpp;
static int vsync_cnt;
static int frame_interval, frame_interval1024, vsync_usec_time;
@@ -111,10 +111,9 @@ static __attribute__((noinline)) void draw_active_chans(int vout_w, int vout_h)
}
}
-void pl_print_hud(int w, int h, int xborder)
+void pl_print_hud(int xborder)
{
- pl_vout_w = w; // used by pollux
- pl_vout_h = h;
+ int w = pl_vout_w, h = pl_vout_h;
if (g_opts & OPT_SHOWSPU)
draw_active_chans(w, h);
@@ -143,58 +142,39 @@ static void *pl_vout_set_mode(int w, int h, int bpp)
pl_vout_h = h;
pl_vout_bpp = bpp;
-#if defined(VOUT_FBDEV)
- vout_fbdev_clear(layer_fb);
- pl_vout_buf = vout_fbdev_resize(layer_fb, w, h, bpp, 0, 0, 0, 0, 3);
-#elif defined(MAEMO)
- extern void *hildon_set_mode(int w, int h);
- pl_vout_buf = hildon_set_mode(w, h);
-#endif
-
+ pl_vout_buf = plat_gvideo_set_mode(&pl_vout_w, &pl_vout_h, &pl_vout_bpp);
if (pl_vout_buf == NULL)
fprintf(stderr, "failed to set mode\n");
- // menu decides on layer size, we commit it
menu_notify_mode_change(w, h, bpp);
- omap_enable_layer(1);
return pl_vout_buf;
}
+// only used if raw flip is not defined
static void *pl_vout_flip(void)
{
pl_rearmed_cbs.flip_cnt++;
if (pl_vout_buf != NULL)
- pl_print_hud(pl_vout_w, pl_vout_h, 0);
+ pl_print_hud(0);
// let's flip now
-#if defined(VOUT_FBDEV)
- pl_vout_buf = vout_fbdev_flip(layer_fb);
-#elif defined(MAEMO)
- extern void *hildon_flip(void);
- pl_vout_buf = hildon_flip();
-#endif
+ pl_vout_buf = plat_gvideo_flip();
return pl_vout_buf;
}
static int pl_vout_open(void)
{
struct timeval now;
+ int h;
- omap_enable_layer(1);
-#if defined(VOUT_FBDEV)
// force mode update
- int h = pl_vout_h;
+ h = pl_vout_h;
pl_vout_h--;
pl_vout_buf = pl_vout_set_mode(pl_vout_w, h, pl_vout_bpp);
- // try to align redraws to vsync
- vout_fbdev_wait_vsync(layer_fb);
-#elif defined(MAEMO)
- extern void *hildon_flip(void);
- pl_vout_buf = hildon_flip();
-#endif
+ plat_gvideo_open();
gettimeofday(&now, 0);
vsync_usec_time = now.tv_usec;
@@ -206,20 +186,20 @@ static int pl_vout_open(void)
static void pl_vout_close(void)
{
- omap_enable_layer(0);
+ plat_gvideo_close();
}
void *pl_prepare_screenshot(int *w, int *h, int *bpp)
{
-#if defined(VOUT_FBDEV)
+ void *ret = plat_prepare_screenshot(w, h, bpp);
+ if (ret != NULL)
+ return ret;
+
*w = pl_vout_w;
*h = pl_vout_h;
*bpp = pl_vout_bpp;
return pl_vout_buf;
-#else
- return plat_prepare_screenshot(w, h, bpp);
-#endif
}
#ifndef MAEMO
diff --git a/frontend/plugin_lib.h b/frontend/plugin_lib.h
index e2f1d55f..8645385d 100644
--- a/frontend/plugin_lib.h
+++ b/frontend/plugin_lib.h
@@ -24,11 +24,14 @@ extern int in_enable_vibration;
extern void *pl_vout_buf;
+extern int g_layer_x, g_layer_y;
+extern int g_layer_w, g_layer_h;
+
void pl_text_out16(int x, int y, const char *texto, ...);
void pl_start_watchdog(void);
void *pl_prepare_screenshot(int *w, int *h, int *bpp);
void pl_init(void);
-void pl_print_hud(int w, int h, int xborder);
+void pl_print_hud(int xborder);
void pl_timing_prepare(int is_pal);
void pl_frame_limit(void);
diff --git a/maemo/hildon.c b/maemo/hildon.c
index 08949ffa..8948c3bb 100644
--- a/maemo/hildon.c
+++ b/maemo/hildon.c
@@ -7,6 +7,7 @@
#include "plugin_lib.h"
#include "main.h"
+#include "plat.h"
#include "../libpcsxcore/psemu_plugin_defs.h"
#include "common/readpng.h"
#include "maemo_common.h"
@@ -16,10 +17,6 @@
#define D_WIDTH 640
#define D_HEIGHT 480
-int g_layer_x = (X_RES - D_WIDTH) / 2;
-int g_layer_y = (Y_RES - D_HEIGHT) / 2;
-int g_layer_w = D_WIDTH, g_layer_h = D_HEIGHT;
-
static GdkImage *image;
static HildonAnimationActor *actor;
static GtkWidget *window, *drawing;
@@ -181,14 +178,20 @@ void maemo_init(int *argc, char ***argv)
gtk_widget_show_all (GTK_WIDGET (actor));
gtk_widget_show_all (GTK_WIDGET (window));
+
+ g_layer_x = (X_RES - D_WIDTH) / 2;
+ g_layer_y = (Y_RES - D_HEIGHT) / 2;
+ g_layer_w = D_WIDTH, g_layer_h = D_HEIGHT;
}
void menu_loop(void)
{
}
-void *hildon_set_mode(int w, int h)
+void *plat_gvideo_set_mode(int *w_, int *h_, int *bpp_)
{
+ int w = *w_, h = *h_;
+
if (w <= 0 || h <= 0)
return pl_vout_buf;
@@ -214,7 +217,7 @@ void *hildon_set_mode(int w, int h)
return pl_vout_buf;
}
-void *hildon_flip(void)
+void *plat_gvideo_flip(void)
{
gtk_widget_queue_draw (drawing);
@@ -244,12 +247,11 @@ void *hildon_flip(void)
return pl_vout_buf;
}
-int omap_enable_layer(int enabled)
+void plat_gvideo_open(void)
{
- return 0;
}
-void menu_notify_mode_change(int w, int h, int bpp)
+void plat_gvideo_close(void)
{
}
--
2.39.5