]> Shamusworld >> Repos - virtualjaguar/blobdiff - src/tom.cpp
Fixed fullscreen mode, shrunk screen width.
[virtualjaguar] / src / tom.cpp
index c9d0f3f04b1574784217a5fef3c0cb30ad08ffe1..845f1551f37f1b88548a3e796ccc39a977300ee6 100644 (file)
 //#define RIGHT_VISIBLE_HC             (1488 - 32)
 // Split the difference? (Seems to be OK for the most part...)
 
-// (-8 +8)*4 is for opening up the display by 16 pixels (may go to 20). Need to change VIRTUAL_SCREEN_WIDTH to match this as well (went from 320 to 336; this is 4 HCs per one of those pixels). <-- actually, not true. This is only place referenced.
-#define LEFT_VISIBLE_HC                        (208 - 16 - (10 * 4))
+// (-10 +10)*4 is for opening up the display by 16 pixels (may go to 20). Need to change VIRTUAL_SCREEN_WIDTH to match this as well (went from 320 to 340; this is 4 HCs per one of those pixels).
+//NB: Went back to 330. May shrink more. :-)
+//#define LEFT_VISIBLE_HC                      (208 - 16 - (8 * 4))
+//#define LEFT_VISIBLE_HC                      (208 - 16 - (3 * 4))
+#define LEFT_VISIBLE_HC                        (208 - 16 - (1 * 4))
 //#define RIGHT_VISIBLE_HC             (1488 - 16 + (10 * 4))
 #define RIGHT_VISIBLE_HC               (LEFT_VISIBLE_HC + (VIRTUAL_SCREEN_WIDTH * 4))
 //#define TOP_VISIBLE_VC               25
 //Are these PAL horizontals correct?
 //They seem to be for the most part, but there are some games that seem to be
 //shifted over to the right from this "window".
-#define LEFT_VISIBLE_HC_PAL            (208 - 16 - (10 * 4))
+//#define LEFT_VISIBLE_HC_PAL          (208 - 16 - (4 * 4))
+//#define LEFT_VISIBLE_HC_PAL          (208 - 16 - (-1 * 4))
+#define LEFT_VISIBLE_HC_PAL            (208 - 16 - (-3 * 4))
 //#define RIGHT_VISIBLE_HC_PAL (1488 - 16 + (10 * 4))
 #define RIGHT_VISIBLE_HC_PAL   (LEFT_VISIBLE_HC_PAL + (VIRTUAL_SCREEN_WIDTH * 4))
 #define TOP_VISIBLE_VC_PAL             67
@@ -558,6 +563,7 @@ uint32 RGB16ToRGB32[0x10000];
 uint32 CRY16ToRGB32[0x10000];
 uint32 MIX16ToRGB32[0x10000];
 
+
 #warning "This is not endian-safe. !!! FIX !!!"
 void TOMFillLookupTables(void)
 {
@@ -589,42 +595,50 @@ void TOMFillLookupTables(void)
        }
 }
 
+
 void TOMSetPendingJERRYInt(void)
 {
        tom_jerry_int_pending = 1;
 }
 
+
 void TOMSetPendingTimerInt(void)
 {
        tom_timer_int_pending = 1;
 }
 
+
 void TOMSetPendingObjectInt(void)
 {
        tom_object_int_pending = 1;
 }
 
+
 void TOMSetPendingGPUInt(void)
 {
        tom_gpu_int_pending = 1;
 }
 
+
 void TOMSetPendingVideoInt(void)
 {
        tom_video_int_pending = 1;
 }
 
+
 uint8 * TOMGetRamPointer(void)
 {
        return tomRam8;
 }
 
+
 uint8 TOMGetVideoMode(void)
 {
        uint16 vmode = GET16(tomRam8, VMODE);
        return ((vmode & VARMOD) >> 6) | ((vmode & MODE) >> 1);
 }
 
+
 //Used in only one place (and for debug purposes): OBJECTP.CPP
 #warning "Used in only one place (and for debug purposes): OBJECTP.CPP !!! FIX !!!"
 uint16 TOMGetVDB(void)
@@ -632,6 +646,8 @@ uint16 TOMGetVDB(void)
        return GET16(tomRam8, VDB);
 }
 
+
+#define LEFT_BG_FIX
 //
 // 16 BPP CRY/RGB mixed mode rendering
 //
