X-Git-Url: https://notaz.gp2x.de/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=pico%2Fvideoport.c;h=3f26d581ecb13b56918b35e377d10edbea9836a4;hb=0c7d1ba332b26f4ac67199e8ecbb826651f8512a;hp=6d62029c112fad288d067a2de1a58435467f4376;hpb=ef090115c8ea74938ed44e36a42e4a53d4447a9e;p=picodrive.git

diff --git a/pico/videoport.c b/pico/videoport.c
index 6d62029..3f26d58 100644
--- a/pico/videoport.c
+++ b/pico/videoport.c
@@ -1,15 +1,17 @@
-// PicoDrive
-
-// (c) Copyright 2004 Dave, All rights reserved.
-// (c) Copyright 2006-2008, Grazvydas "notaz" Ignotas
-// Free for non-commercial use.
-
-// For commercial use, separate licencing terms must be obtained.
-
+/*
+ * PicoDrive
+ * (c) Copyright Dave, 2004
+ * (C) notaz, 2006-2009
+ *
+ * This work is licensed under the terms of MAME license.
+ * See COPYING file in the top-level directory.
+ */
 
 #include "pico_int.h"
-#include "cd/gfx_cd.h"
+#define NEED_DMA_SOURCE
+#include "memory.h"
 
+int line_base_cycles;
 extern const unsigned char  hcounts_32[];
 extern const unsigned char  hcounts_40[];
 
@@ -20,7 +22,7 @@ typedef unsigned int   u32;
 #define UTYPES_DEFINED
 #endif
 
-int (*PicoDmaHook)(unsigned int source, int len, unsigned short **srcp, unsigned short **limitp) = NULL;
+int (*PicoDmaHook)(unsigned int source, int len, unsigned short **base, unsigned int *mask) = NULL;
 
 static __inline void AutoIncrement(void)
 {
@@ -36,7 +38,7 @@ static void VideoWrite(u16 d)
     case 1: if(a&1) d=(u16)((d<<8)|(d>>8)); // If address is odd, bytes are swapped (which game needs this?)
             Pico.vram [(a>>1)&0x7fff]=d;
             if (a - ((unsigned)(Pico.video.reg[5]&0x7f) << 9) < 0x400)
-              rendstatus |= PDRAW_DIRTY_SPRITES;
+              Pico.est.rendstatus |= PDRAW_DIRTY_SPRITES;
             break;
     case 3: Pico.m.dirtyPal = 1;
             Pico.cram [(a>>1)&0x003f]=d; break; // wraps (Desert Strike)
@@ -72,143 +74,121 @@ static int GetDmaLength(void)
   // 16-bit words to transfer:
   len =pvid->reg[0x13];
   len|=pvid->reg[0x14]<<8;
-  // Charles MacDonald:
-  if(!len) len = 0xffff;
+  len = ((len - 1) & 0xffff) + 1;
   return len;
 }
 
