From fb4c6fba4c83b2f00a78c331fd56e5d995ce02ab Mon Sep 17 00:00:00 2001
From: notaz <notasas@gmail.com>
Date: Wed, 7 Dec 2011 00:51:52 +0200
Subject: [PATCH] gpu_neon: fix several frameskip issues

---
 plugins/gpu_neon/gpu.c     | 16 ++++++++++++++--
 plugins/gpu_neon/gpu.h     |  3 ++-
 plugins/gpu_neon/vout_fb.c |  7 +++----
 3 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/plugins/gpu_neon/gpu.c b/plugins/gpu_neon/gpu.c
index a5d0b876..978f4d1e 100644
--- a/plugins/gpu_neon/gpu.c
+++ b/plugins/gpu_neon/gpu.c
@@ -132,8 +132,10 @@ void GPUwriteStatus(uint32_t data)
     case 0x05:
       gpu.screen.x = data & 0x3ff;
       gpu.screen.y = (data >> 10) & 0x3ff;
-      if (gpu.frameskip.set)
+      if (gpu.frameskip.set && gpu.frameskip.last_flip_frame != *gpu.state.frame_count) {
         decide_frameskip();
+        gpu.frameskip.last_flip_frame = *gpu.state.frame_count;
+      }
       break;
     case 0x06:
       gpu.screen.x1 = data & 0xfff;
@@ -291,6 +293,16 @@ static int check_cmd(uint32_t *data, int count)
         gpu.ex_regs[1] &= ~0x1ff;
         gpu.ex_regs[1] |= list[5] & 0x1ff;
       }
+      else if (cmd == 0xe3)
+      {
+        // no frameskip if it decides to draw to display area,
+        // but not for interlace since it'll most likely always do that
+        uint32_t x = list[0] & 0x3ff;
+        uint32_t y = (list[0] >> 10) & 0x3ff;
+        gpu.frameskip.allow = gpu.status.interlace ||
+          (uint32_t)(x - gpu.screen.x) >= (uint32_t)gpu.screen.w ||
+          (uint32_t)(y - gpu.screen.y) >= (uint32_t)gpu.screen.h;
+      }
       if (2 <= cmd && cmd < 0xc0)
         vram_dirty = 1;
       else if ((cmd & 0xf8) == 0xe0)
@@ -306,7 +318,7 @@ static int check_cmd(uint32_t *data, int count)
     }
 
     if (pos - start > 0) {
-      if (!gpu.frameskip.active)
+      if (!gpu.frameskip.active || !gpu.frameskip.allow)
         do_cmd_list(data + start, pos - start);
       start = pos;
     }
diff --git a/plugins/gpu_neon/gpu.h b/plugins/gpu_neon/gpu.h
index 7269a526..20194649 100644
--- a/plugins/gpu_neon/gpu.h
+++ b/plugins/gpu_neon/gpu.h
@@ -76,9 +76,10 @@ struct psx_gpu {
   struct {
     int32_t set:3; /* -1 auto, 0 off, 1 fixed */
     uint32_t active:1;
+    uint32_t allow:1;
     uint32_t frame_ready:1;
-    uint32_t skipped_blits:5;
     const int *advice;
+    uint32_t last_flip_frame;
   } frameskip;
 };
 
diff --git a/plugins/gpu_neon/vout_fb.c b/plugins/gpu_neon/vout_fb.c
index 394f6a3b..65fbc038 100644
--- a/plugins/gpu_neon/vout_fb.c
+++ b/plugins/gpu_neon/vout_fb.c
@@ -89,12 +89,11 @@ void GPUupdateLace(void)
     return;
 
   if (gpu.frameskip.set) {
-    if (!gpu.frameskip.frame_ready && gpu.frameskip.skipped_blits < 9) {
-      gpu.frameskip.skipped_blits++;
-      return;
+    if (!gpu.frameskip.frame_ready) {
+      if (*gpu.state.frame_count - gpu.frameskip.last_flip_frame < 9)
+        return;
     }
     gpu.frameskip.frame_ready = 0;
-    gpu.frameskip.skipped_blits = 0;
   }
 
   renderer_flush_queues();
-- 
2.39.2