@@ -646,15 +662,32 @@ void tom_render_16bpp_cry_rgb_mix_scanline(uint32 * backbuffer)
        //NOTE: May have to check HDB2 as well!
        // Get start position in HC ticks
        int16 startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);
+       // Convert to pixels
        startPos /= pwidth;
+
        if (startPos < 0)
+               // This is x2 because current_line_buffer is uint8 & we're in a 16bpp mode
                current_line_buffer += 2 * -startPos;
        else
 //This case doesn't properly handle the "start on the right side of virtual screen" case
 //Dunno why--looks Ok...
-//What *is* for sure wrong is that it doesn't copy the linebuffer's BG pixels...
+//What *is* for sure wrong is that it doesn't copy the linebuffer's BG pixels... [FIXED NOW]
 //This should likely be 4 instead of 2 (?--not sure)
+// Actually, there should be NO multiplier, as startPos is expressed in PIXELS
+// and so is the backbuffer.
+#ifdef LEFT_BG_FIX
+       {
+               uint8 g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32 pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16 i=0; i<startPos; i++)
+                       *backbuffer++ = pixel;
+
+               width -= startPos;
+       }
+#else
                backbuffer += 2 * startPos, width -= startPos;
+#endif
 
        while (width)
        {
@@ -665,6 +698,7 @@ void tom_render_16bpp_cry_rgb_mix_scanline(uint32 * backbuffer)
        }
 }
 
+
 //
 // 16 BPP CRY mode rendering
 //
@@ -682,8 +716,20 @@ void tom_render_16bpp_cry_scanline(uint32 * backbuffer)
        if (startPos < 0)
                current_line_buffer += 2 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8 g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32 pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16 i=0; i<startPos; i++)
+                       *backbuffer++ = pixel;
+
+               width -= startPos;
+       }
+#else
 //This should likely be 4 instead of 2 (?--not sure)
                backbuffer += 2 * startPos, width -= startPos;
+#endif
 
        while (width)
        {
@@ -694,6 +740,7 @@ void tom_render_16bpp_cry_scanline(uint32 * backbuffer)
        }
 }
 
+
 //
 // 24 BPP mode rendering
 //
@@ -711,8 +758,20 @@ void tom_render_24bpp_scanline(uint32 * backbuffer)
        if (startPos < 0)
                current_line_buffer += 4 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8 g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32 pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16 i=0; i<startPos; i++)
+                       *backbuffer++ = pixel;
+
+               width -= startPos;
+       }
+#else
 //This should likely be 4 instead of 2 (?--not sure)
                backbuffer += 2 * startPos, width -= startPos;
+#endif
 
        while (width)
        {
@@ -726,6 +785,7 @@ void tom_render_24bpp_scanline(uint32 * backbuffer)
        }
 }
 
+
 //Seems to me that this is NOT a valid mode--the JTRM seems to imply that you would need
 //extra hardware outside of the Jaguar console to support this!
 //
@@ -745,6 +805,7 @@ void tom_render_16bpp_direct_scanline(uint32 * backbuffer)
        }
 }
 
+
 //
 // 16 BPP RGB mode rendering
 //
@@ -765,8 +826,20 @@ void tom_render_16bpp_rgb_scanline(uint32 * backbuffer)
        if (startPos < 0)
                current_line_buffer += 2 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8 g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32 pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16 i=0; i<startPos; i++)
+                       *backbuffer++ = pixel;
+
+               width -= startPos;
+       }
+#else
 //This should likely be 4 instead of 2 (?--not sure)
                backbuffer += 2 * startPos, width -= startPos;
+#endif
 
        while (width)
        {
@@ -778,11 +851,6 @@ void tom_render_16bpp_rgb_scanline(uint32 * backbuffer)
 }
 
 
-/*void TOMResetBackbuffer(uint32 * backbuffer)
-{
-       TOMBackbuffer = backbuffer;
-}*/
-
 //
 // Process a single scanline
 // (this is bad terminology; each tick of the VC is actually a half-line)
@@ -1000,6 +1068,7 @@ void tom_render_24bpp_scanline(uint32 * backbuffer)
        }
 }
 
+
 //
 // TOM initialization
 //
@@ -1011,6 +1080,7 @@ void TOMInit(void)
        TOMReset();
 }
 
+
 void TOMDone(void)
 {
        OPDone();
@@ -1026,6 +1096,7 @@ void TOMDone(void)
 //     memory_free(tom_cry_rgb_mix_lut);
 }
 