-static void DmaSlow(int len)
+static void DmaSlow(int len, unsigned int source)
 {
-  u16 *pd=0, *pdend, *r;
-  unsigned int a=Pico.video.addr, a2, d;
-  unsigned char inc=Pico.video.reg[0xf];
-  unsigned int source;
-
-  source =Pico.video.reg[0x15]<<1;
-  source|=Pico.video.reg[0x16]<<9;
-  source|=Pico.video.reg[0x17]<<17;
+  u32 inc = Pico.video.reg[0xf];
+  u32 a = Pico.video.addr;
+  u16 *r, *base = NULL;
+  u32 mask = 0x1ffff;
 
-  elprintf(EL_VDPDMA, "DmaSlow[%i] %06x->%04x len %i inc=%i blank %i [%i] @ %06x",
+  elprintf(EL_VDPDMA, "DmaSlow[%i] %06x->%04x len %i inc=%i blank %i [%u] @ %06x",
     Pico.video.type, source, a, len, inc, (Pico.video.status&8)||!(Pico.video.reg[1]&0x40),
     SekCyclesDone(), SekPc);
 
   Pico.m.dma_xfers += len;
-  if ((PicoAHW & PAHW_MCD) && (PicoOpt & POPT_EN_MCD_PSYNC)) SekCyclesBurn(CheckDMA());
-  else SekEndTimeslice(SekCyclesLeftNoMCD - CheckDMA());
+  if (Pico.m.dma_xfers < len) // lame 16bit var
+    Pico.m.dma_xfers = ~0;
+  SekCyclesBurnRun(CheckDMA());
 
-  if ((source&0xe00000)==0xe00000) { // Ram
-    pd=(u16 *)(Pico.ram+(source&0xfffe));
-    pdend=(u16 *)(Pico.ram+0x10000);
+  if ((source & 0xe00000) == 0xe00000) { // Ram
+    base = (u16 *)Pico.ram;
+    mask = 0xffff;
   }
   else if (PicoAHW & PAHW_MCD)
   {
-    elprintf(EL_VDPDMA, "DmaSlow CD, r3=%02x", Pico_mcd->s68k_regs[3]);
-    if(source<0x20000) { // Bios area
-      pd=(u16 *)(Pico_mcd->bios+(source&~1));
-      pdend=(u16 *)(Pico_mcd->bios+0x20000);
-    } else if ((source&0xfc0000)==0x200000) { // Word Ram
-      source -= 2;
-      if (!(Pico_mcd->s68k_regs[3]&4)) { // 2M mode
-        pd=(u16 *)(Pico_mcd->word_ram2M+(source&0x3fffe));
-        pdend=(u16 *)(Pico_mcd->word_ram2M+0x40000);
+    u8 r3 = Pico_mcd->s68k_regs[3];
+    elprintf(EL_VDPDMA, "DmaSlow CD, r3=%02x", r3);
+    if (source < 0x20000) { // Bios area
+      base = (u16 *)Pico_mcd->bios;
+    } else if ((source & 0xfc0000) == 0x200000) { // Word Ram
+      if (!(r3 & 4)) { // 2M mode
+        base = (u16 *)(Pico_mcd->word_ram2M + (source & 0x20000));
       } else {
         if (source < 0x220000) { // 1M mode
-          int bank = Pico_mcd->s68k_regs[3]&1;
-          pd=(u16 *)(Pico_mcd->word_ram1M[bank]+(source&0x1fffe));
-          pdend=(u16 *)(Pico_mcd->word_ram1M[bank]+0x20000);
+          int bank = r3 & 1;
+          base = (u16 *)(Pico_mcd->word_ram1M[bank]);
         } else {
-          DmaSlowCell(source, a, len, inc);
+          DmaSlowCell(source - 2, a, len, inc);
           return;
         }
       }
-    } else if ((source&0xfe0000)==0x020000) { // Prg Ram
-      u8 *prg_ram = Pico_mcd->prg_ram_b[Pico_mcd->s68k_regs[3]>>6];
-      pd=(u16 *)(prg_ram+(source&0x1fffe));
-      pdend=(u16 *)(prg_ram+0x20000);
-    } else {
-      elprintf(EL_VDPDMA|EL_ANOMALY, "DmaSlow[%i] %06x->%04x: FIXME: unsupported src", Pico.video.type, source, a);
-      return;
+      source -= 2;
+    } else if ((source & 0xfe0000) == 0x020000) { // Prg Ram
+      base = (u16 *)Pico_mcd->prg_ram_b[r3 >> 6];
+      source -= 2; // XXX: test
     }
   }
   else
   {
     // if we have DmaHook, let it handle ROM because of possible DMA delay
-    if (PicoDmaHook && PicoDmaHook(source, len, &pd, &pdend));
-    else if (source<Pico.romsize) { // Rom
-      pd=(u16 *)(Pico.rom+(source&~1));
-      pdend=(u16 *)(Pico.rom+Pico.romsize);
-    }
-    else {
-      elprintf(EL_VDPDMA|EL_ANOMALY, "DmaSlow[%i] %06x->%04x: invalid src", Pico.video.type, source, a);
-      return;
-    }
+    u32 source2;
+    if (PicoDmaHook && (source2 = PicoDmaHook(source, len, &base, &mask)))
+      source = source2;
+    else // Rom
+      base = m68k_dma_source(source);
   }
-
-  // overflow protection, might break something..
-  if (len > pdend - pd) {
-    len = pdend - pd;
-    elprintf(EL_VDPDMA|EL_ANOMALY, "DmaSlow overflow");
+  if (!base) {
+    elprintf(EL_VDPDMA|EL_ANOMALY, "DmaSlow[%i] %06x->%04x: invalid src", Pico.video.type, source, a);
+    return;
   }
 
+  // operate in words
+  source >>= 1;
+  mask >>= 1;
+
   switch (Pico.video.type)
   {
     case 1: // vram
       r = Pico.vram;
-      if (inc == 2 && !(a&1) && a+len*2 < 0x10000)
+      if (inc == 2 && !(a & 1) && a + len * 2 < 0x10000
+          && !(((source + len - 1) ^ source) & ~mask))
       {
         // most used DMA mode
-        memcpy16(r + (a>>1), pd, len);
-        a += len*2;
+        memcpy((char *)r + a, base + (source & mask), len * 2);
+        a += len * 2;
       }
       else
       {
         for(; len; len--)
         {
-          d=*pd++;
-          if(a&1) d=(d<<8)|(d>>8);
-          r[a>>1] = (u16)d; // will drop the upper bits
+          u16 d = base[source++ & mask];
+          if(a & 1) d=(d<<8)|(d>>8);
+          r[a >> 1] = d;
           // AutoIncrement
-          a=(u16)(a+inc);
-          // didn't src overlap?
-          //if(pd >= pdend) pd-=0x8000; // should be good for RAM, bad for ROM
+          a = (u16)(a + inc);
         }
       }
-      rendstatus |= PDRAW_DIRTY_SPRITES;
+      Pico.est.rendstatus |= PDRAW_DIRTY_SPRITES;
       break;
 
     case 3: // cram
       Pico.m.dirtyPal = 1;
       r = Pico.cram;
-      for(a2=a&0x7f; len; len--)
+      for (; len; len--)
       {
-        r[a2>>1] = (u16)*pd++; // bit 0 is ignored
+        r[(a / 2) & 0x3f] = base[source++ & mask];
         // AutoIncrement
-        a2+=inc;
-        // didn't src overlap?
-        //if(pd >= pdend) pd-=0x8000;
-        // good dest?
-        if(a2 >= 0x80) break; // Todds Adventures in Slime World / Andre Agassi tennis
+        a += inc;
       }
-      a=(a&0xff00)|a2;
       break;
 
-    case 5: // vsram[a&0x003f]=d;
+    case 5: // vsram
       r = Pico.vsram;
-      for(a2=a&0x7f; len; len--)
+      for (; len; len--)
       {
-        r[a2>>1] = (u16)*pd++;
+        r[(a / 2) & 0x3f] = base[source++ & mask];
         // AutoIncrement
-        a2+=inc;
-        // didn't src overlap?
-        //if(pd >= pdend) pd-=0x8000;
-        // good dest?
-        if(a2 >= 0x80) break;
+        a += inc;
       }
-      a=(a&0xff00)|a2;
       break;
 
     default:
-      elprintf(EL_VDPDMA|EL_ANOMALY, "DMA with bad type %i", Pico.video.type);
+      if (Pico.video.type != 0 || (EL_LOGMASK & EL_VDPDMA))
+        elprintf(EL_VDPDMA|EL_ANOMALY, "DMA with bad type %i", Pico.video.type);
       break;
   }
   // remember addr
@@ -219,82 +199,108 @@ static void DmaCopy(int len)
 {
   u16 a=Pico.video.addr;
   unsigned char *vr = (unsigned char *) Pico.vram;
-  unsigned char *vrs;
   unsigned char inc=Pico.video.reg[0xf];
   int source;
   elprintf(EL_VDPDMA, "DmaCopy len %i [%i]", len, SekCyclesDone());
 
   Pico.m.dma_xfers += len;
+  if (Pico.m.dma_xfers < len)
+    Pico.m.dma_xfers = ~0;
   Pico.video.status |= 2; // dma busy
 
   source =Pico.video.reg[0x15];
   source|=Pico.video.reg[0x16]<<8;
-  vrs=vr+source;
-
-  if (source+len > 0x10000) len=0x10000-source; // clip??
 
   for (; len; len--)
   {
-    vr[a] = *vrs++;
+    vr[a] = vr[source++ & 0xffff];
     // AutoIncrement
     a=(u16)(a+inc);
   }
   // remember addr
   Pico.video.addr=a;
-  rendstatus |= PDRAW_DIRTY_SPRITES;
+  Pico.est.rendstatus |= PDRAW_DIRTY_SPRITES;
 }
 
-// check: Contra, Megaman
-// note: this is still inaccurate
-static void DmaFill(int data)
+static NOINLINE void DmaFill(int data)
 {
-  int len;
   unsigned short a=Pico.video.addr;
   unsigned char *vr=(unsigned char *) Pico.vram;
   unsigned char high = (unsigned char) (data >> 8);
   unsigned char inc=Pico.video.reg[0xf];
+  int source;
+  int len, l;
 
-  len=GetDmaLength();
+  len = GetDmaLength();
   elprintf(EL_VDPDMA, "DmaFill len %i inc %i [%i]", len, inc, SekCyclesDone());
 
   Pico.m.dma_xfers += len;
+  if (Pico.m.dma_xfers < len) // lame 16bit var
+    Pico.m.dma_xfers = ~0;
   Pico.video.status |= 2; // dma busy
 
-  // from Charles MacDonald's genvdp.txt:
-  // Write lower byte to address specified
-  vr[a] = (unsigned char) data;
-  a=(u16)(a+inc);
-
-  if (!inc) len=1;
-
-  for (; len; len--) {
-    // Write upper byte to adjacent address
-    // (here we are byteswapped, so address is already 'adjacent')
-    vr[a] = high;
+  switch (Pico.video.type)
+  {
+    case 1: // vram
+      for (l = len; l; l--) {
+        // Write upper byte to adjacent address
+        // (here we are byteswapped, so address is already 'adjacent')
+        vr[a] = high;
 
-    // Increment address register
-    a=(u16)(a+inc);
+        // Increment address register
+        a = (u16)(a + inc);
+      }
+      break;
+    case 3:   // cram
+    case 5: { // vsram
+      // TODO: needs fifo; anyone using these?
+      static int once;
+      if (!once++)
+        elprintf(EL_STATUS|EL_ANOMALY|EL_VDPDMA, "TODO: cram/vsram fill");
+    }
+    default:
+      a += len * inc;
+      break;
   }
-  // remember addr
-  Pico.video.addr=a;
-  // update length
-  Pico.video.reg[0x13] = Pico.video.reg[0x14] = 0; // Dino Dini's Soccer (E) (by Haze)
 
-  rendstatus |= PDRAW_DIRTY_SPRITES;
+  // remember addr
+  Pico.video.addr = a;
+  // register update
+  Pico.video.reg[0x13] = Pico.video.reg[0x14] = 0;
+  source  = Pico.video.reg[0x15];
+  source |= Pico.video.reg[0x16] << 8;
+  source += len;
+  Pico.video.reg[0x15] = source;
+  Pico.video.reg[0x16] = source >> 8;
+
+  Pico.est.rendstatus |= PDRAW_DIRTY_SPRITES;
 }
 
-static void CommandDma(void)
+static NOINLINE void CommandDma(void)
 {
   struct PicoVideo *pvid=&Pico.video;
-  int len=0,method=0;
+  u32 len, method;
+  u32 source;
 
   if ((pvid->reg[1]&0x10)==0) return; // DMA not enabled
 
-  len=GetDmaLength();
+  len = GetDmaLength();
+  source =Pico.video.reg[0x15];
+  source|=Pico.video.reg[0x16] << 8;
+  source|=Pico.video.reg[0x17] << 16;
 
   method=pvid->reg[0x17]>>6;
-  if (method< 2) DmaSlow(len); // 68000 to VDP
-  if (method==3) DmaCopy(len); // VRAM Copy
+  if (method < 2)
+    DmaSlow(len, source << 1); // 68000 to VDP
+  else if (method == 3)
+    DmaCopy(len); // VRAM Copy
+  else
+    return;
+
+  source += len;
+  Pico.video.reg[0x13] = Pico.video.reg[0x14] = 0;
+  Pico.video.reg[0x15] = source;
+  Pico.video.reg[0x16] = source >> 8;
 }
 
 static void CommandChange(void)
@@ -311,15 +317,12 @@ static void CommandChange(void)
   addr =(cmd>>16)&0x3fff;
   addr|=(cmd<<14)&0xc000;
   pvid->addr=(unsigned short)addr;
-
-  // Check for dma:
-  if (cmd&0x80) CommandDma();
 }
 
-static __inline void DrawSync(int blank_on)
+static void DrawSync(int blank_on)
 {
   if (Pico.m.scanline < 224 && !(PicoOpt & POPT_ALT_RENDERER) &&
-      !PicoSkipFrame && DrawScanline <= Pico.m.scanline) {
+      !PicoSkipFrame && Pico.est.DrawScanline <= Pico.m.scanline) {
     //elprintf(EL_ANOMALY, "sync");
     PicoDrawSync(Pico.m.scanline, blank_on);
   }
@@ -347,28 +350,23 @@ PICO_INTERNAL_ASM void PicoVideoWrite(unsigned int a,unsigned short d)
       pvid->pending=0;
     }
 
-    // If a DMA fill has been set up, do it
-    if ((pvid->command&0x80) && (pvid->reg[1]&0x10) && (pvid->reg[0x17]>>6)==2)
+    // preliminary FIFO emulation for Chaos Engine, The (E)
+    if (!(pvid->status&8) && (pvid->reg[1]&0x40) && !(PicoOpt&POPT_DIS_VDP_FIFO)) // active display?
     {
-      DmaFill(d);
-    }
-    else
-    {
-      // preliminary FIFO emulation for Chaos Engine, The (E)
-      if (!(pvid->status&8) && (pvid->reg[1]&0x40) && !(PicoOpt&POPT_DIS_VDP_FIFO)) // active display?
-      {
-        pvid->status&=~0x200; // FIFO no longer empty
-        pvid->lwrite_cnt++;
-        if (pvid->lwrite_cnt >= 4) pvid->status|=0x100; // FIFO full
-        if (pvid->lwrite_cnt >  4) {
-          SekCyclesBurn(32); // penalty // 488/12-8
-          if (SekCycleCnt>=SekCycleAim) SekEndRun(0);
-        }
-        elprintf(EL_ASVDP, "VDP data write: %04x [%06x] {%i} #%i @ %06x", d, Pico.video.addr,
-                 Pico.video.type, pvid->lwrite_cnt, SekPc);
+      pvid->status&=~0x200; // FIFO no longer empty
+      pvid->lwrite_cnt++;
+      if (pvid->lwrite_cnt >= 4) pvid->status|=0x100; // FIFO full
+      if (pvid->lwrite_cnt >  4) {
+        SekCyclesBurnRun(32); // penalty // 488/12-8
       }
-      VideoWrite(d);
+      elprintf(EL_ASVDP, "VDP data write: %04x [%06x] {%i} #%i @ %06x", d, Pico.video.addr,
+               Pico.video.type, pvid->lwrite_cnt, SekPc);
     }
+    VideoWrite(d);
+
+    if ((pvid->command&0x80) && (pvid->reg[1]&0x10) && (pvid->reg[0x17]>>6)==2)
+      DmaFill(d);
+
     return;
   }
 
@@ -376,12 +374,16 @@ PICO_INTERNAL_ASM void PicoVideoWrite(unsigned int a,unsigned short d)
   {
     if (pvid->pending)
     {
-      if (d & 0x80) DrawSync(0); // only need sync for DMA
       // Low word of command:
-      pvid->command&=0xffff0000;
-      pvid->command|=d;
-      pvid->pending=0;
+      pvid->command &= 0xffff0000;
+      pvid->command |= d;
+      pvid->pending = 0;
       CommandChange();
+      // Check for dma:
+      if (d & 0x80) {
+        DrawSync(0);
+        CommandDma();
+      }
     }
     else
     {
@@ -397,7 +399,8 @@ PICO_INTERNAL_ASM void PicoVideoWrite(unsigned int a,unsigned short d)
           return;
         }
 
-        if (num == 1 && !(d&0x40) && SekCyclesLeft > 390) blank_on = 1;
+        if (num == 1 && !(d&0x40) && SekCyclesDone() - line_base_cycles <= 488-390)
+          blank_on = 1;
         DrawSync(blank_on);
         pvid->reg[num]=(unsigned char)d;
         switch (num)
@@ -412,7 +415,7 @@ PICO_INTERNAL_ASM void PicoVideoWrite(unsigned int a,unsigned short d)
             goto update_irq;
           case 0x05:
             //elprintf(EL_STATUS, "spritep moved to %04x", (unsigned)(Pico.video.reg[5]&0x7f) << 9);
-            if (d^dold) rendstatus |= PDRAW_SPRITES_MOVED;
+            if (d^dold) Pico.est.rendstatus |= PDRAW_SPRITES_MOVED;
             break;
           case 0x0c:
             // renderers should update their palettes if sh/hi mode is changed
@@ -424,7 +427,7 @@ PICO_INTERNAL_ASM void PicoVideoWrite(unsigned int a,unsigned short d)
 update_irq:
 #ifndef EMU_CORE_DEBUG
         // update IRQ level
-        if (!SekShouldInterrupt) // hack
+        if (!SekShouldInterrupt()) // hack
         {
           int lines, pints, irq=0;
           lines = (pvid->reg[1] & 0x20) | (pvid->reg[0] & 0x10);
@@ -458,7 +461,8 @@ PICO_INTERNAL_ASM unsigned int PicoVideoRead(unsigned int a)
     unsigned int d;
     d=pv->status;
     //if (PicoOpt&POPT_ALT_RENDERER) d|=0x0020; // sprite collision (Shadow of the Beast)
-    if (SekCyclesLeft < 84+4)      d|=0x0004; // H-Blank (Sonic3 vs)
+    if (SekCyclesDone() - line_base_cycles >= 488-88)
+      d|=0x0004; // H-Blank (Sonic3 vs)
 
     d |= ((pv->reg[1]&0x40)^0x40) >> 3;  // set V-Blank if display is disabled
     d |= (pv->pending_ints&0x20)<<2;     // V-int pending?
@@ -488,12 +492,11 @@ PICO_INTERNAL_ASM unsigned int PicoVideoRead(unsigned int a)
   if ((a&0x1c)==0x08)
   {
     unsigned int d;
-    int lineCycles;
 
-    lineCycles = (488-SekCyclesLeft)&0x1ff;
+    d = (SekCyclesDone() - line_base_cycles) & 0x1ff; // FIXME
     if (Pico.video.reg[12]&1)
-         d = hcounts_40[lineCycles];
-    else d = hcounts_32[lineCycles];
+         d = hcounts_40[d];
+    else d = hcounts_32[d];
 
     elprintf(EL_HVCNT, "hv: %02x %02x (%i) @ %06x", d, Pico.video.v_counter, SekCyclesDone(), SekPc);
     return d | (Pico.video.v_counter << 8);
@@ -527,7 +530,7 @@ unsigned int PicoVideoRead8(unsigned int a)
       //if (PicoOpt&POPT_ALT_RENDERER) d|=0x0020; // sprite collision (Shadow of the Beast)
       d |= ((Pico.video.reg[1]&0x40)^0x40) >> 3;  // set V-Blank if display is disabled
       d |= (Pico.video.pending_ints&0x20)<<2;     // V-int pending?
-      if (SekCyclesLeft < 84+4) d |= 4;    // H-Blank
+      if (SekCyclesDone() - line_base_cycles >= 488-88) d |= 4;    // H-Blank
       Pico.video.pending = 0;
       elprintf(EL_SR, "SR read (l): %02x @ %06x", d, SekPc);
       return d;
@@ -535,7 +538,7 @@ unsigned int PicoVideoRead8(unsigned int a)
       elprintf(EL_HVCNT, "vcounter: %02x (%i) @ %06x", Pico.video.v_counter, SekCyclesDone(), SekPc);
       return Pico.video.v_counter;
     case 9:
-      d = (488-SekCyclesLeft)&0x1ff;
+      d = (SekCyclesDone() - line_base_cycles) & 0x1ff; // FIXME
       if (Pico.video.reg[12]&1)
            d = hcounts_40[d];
       else d = hcounts_32[d];
@@ -545,3 +548,5 @@ unsigned int PicoVideoRead8(unsigned int a)
 
   return 0;
 }
+
+// vim:shiftwidth=2:ts=2:expandtab