+
 uint32 TOMGetVideoModeWidth(void)
 {
        //These widths are pretty bogus. Should use HDB1/2 & HDE/HBB & PWIDTH to calc the width...
@@ -1084,6 +1155,7 @@ uint32 TOMGetVideoModeWidth(void)
 // That's basically what we're doing now...!
 }
 
+
 // *** SPECULATION ***
 // It might work better to virtualize the height settings, i.e., set the vertical
 // height at 240 lines and clip using the VDB and VDE/VP registers...
@@ -1119,6 +1191,7 @@ uint32 TOMGetVideoModeHeight(void)
        return (vjs.hardwareTypeNTSC ? 240 : 256);
 }
 
+
 //
 // TOM reset code
 // Now PAL friendly!
@@ -1222,6 +1295,7 @@ void TOMReset(void)
        tomTimerCounter = 0;
 }
 
+
 //
 // TOM byte access (read)
 //
@@ -1257,6 +1331,7 @@ uint8 TOMReadByte(uint32 offset, uint32 who/*=UNKNOWN*/)
        return tomRam8[offset & 0x3FFF];
 }
 
+
 //
 // TOM word access (read)
 //
@@ -1268,7 +1343,7 @@ uint16 TOMReadWord(uint32 offset, uint32 who/*=UNKNOWN*/)
        WriteLog("TOM: Reading word at %06X for %s\n", offset, whoName[who]);
 #endif
 if (offset >= 0xF02000 && offset <= 0xF020FF)
-       WriteLog("TOM: Read attempted from GPU register file by %s (unimplemented)!\n", whoName[who]);
+       WriteLog("TOM: ReadWord attempted from GPU register file by %s (unimplemented)!\n", whoName[who]);
 
        if (offset == 0xF000E0)
        {
@@ -1314,6 +1389,7 @@ if (offset >= 0xF02000 && offset <= 0xF020FF)
        return (TOMReadByte(offset, who) << 8) | TOMReadByte(offset + 1, who);
 }
 
+
 #define TOM_STRICT_MEMORY_ACCESS
 //
 // TOM byte access (write)
@@ -1397,6 +1473,7 @@ void TOMWriteByte(uint32 offset, uint8 data, uint32 who/*=UNKNOWN*/)
        tomRam8[offset & 0x3FFF] = data;
 }
 
+
 //
 // TOM word access (write)
 //
@@ -1429,7 +1506,7 @@ void TOMWriteWord(uint32 offset, uint16 data, uint32 who/*=UNKNOWN*/)
 //if (offset == 0xF00000 + MEMCON2)
 //     WriteLog("TOM: Memory Configuration 2 written by %s: %04X\n", whoName[who], data);
 if (offset >= 0xF02000 && offset <= 0xF020FF)
-       WriteLog("TOM: Write attempted to GPU register file by %s (unimplemented)!\n", whoName[who]);
+       WriteLog("TOM: WriteWord attempted to GPU register file by %s (unimplemented)!\n", whoName[who]);
 
        if ((offset >= GPU_CONTROL_RAM_BASE) && (offset < GPU_CONTROL_RAM_BASE+0x20))
        {
@@ -1596,6 +1673,7 @@ if (offset == HEQ)
 #endif
 }
 
+
 int TOMIRQEnabled(int irq)
 {
        // This is the correct byte in big endian... D'oh!
@@ -1603,6 +1681,7 @@ int TOMIRQEnabled(int irq)
        return tomRam8[INT1 + 1/*0xE1*/] & (1 << irq);
 }
 
+
 // NEW:
 // TOM Programmable Interrupt Timer handler
 // NOTE: TOM's PIT is only enabled if the prescaler is != 0
@@ -1610,6 +1689,7 @@ int TOMIRQEnabled(int irq)
 
 void TOMPITCallback(void);
 
+
 void TOMResetPIT(void)
 {
 #ifndef NEW_TIMER_SYSTEM
@@ -1632,6 +1712,7 @@ void TOMResetPIT(void)
 #endif
 }
 
+
 //
 // TOM Programmable Interrupt Timer handler
 // NOTE: TOM's PIT is only enabled if the prescaler is != 0
@@ -1657,6 +1738,7 @@ void TOMExecPIT(uint32 cycles)
        }
 }
 
+
 void TOMPITCallback(void)
 {
 //     INT1_RREG |= 0x08;                                                      // Set TOM PIT interrupt pending
@@ -1669,3 +1751,4 @@ void TOMPITCallback(void)
 
        TOMResetPIT();
 }
+