]> Shamusworld >> Repos - virtualjaguar/blobdiff - src/tom.cpp
Added Memory Track support. One small step towards full CD-ROM support.
[virtualjaguar] / src / tom.cpp
index 49da93e377c45e6767b87bc3e78d03a916993e0a..6de61169c51bd81910cc183278b2c832a7bf001c 100644 (file)
@@ -3,19 +3,16 @@
 //
 // Originally by David Raingeard (cal2)
 // GCC/SDL port by Niels Wagenaar (Linux/WIN32) and Caz (BeOS)
-// Cleanups and endian wrongness amelioration by James Hammons
+// Cleanups, endian wrongness amelioration, and extensive fixes by James Hammons
 // (C) 2010 Underground Software
 //
 // JLH = James Hammons <jlhamm@acm.org>
 //
 // Who  When        What
-// ---  ----------  -------------------------------------------------------------
+// ---  ----------  -----------------------------------------------------------
 // JLH  01/16/2010  Created this log ;-)
 // JLH  01/20/2011  Change rendering to RGBA, removed unnecessary code
 //
-// Note: Endian wrongness probably stems from the MAME origins of this emu and
-//       the braindead way in which MAME used to handle memory. :-}
-//
 // Note: TOM has only a 16K memory space
 //
 //     ------------------------------------------------------------
 #define MEMCON2                0x02
 #define HC                     0x04
 #define VC                     0x06
-#define OLP                    0x20            // Object list pointer
-#define OBF                    0x26            // Object processor flag
+#define OLP                    0x20    // Object list pointer
+#define OBF                    0x26    // Object processor flag
 #define VMODE          0x28
-#define   MODE         0x0006          // Line buffer to video generator mode
-#define   BGEN         0x0080          // Background enable (CRY & RGB16 only)
-#define   VARMOD       0x0100          // Mixed CRY/RGB16 mode (only works in MODE 0!)
-#define   PWIDTH       0x0E00          // Pixel width in video clock cycles (value written + 1)
-#define BORD1          0x2A            // Border green/red values (8 BPP)
-#define BORD2          0x2C            // Border blue value (8 BPP)
-#define HP                     0x2E            // Values range from 1 - 1024 (value written + 1)
-#define HBB                    0x30            // Horizontal blank begin
+#define   MODE         0x0006  // Line buffer to video generator mode
+#define   BGEN         0x0080  // Background enable (CRY & RGB16 only)
+#define   VARMOD       0x0100  // Mixed CRY/RGB16 mode (only works in MODE 0!)
+#define   PWIDTH       0x0E00  // Pixel width in video clock cycles (value written + 1)
+#define BORD1          0x2A    // Border green/red values (8 BPP)
+#define BORD2          0x2C    // Border blue value (8 BPP)
+#define HP                     0x2E    // Values range from 1 - 1024 (value written + 1)
+#define HBB                    0x30    // Horizontal blank begin
 #define HBE                    0x32
-#define HS                     0x34            // Horizontal sync
-#define HVS                    0x36            // Horizontal vertical sync
-#define HDB1           0x38            // Horizontal display begin 1
+#define HS                     0x34    // Horizontal sync
+#define HVS                    0x36    // Horizontal vertical sync
+#define HDB1           0x38    // Horizontal display begin 1
 #define HDB2           0x3A
 #define HDE                    0x3C
-#define VP                     0x3E            // Value ranges from 1 - 2048 (value written + 1)
-#define VBB                    0x40            // Vertical blank begin
+#define VP                     0x3E    // Value ranges from 1 - 2048 (value written + 1)
+#define VBB                    0x40    // Vertical blank begin
 #define VBE                    0x42
-#define VS                     0x44            // Vertical sync
-#define VDB                    0x46            // Vertical display begin
+#define VS                     0x44    // Vertical sync
+#define VDB                    0x46    // Vertical display begin
 #define VDE                    0x48
-#define VEB                    0x4A            // Vertical equalization begin
-#define VEE                    0x4C            // Vertical equalization end
-#define VI                     0x4E            // Vertical interrupt
+#define VEB                    0x4A    // Vertical equalization begin
+#define VEE                    0x4C    // Vertical equalization end
+#define VI                     0x4E    // Vertical interrupt
 #define PIT0           0x50
 #define PIT1           0x52
-#define HEQ                    0x54            // Horizontal equalization end
-#define BG                     0x58            // Background color
+#define HEQ                    0x54    // Horizontal equalization end
+#define BG                     0x58    // Background color
 #define INT1           0xE0
+#define INT2           0xE2
 
-//NOTE: These arbitrary cutoffs are NOT taken into account for PAL jaguar screens. !!! FIX !!! [DONE]
-
-// Arbitrary video cutoff values (i.e., first/last visible spots on a TV, in HC ticks)
+// Arbitrary video cutoff values (i.e., first/last visible spots on a TV, in HC
+// ticks)
 // Also note that VC is in *half* lines, i.e. divide by 2 to get the scanline
 /*#define LEFT_VISIBLE_HC                      208
 #define RIGHT_VISIBLE_HC               1528//*/
 // Split the difference? (Seems to be OK for the most part...)
 
 // (-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).
-#define LEFT_VISIBLE_HC                        (208 - 16 - (8 * 4))
+//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 - (4 * 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
 //(It's easier to do it here, though...)
 //#define TOM_DEBUG
 
-uint8 tomRam8[0x4000];
-uint32 tomWidth, tomHeight;
-uint32 tomTimerPrescaler;
-uint32 tomTimerDivider;
-int32 tomTimerCounter;
-uint16 tom_jerry_int_pending, tom_timer_int_pending, tom_object_int_pending,
+uint8_t tomRam8[0x4000];
+uint32_t tomWidth, tomHeight;
+uint32_t tomTimerPrescaler;
+uint32_t tomTimerDivider;
+int32_t tomTimerCounter;
+uint16_t tom_jerry_int_pending, tom_timer_int_pending, tom_object_int_pending,
        tom_gpu_int_pending, tom_video_int_pending;
 
 // These are set by the "user" of the Jaguar core lib, since these are
 // OS/system dependent.
-uint32 * screenBuffer;
-uint32 screenPitch;
+uint32_t * screenBuffer;
+uint32_t screenPitch;
 
 static const char * videoMode_to_str[8] =
        { "16 BPP CRY", "24 BPP RGB", "16 BPP DIRECT", "16 BPP RGB",
          "Mixed mode", "24 BPP RGB", "16 BPP DIRECT", "16 BPP RGB" };
 
-typedef void (render_xxx_scanline_fn)(uint32 *);
+typedef void (render_xxx_scanline_fn)(uint32_t *);
 
 // Private function prototypes
 
-void tom_render_16bpp_cry_scanline(uint32 * backbuffer);
-void tom_render_24bpp_scanline(uint32 * backbuffer);
-void tom_render_16bpp_direct_scanline(uint32 * backbuffer);
-void tom_render_16bpp_rgb_scanline(uint32 * backbuffer);
-void tom_render_16bpp_cry_rgb_mix_scanline(uint32 * backbuffer);
+void tom_render_16bpp_cry_scanline(uint32_t * backbuffer);
+void tom_render_24bpp_scanline(uint32_t * backbuffer);
+void tom_render_16bpp_direct_scanline(uint32_t * backbuffer);
+void tom_render_16bpp_rgb_scanline(uint32_t * backbuffer);
+void tom_render_16bpp_cry_rgb_mix_scanline(uint32_t * backbuffer);
 
 //render_xxx_scanline_fn * scanline_render_normal[] =
 render_xxx_scanline_fn * scanline_render[] =
@@ -554,223 +556,306 @@ Trevor McFur
 Vertical resolution: 238 lines
 */
 
-uint32 RGB16ToRGB32[0x10000];
-uint32 CRY16ToRGB32[0x10000];
-uint32 MIX16ToRGB32[0x10000];
+// 16-bit color lookup tables
+uint32_t RGB16ToRGB32[0x10000];
+uint32_t CRY16ToRGB32[0x10000];
+uint32_t MIX16ToRGB32[0x10000];
+
 
 #warning "This is not endian-safe. !!! FIX !!!"
 void TOMFillLookupTables(void)
 {
        // NOTE: Jaguar 16-bit (non-CRY) color is RBG 556 like so:
        //       RRRR RBBB BBGG GGGG
-       for(uint32 i=0; i<0x10000; i++)
-//hm.          RGB16ToRGB32[i] = 0xFF000000
-//                     | ((i & 0xF100) >> 8)  | ((i & 0xE000) >> 13)
-//                     | ((i & 0x07C0) << 13) | ((i & 0x0700) << 8)
-//                     | ((i & 0x003F) << 10) | ((i & 0x0030) << 4);
+       for(uint32_t i=0; i<0x10000; i++)
                RGB16ToRGB32[i] = 0x000000FF
-                       | ((i & 0xF100) << 16)                                  // Red
+                       | ((i & 0xF800) << 16)                                  // Red
                        | ((i & 0x003F) << 18)                                  // Green
                        | ((i & 0x07C0) << 5);                                  // Blue
 
-       for(uint32 i=0; i<0x10000; i++)
+       for(uint32_t i=0; i<0x10000; i++)
        {
-               uint32 cyan = (i & 0xF000) >> 12,
+               uint32_t cyan = (i & 0xF000) >> 12,
                        red = (i & 0x0F00) >> 8,
                        intensity = (i & 0x00FF);
 
-               uint32 r = (((uint32)redcv[cyan][red]) * intensity) >> 8,
-                       g = (((uint32)greencv[cyan][red]) * intensity) >> 8,
-                       b = (((uint32)bluecv[cyan][red]) * intensity) >> 8;
+               uint32_t r = (((uint32_t)redcv[cyan][red]) * intensity) >> 8,
+                       g = (((uint32_t)greencv[cyan][red]) * intensity) >> 8,
+                       b = (((uint32_t)bluecv[cyan][red]) * intensity) >> 8;
 
-//hm.          CRY16ToRGB32[i] = 0xFF000000 | (b << 16) | (g << 8) | r;
                CRY16ToRGB32[i] = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
                MIX16ToRGB32[i] = (i & 0x01 ? RGB16ToRGB32[i] : CRY16ToRGB32[i]);
        }
 }
 
+
 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)
+
+uint8_t * TOMGetRamPointer(void)
 {
        return tomRam8;
 }
 
-uint8 TOMGetVideoMode(void)
+
+uint8_t TOMGetVideoMode(void)
 {
-       uint16 vmode = GET16(tomRam8, VMODE);
+       uint16_t 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)
+uint16_t TOMGetVDB(void)
 {
        return GET16(tomRam8, VDB);
 }
 
+
+uint16_t TOMGetHC(void)
+{
+       return GET16(tomRam8, HC);
+}
+
+
+uint16_t TOMGetVP(void)
+{
+       return GET16(tomRam8, VP);
+}
+
+
+uint16_t TOMGetMEMCON1(void)
+{
+       return GET16(tomRam8, MEMCON1);
+}
+
+
+#define LEFT_BG_FIX
 //
 // 16 BPP CRY/RGB mixed mode rendering
 //
-void tom_render_16bpp_cry_rgb_mix_scanline(uint32 * backbuffer)
+void tom_render_16bpp_cry_rgb_mix_scanline(uint32_t * backbuffer)
 {
 //CHANGED TO 32BPP RENDERING
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        //New stuff--restrict our drawing...
-       uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        //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);
+       int16_t 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_t & 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_t g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32_t pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16_t i=0; i<startPos; i++)
+                       *backbuffer++ = pixel;
+
+               width -= startPos;
+       }
+#else
                backbuffer += 2 * startPos, width -= startPos;
+#endif
 
        while (width)
        {
-               uint16 color = (*current_line_buffer++) << 8;
+               uint16_t color = (*current_line_buffer++) << 8;
                color |= *current_line_buffer++;
                *backbuffer++ = MIX16ToRGB32[color];
                width--;
        }
 }
 
+
 //
 // 16 BPP CRY mode rendering
 //
-void tom_render_16bpp_cry_scanline(uint32 * backbuffer)
+void tom_render_16bpp_cry_scanline(uint32_t * backbuffer)
 {
 //CHANGED TO 32BPP RENDERING
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        //New stuff--restrict our drawing...
-       uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        //NOTE: May have to check HDB2 as well!
-       int16 startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);// Get start position in HC ticks
+       int16_t startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);// Get start position in HC ticks
        startPos /= pwidth;
+
        if (startPos < 0)
                current_line_buffer += 2 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8_t g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32_t pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16_t 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)
        {
-               uint16 color = (*current_line_buffer++) << 8;
+               uint16_t color = (*current_line_buffer++) << 8;
                color |= *current_line_buffer++;
                *backbuffer++ = CRY16ToRGB32[color];
                width--;
        }
 }
 
+
 //
 // 24 BPP mode rendering
 //
-void tom_render_24bpp_scanline(uint32 * backbuffer)
+void tom_render_24bpp_scanline(uint32_t * backbuffer)
 {
 //CHANGED TO 32BPP RENDERING
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        //New stuff--restrict our drawing...
-       uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        //NOTE: May have to check HDB2 as well!
-       int16 startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL); // Get start position in HC ticks
+       int16_t startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);       // Get start position in HC ticks
        startPos /= pwidth;
+
        if (startPos < 0)
                current_line_buffer += 4 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8_t g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32_t pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16_t 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)
        {
-               uint32 g = *current_line_buffer++;
-               uint32 r = *current_line_buffer++;
+               uint32_t g = *current_line_buffer++;
+               uint32_t r = *current_line_buffer++;
                current_line_buffer++;
-               uint32 b = *current_line_buffer++;
-//hm.          *backbuffer++ = 0xFF000000 | (b << 16) | (g << 8) | r;
+               uint32_t b = *current_line_buffer++;
                *backbuffer++ = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
                width--;
        }
 }
 
-//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!
+
+// 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!
 //
 // 16 BPP direct mode rendering
 //
-void tom_render_16bpp_direct_scanline(uint32 * backbuffer)
+void tom_render_16bpp_direct_scanline(uint32_t * backbuffer)
 {
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        while (width)
        {
-               uint16 color = (*current_line_buffer++) << 8;
+               uint16_t color = (*current_line_buffer++) << 8;
                color |= *current_line_buffer++;
                *backbuffer++ = color >> 1;
                width--;
        }
 }
 
+
 //
 // 16 BPP RGB mode rendering
 //
-void tom_render_16bpp_rgb_scanline(uint32 * backbuffer)
+void tom_render_16bpp_rgb_scanline(uint32_t * backbuffer)
 {
 //CHANGED TO 32BPP RENDERING
        // 16 BPP RGB: 0-5 green, 6-10 blue, 11-15 red
 
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        //New stuff--restrict our drawing...
-       uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        //NOTE: May have to check HDB2 as well!
-       int16 startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL); // Get start position in HC ticks
+       int16_t startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);       // Get start position in HC ticks
        startPos /= pwidth;
 
        if (startPos < 0)
                current_line_buffer += 2 * -startPos;
        else
+#ifdef LEFT_BG_FIX
+       {
+               uint8_t g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+               uint32_t pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+
+               for(int16_t 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)
        {
-               uint32 color = (*current_line_buffer++) << 8;
+               uint32_t color = (*current_line_buffer++) << 8;
                color |= *current_line_buffer++;
                *backbuffer++ = RGB16ToRGB32[color];
                width--;
@@ -778,43 +863,36 @@ 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)
+// Process a single halfline
 //
-void TOMExecHalfline(uint16 halfline, bool render)
+void TOMExecHalfline(uint16_t halfline, bool render)
 {
-#warning "!!! Need to handle multiple fields properly !!!"
-       // We ignore the problem for now
-       halfline &= 0x7FF;
-
+       uint16_t field2 = halfline & 0x0800;
+       halfline &= 0x07FF;
        bool inActiveDisplayArea = true;
 
-//Interlacing is still not handled correctly here... !!! FIX !!!
-       if (halfline & 0x01)                                                    // Execute OP only on even halflines (non-interlaced only!)
+       // Execute OP only on even halflines (skip higher resolutions for now...)
+       if (halfline & 0x01)
                return;
 
 //Hm, it seems that the OP needs to execute from zero, so let's try it:
-// And it works! But need to do some optimizations in the OP to keep it from attempting
-// to do a scanline render in the non-display area... [DONE]
+// And it works! But need to do some optimizations in the OP to keep it from
+// attempting to do a scanline render in the non-display area... [DONE]
 //this seems to cause a regression in certain games, like rayman
 //which means I have to dig thru the asic nets to see what's wrong...
 /*
-No, the OP doesn't start until VDB, that much is certain. The thing is, VDB is the
-HALF line that the OP starts on--which means that it needs to start at VDB / 2!!!
-
-Hrm, doesn't seem to be enough, though it should be... still sticks for 20 frames.
+No, the OP doesn't start until VDB, that much is certain. The thing is, VDB is
+the HALF line that the OP starts on--which means that it needs to start at
+VDB / 2!!!
 
+Hrm, doesn't seem to be enough, though it should be... still sticks for 20
+frames.
 
-What triggers this is writing $FFFF to VDE. This causes the OP start signal in VID to 
-latch on, which in effect sets VDB to zero. So that much is correct. But the thing with
-Rayman is that it shouldn't cause the graphical glitches seen there, so still have to
-investigate what's going on there. By all rights, it shouldn't glitch because:
+What triggers this is writing $FFFF to VDE. This causes the OP start signal in VID to latch on, which in effect sets VDB to zero. So that much is correct. But
+the thing with Rayman is that it shouldn't cause the graphical glitches seen
+there, so still have to investigate what's going on there. By all rights, it
+shouldn't glitch because:
 
 00006C00: 0000000D 82008F73 (BRANCH) YPOS=494, CC=">", link=$00006C10
 00006C08: 000003FF 00008173 (BRANCH) YPOS=46, CC=">", link=$001FF800
@@ -822,8 +900,8 @@ investigate what's going on there. By all rights, it shouldn't glitch because:
 001FF800: 12FC2BFF 02380000 (BITMAP)
           00008004 8180CFF1
 
-Even if the OP is running all the time, the link should tell it to stop at the right
-place (which it seems to do). But we still get glitchy screen.
+Even if the OP is running all the time, the link should tell it to stop at the
+right place (which it seems to do). But we still get glitchy screen.
 
 Seems the glitchy screen went away... Maybe the GPU alignment fixes fixed it???
 Just need to add the proper checking here then.
@@ -851,33 +929,28 @@ TOM: Vertical Display Begin written by M68K: 41
 TOM: Vertical Display End written by M68K: 2047
 TOM: Vertical Interrupt written by M68K: 491
 */
-#if 1
+
        // Initial values that "well behaved" programs use
-       uint16 startingHalfline = GET16(tomRam8, VDB);
-       uint16 endingHalfline = GET16(tomRam8, VDE);
+       uint16_t startingHalfline = GET16(tomRam8, VDB);
+       uint16_t endingHalfline = GET16(tomRam8, VDE);
 
        // Simulate the OP start bug here!
        // Really, this value is somewhere around 507 for an NTSC Jaguar. But this
-       // should work in a majority of cases, at least until we can figure it out properly.
+       // should work in a majority of cases, at least until we can figure it out
+       // properly.
        if (endingHalfline > GET16(tomRam8, VP))
                startingHalfline = 0;
 
-       if (halfline >= startingHalfline && halfline < endingHalfline)
-//     if (halfline >= 0 && halfline < (uint16)GET16(tomRam8, VDE))
-// 16 isn't enough, and neither is 32 for raptgun. 32 fucks up Rayman
-//     if (halfline >= ((uint16)GET16(tomRam8, VDB) / 2) && halfline < ((uint16)GET16(tomRam8, VDE) / 2))
-//     if (halfline >= ((uint16)GET16(tomRam8, VDB) - 16) && halfline < (uint16)GET16(tomRam8, VDE))
-//     if (halfline >= 20 && halfline < (uint16)GET16(tomRam8, VDE))
-//     if (halfline >= (uint16)GET16(tomRam8, VDB) && halfline < (uint16)GET16(tomRam8, VDE))
+       if ((halfline >= startingHalfline) && (halfline < endingHalfline))
        {
                if (render)
                {
-                       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
-                       uint8 bgHI = tomRam8[BG], bgLO = tomRam8[BG + 1];
+                       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
+                       uint8_t bgHI = tomRam8[BG], bgLO = tomRam8[BG + 1];
 
                        // Clear line buffer with BG
                        if (GET16(tomRam8, VMODE) & BGEN) // && (CRY or RGB16)...
-                               for(uint32 i=0; i<720; i++)
+                               for(uint32_t i=0; i<720; i++)
                                        *current_line_buffer++ = bgHI, *current_line_buffer++ = bgLO;
 
                        OPProcessList(halfline, render);
@@ -885,49 +958,33 @@ TOM: Vertical Interrupt written by M68K: 491
        }
        else
                inActiveDisplayArea = false;
-#else
-       inActiveDisplayArea =
-               (halfline >= (uint16)GET16(tomRam8, VDB) && halfline < (uint16)GET16(tomRam8, VDE)
-                       ? true : false);
 
-       if (halfline < (uint16)GET16(tomRam8, VDE))
-       {
-               if (render)//With JaguarExecuteNew() this is always true...
-               {
-                       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
-                       uint8 bgHI = tomRam8[BG], bgLO = tomRam8[BG + 1];
+       // Take PAL into account...
 
-                       // Clear line buffer with BG
-                       if (GET16(tomRam8, VMODE) & BGEN) // && (CRY or RGB16)...
-                               for(uint32 i=0; i<720; i++)
-                                       *current_line_buffer++ = bgHI, *current_line_buffer++ = bgLO;
-
-//                     OPProcessList(halfline, render);
-//This seems to take care of it...
-                       OPProcessList(halfline, inActiveDisplayArea);
-               }
-       }
-#endif
-
-       // Try to take PAL into account... [We do now!]
-
-       uint16 topVisible = (vjs.hardwareTypeNTSC ? TOP_VISIBLE_VC : TOP_VISIBLE_VC_PAL),
+       uint16_t topVisible = (vjs.hardwareTypeNTSC ? TOP_VISIBLE_VC : TOP_VISIBLE_VC_PAL),
                bottomVisible = (vjs.hardwareTypeNTSC ? BOTTOM_VISIBLE_VC : BOTTOM_VISIBLE_VC_PAL);
-       uint32 * TOMCurrentLine = &(screenBuffer[((halfline - topVisible) / 2) * screenPitch]);
+       uint32_t * TOMCurrentLine = 0;
+
+       // Bit 0 in VP is interlace flag. 0 = interlace, 1 = non-interlaced
+       if (tomRam8[VP + 1] & 0x01)
+               TOMCurrentLine = &(screenBuffer[((halfline - topVisible) / 2) * screenPitch]);//non-interlace
+       else
+               TOMCurrentLine = &(screenBuffer[(((halfline - topVisible) / 2) * screenPitch * 2) + (field2 ? 0 : screenPitch)]);//interlace
 
        // Here's our virtualized scanline code...
 
-       if (halfline >= topVisible && halfline < bottomVisible)
+       if ((halfline >= topVisible) && (halfline < bottomVisible))
        {
                if (inActiveDisplayArea)
                {
-//NOTE: The following doesn't put BORDER color on the sides... !!! FIX !!!
 #warning "The following doesn't put BORDER color on the sides... !!! FIX !!!"
                        if (vjs.renderType == RT_NORMAL)
-//                             scanline_render[TOMGetVideoMode()](TOMBackbuffer);
+                       {
                                scanline_render[TOMGetVideoMode()](TOMCurrentLine);
-                       else//TV type render
+                       }
+                       else
                        {
+                               // TV type render
 /*
        tom_render_16bpp_cry_scanline,
        tom_render_24bpp_scanline,
@@ -941,30 +998,31 @@ TOM: Vertical Interrupt written by M68K: 491
 #define   MODE         0x0006          // Line buffer to video generator mode
 #define   VARMOD       0x0100          // Mixed CRY/RGB16 mode (only works in MODE 0!)
 */
-                               uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
-                               uint8 mode = ((GET16(tomRam8, VMODE) & MODE) >> 1);
+                               uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+                               uint8_t mode = ((GET16(tomRam8, VMODE) & MODE) >> 1);
                                bool varmod = GET16(tomRam8, VMODE) & VARMOD;
-//The video texture line buffer ranges from 0 to 1279, with its left edge starting at
-//LEFT_VISIBLE_HC. So, we need to start writing into the backbuffer at HDB1, using pwidth
-//as our scaling factor. The way it generates its image on a real TV!
-
-//So, for example, if HDB1 is less than LEFT_VISIBLE_HC, then we have to figure out where
-//in the VTLB that we start writing pixels from the Jaguar line buffer (VTLB start=0,
-//JLB=something).
+//The video texture line buffer ranges from 0 to 1279, with its left edge
+//starting at LEFT_VISIBLE_HC. So, we need to start writing into the backbuffer
+//at HDB1, using pwidth as our scaling factor. The way it generates its image
+//on a real TV!
+
+//So, for example, if HDB1 is less than LEFT_VISIBLE_HC, then we have to figure
+//out where in the VTLB that we start writing pixels from the Jaguar line
+//buffer (VTLB start=0, JLB=something).
 #if 0
 //
 // 24 BPP mode rendering
 //
-void tom_render_24bpp_scanline(uint32 * backbuffer)
+void tom_render_24bpp_scanline(uint32_t * backbuffer)
 {
 //CHANGED TO 32BPP RENDERING
-       uint16 width = tomWidth;
-       uint8 * current_line_buffer = (uint8 *)&tomRam8[0x1800];
+       uint16_t width = tomWidth;
+       uint8_t * current_line_buffer = (uint8_t *)&tomRam8[0x1800];
 
        //New stuff--restrict our drawing...
-       uint8 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       uint8_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        //NOTE: May have to check HDB2 as well!
-       int16 startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL); // Get start position in HC ticks
+       int16_t startPos = GET16(tomRam8, HDB1) - (vjs.hardwareTypeNTSC ? LEFT_VISIBLE_HC : LEFT_VISIBLE_HC_PAL);       // Get start position in HC ticks
        startPos /= pwidth;
        if (startPos < 0)
                current_line_buffer += 4 * -startPos;
@@ -974,10 +1032,10 @@ void tom_render_24bpp_scanline(uint32 * backbuffer)
 
        while (width)
        {
-               uint32 g = *current_line_buffer++;
-               uint32 r = *current_line_buffer++;
+               uint32_t g = *current_line_buffer++;
+               uint32_t r = *current_line_buffer++;
                current_line_buffer++;
-               uint32 b = *current_line_buffer++;
+               uint32_t b = *current_line_buffer++;
                *backbuffer++ = 0xFF000000 | (b << 16) | (g << 8) | r;
                width--;
        }
@@ -989,17 +1047,18 @@ void tom_render_24bpp_scanline(uint32 * backbuffer)
                else
                {
                        // If outside of VDB & VDE, then display the border color
-                       uint32 * currentLineBuffer = TOMCurrentLine;
-                       uint8 g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
-//Hm.                  uint32 pixel = 0xFF000000 | (b << 16) | (g << 8) | r;
-                       uint32 pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
+                       uint32_t * currentLineBuffer = TOMCurrentLine;
+                       uint8_t g = tomRam8[BORD1], r = tomRam8[BORD1 + 1], b = tomRam8[BORD2 + 1];
+//Hm.                  uint32_t pixel = 0xFF000000 | (b << 16) | (g << 8) | r;
+                       uint32_t pixel = 0x000000FF | (r << 24) | (g << 16) | (b << 8);
 
-                       for(uint32 i=0; i<tomWidth; i++)
+                       for(uint32_t i=0; i<tomWidth; i++)
                                *currentLineBuffer++ = pixel;
                }
        }
 }
 
+
 //
 // TOM initialization
 //
@@ -1011,29 +1070,21 @@ void TOMInit(void)
        TOMReset();
 }
 
+
 void TOMDone(void)
 {
+       TOMDumpIORegistersToLog();
        OPDone();
        BlitterDone();
-       WriteLog("TOM: Resolution %i x %i %s\n", TOMGetVideoModeWidth(), TOMGetVideoModeHeight(),
-               videoMode_to_str[TOMGetVideoMode()]);
-//     WriteLog("\ntom: object processor:\n");
-//     WriteLog("tom: pointer to object list: 0x%.8x\n",op_get_list_pointer());
-//     WriteLog("tom: INT1=0x%.2x%.2x\n",TOMReadByte(0xf000e0),TOMReadByte(0xf000e1));
-//     gpu_done();
-//     dsp_done();
-//     memory_free(tomRam8);
-//     memory_free(tom_cry_rgb_mix_lut);
+       WriteLog("TOM: Resolution %i x %i %s\n", TOMGetVideoModeWidth(),
+               TOMGetVideoModeHeight(), videoMode_to_str[TOMGetVideoMode()]);
 }
 
-uint32 TOMGetVideoModeWidth(void)
-{
-       //These widths are pretty bogus. Should use HDB1/2 & HDE/HBB & PWIDTH to calc the width...
-//     uint32 width[8] = { 1330, 665, 443, 332, 266, 222, 190, 166 };
-//Temporary, for testing Doom...
-//     uint32 width[8] = { 1330, 665, 443, 332, 266, 222, 190, 332 };
 
-       // Note that the following PWIDTH values have the following pixel aspect ratios:
+uint32_t TOMGetVideoModeWidth(void)
+{
+       // Note that the following PWIDTH values have the following pixel aspect
+       // ratios:
        // PWIDTH = 1 -> 0.25:1 (1:4) pixels (X:Y ratio)
        // PWIDTH = 2 -> 0.50:1 (1:2) pixels
        // PWIDTH = 3 -> 0.75:1 (3:4) pixels
@@ -1043,82 +1094,25 @@ uint32 TOMGetVideoModeWidth(void)
        // PWIDTH = 7 -> 1.75:1 (7:4) pixels
        // PWIDTH = 8 -> 2.00:1 (2:1) pixels
 
-       // Also note that the JTRM says that PWIDTH of 4 gives pixels that are "about" square--
-       // this implies that the other modes have pixels that are *not* square!
+       // Also note that the JTRM says that PWIDTH of 4 gives pixels that are
+       // "about" square--this implies that the other modes have pixels that are
+       // *not* square (and they aren't)!
        // Also, I seriously doubt that you will see any games that use PWIDTH = 1!
 
-       // NOTE: Even though the PWIDTH value is + 1, here we're using a zero-based index and
-       //       so we don't bother to add one...
-//     return width[(GET16(tomRam8, VMODE) & PWIDTH) >> 9];
-
-       // Now, we just calculate it...
-/*     uint16 hdb1 = GET16(tomRam8, HDB1), hde = GET16(tomRam8, HDE),
-               hbb = GET16(tomRam8, HBB), pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
-//     return ((hbb < hde ? hbb : hde) - hdb1) / pwidth;
-//Temporary, for testing Doom...
-       return ((hbb < hde ? hbb : hde) - hdb1) / (pwidth == 8 ? 4 : pwidth);*/
-
-       // To make it easier to make a quasi-fixed display size, we restrict the viewing
-       // area to an arbitrary range of the Horizontal Count.
-       uint16 pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
+       // To make it easier to make a quasi-fixed display size, we restrict the
+       // viewing area to an arbitrary range of the Horizontal Count.
+       uint16_t pwidth = ((GET16(tomRam8, VMODE) & PWIDTH) >> 9) + 1;
        return (vjs.hardwareTypeNTSC ? RIGHT_VISIBLE_HC - LEFT_VISIBLE_HC : RIGHT_VISIBLE_HC_PAL - LEFT_VISIBLE_HC_PAL) / pwidth;
-//Temporary, for testing Doom...
-//     return (RIGHT_VISIBLE_HC - LEFT_VISIBLE_HC) / (pwidth == 8 ? 4 : pwidth);
-////   return (RIGHT_VISIBLE_HC - LEFT_VISIBLE_HC) / (pwidth == 4 ? 8 : pwidth);
-
-// More speculating...
-// According to the JTRM, the number of potential pixels across is given by the
-// Horizontal Period (HP - in NTSC this is 845). The Horizontal Count counts from
-// zero to this value twice per scanline (the high bit is set on the second count).
-// HBE and HBB define the absolute "black" limits of the screen, while HDB1/2 and
-// HDE determine the extent of the OP "on" time. I.e., when the OP is turned on by
-// HDB1, it starts fetching the line from position 0 in LBUF.
-
-// The trick, it would seem, is to figure out how long the typical visible scanline
-// of a TV is in HP ticks and limit the visible area to that (divided by PWIDTH, of
-// course). Using that length, we can establish an "absolute left display limit" with
-// which to measure HBB & HDB1/2 against when rendering LBUF (i.e., if HDB1 is 20 ticks
-// to the right of the ALDL and PWIDTH is 4, then start writing the LBUF starting at
-// backbuffer + 5 pixels).
-
-// 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...
-// Same with the width... [Width is pretty much virtualized now.]
 
-// Now that that the width is virtualized, let's virtualize the height. :-)
-uint32 TOMGetVideoModeHeight(void)
+uint32_t TOMGetVideoModeHeight(void)
 {
-//     uint16 vmode = GET16(tomRam8, VMODE);
-//     uint16 vbe = GET16(tomRam8, VBE);
-//     uint16 vbb = GET16(tomRam8, VBB);
-//     uint16 vdb = GET16(tomRam8, VDB);
-//     uint16 vde = GET16(tomRam8, VDE);
-//     uint16 vp = GET16(tomRam8, VP);
-
-/*     if (vde == 0xFFFF)
-               vde = vbb;//*/
-
-//     return 227;//WAS:(vde/*-vdb*/) >> 1;
-       // The video mode height probably works this way:
-       // VC counts from 0 to VP. VDB starts the OP. Either when
-       // VDE is reached or VP, the OP is stopped. Let's try it...
-       // Also note that we're conveniently ignoring interlaced display modes...!
-//     return ((vde > vp ? vp : vde) - vdb) >> 1;
-//     return ((vde > vbb ? vbb : vde) - vdb) >> 1;
-//Let's try from the Vertical Blank interval...
-//Seems to work OK!
-//     return (vbb - vbe) >> 1;        // Again, doesn't take interlacing into account...
-// This of course doesn't take interlacing into account. But I haven't seen any
-// Jaguar software that takes advantage of it either...
-//Also, doesn't reflect PAL Jaguar either... !!! FIX !!! [DONE]
-//     return 240;                                                                             // Set virtual screen height to 240 lines...
+       // Set virtual screen height to 240 (NTSC) or 256 (PAL) lines...
        return (vjs.hardwareTypeNTSC ? 240 : 256);
 }
 
+
 //
 // TOM reset code
 // Now PAL friendly!
@@ -1176,6 +1170,7 @@ void TOMReset(void)
        if (vjs.hardwareTypeNTSC)
        {
                SET16(tomRam8, MEMCON1, 0x1861);
+//             SET16(tomRam8, MEMCON1, 0x1865);//Bunch of BS
                SET16(tomRam8, MEMCON2, 0x35CC);
                SET16(tomRam8, HP, 844);                        // Horizontal Period (1-based; HP=845)
                SET16(tomRam8, HBB, 1713);                      // Horizontal Blank Begin
@@ -1222,10 +1217,55 @@ void TOMReset(void)
        tomTimerCounter = 0;
 }
 
+
+//
+// Dump all TOM register values to the log
+//
+void TOMDumpIORegistersToLog(void)
+{
+       WriteLog("\n\n---------------------------------------------------------------------\n");
+       WriteLog("TOM I/O Registers\n");
+       WriteLog("---------------------------------------------------------------------\n");
+       WriteLog("F000%02X (MEMCON1): $%04X\n", MEMCON1, GET16(tomRam8, MEMCON1));
+       WriteLog("F000%02X (MEMCON2): $%04X\n", MEMCON2, GET16(tomRam8, MEMCON2));
+       WriteLog("F000%02X      (HC): $%04X\n", HC,      GET16(tomRam8, HC));
+       WriteLog("F000%02X      (VC): $%04X\n", VC,      GET16(tomRam8, VC));
+       WriteLog("F000%02X     (OLP): $%08X\n", OLP,     GET32(tomRam8, OLP));
+       WriteLog("F000%02X     (OBF): $%04X\n", OBF,     GET16(tomRam8, OBF));
+       WriteLog("F000%02X   (VMODE): $%04X\n", VMODE,   GET16(tomRam8, VMODE));
+       WriteLog("F000%02X   (BORD1): $%04X\n", BORD1,   GET16(tomRam8, BORD1));
+       WriteLog("F000%02X   (BORD2): $%04X\n", BORD2,   GET16(tomRam8, BORD2));
+       WriteLog("F000%02X      (HP): $%04X\n", HP,      GET16(tomRam8, HP));
+       WriteLog("F000%02X     (HBB): $%04X\n", HBB,     GET16(tomRam8, HBB));
+       WriteLog("F000%02X     (HBE): $%04X\n", HBE,     GET16(tomRam8, HBE));
+       WriteLog("F000%02X      (HS): $%04X\n", HS,      GET16(tomRam8, HS));
+       WriteLog("F000%02X     (HVS): $%04X\n", HVS,     GET16(tomRam8, HVS));
+       WriteLog("F000%02X    (HDB1): $%04X\n", HDB1,    GET16(tomRam8, HDB1));
+       WriteLog("F000%02X    (HDB2): $%04X\n", HDB2,    GET16(tomRam8, HDB2));
+       WriteLog("F000%02X     (HDE): $%04X\n", HDE,     GET16(tomRam8, HDE));
+       WriteLog("F000%02X      (VP): $%04X\n", VP,      GET16(tomRam8, VP));
+       WriteLog("F000%02X     (VBB): $%04X\n", VBB,     GET16(tomRam8, VBB));
+       WriteLog("F000%02X     (VBE): $%04X\n", VBE,     GET16(tomRam8, VBE));
+       WriteLog("F000%02X      (VS): $%04X\n", VS,      GET16(tomRam8, VS));
+       WriteLog("F000%02X     (VDB): $%04X\n", VDB,     GET16(tomRam8, VDB));
+       WriteLog("F000%02X     (VDE): $%04X\n", VDE,     GET16(tomRam8, VDE));
+       WriteLog("F000%02X     (VEB): $%04X\n", VEB,     GET16(tomRam8, VEB));
+       WriteLog("F000%02X     (VEE): $%04X\n", VEE,     GET16(tomRam8, VEE));
+       WriteLog("F000%02X      (VI): $%04X\n", VI,      GET16(tomRam8, VI));
+       WriteLog("F000%02X    (PIT0): $%04X\n", PIT0,    GET16(tomRam8, PIT0));
+       WriteLog("F000%02X    (PIT1): $%04X\n", PIT1,    GET16(tomRam8, PIT1));
+       WriteLog("F000%02X     (HEQ): $%04X\n", HEQ,     GET16(tomRam8, HEQ));
+       WriteLog("F000%02X      (BG): $%04X\n", BG,      GET16(tomRam8, BG));
+       WriteLog("F000%02X    (INT1): $%04X\n", INT1,    GET16(tomRam8, INT1));
+       WriteLog("F000%02X    (INT2): $%04X\n", INT2,    GET16(tomRam8, INT2));
+       WriteLog("---------------------------------------------------------------------\n\n\n");
+}
+
+
 //
 // TOM byte access (read)
 //
-uint8 TOMReadByte(uint32 offset, uint32 who/*=UNKNOWN*/)
+uint8_t TOMReadByte(uint32_t offset, uint32_t who/*=UNKNOWN*/)
 {
 //???Is this needed???
 // It seems so. Perhaps it's the +$8000 offset being written to (32-bit interface)?
@@ -1257,10 +1297,11 @@ uint8 TOMReadByte(uint32 offset, uint32 who/*=UNKNOWN*/)
        return tomRam8[offset & 0x3FFF];
 }
 
+
 //
 // TOM word access (read)
 //
-uint16 TOMReadWord(uint32 offset, uint32 who/*=UNKNOWN*/)
+uint16_t TOMReadWord(uint32_t offset, uint32_t who/*=UNKNOWN*/)
 {
 //???Is this needed???
 //     offset &= 0xFF3FFF;
@@ -1268,12 +1309,12 @@ 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)
        {
                // For reading, should only return the lower 5 bits...
-               uint16 data = (tom_jerry_int_pending << 4) | (tom_timer_int_pending << 3)
+               uint16_t data = (tom_jerry_int_pending << 4) | (tom_timer_int_pending << 3)
                        | (tom_object_int_pending << 2) | (tom_gpu_int_pending << 1)
                        | (tom_video_int_pending << 0);
                //WriteLog("tom: interrupt status is 0x%.4x \n",data);
@@ -1314,12 +1355,16 @@ if (offset >= 0xF02000 && offset <= 0xF020FF)
        return (TOMReadByte(offset, who) << 8) | TOMReadByte(offset + 1, who);
 }
 
+
 #define TOM_STRICT_MEMORY_ACCESS
 //
 // TOM byte access (write)
 //
-void TOMWriteByte(uint32 offset, uint8 data, uint32 who/*=UNKNOWN*/)
+void TOMWriteByte(uint32_t offset, uint8_t data, uint32_t who/*=UNKNOWN*/)
 {
+       // Moved here tentatively, so we can see everything written to TOM.
+       tomRam8[offset & 0x3FFF] = data;
+
 #ifdef TOM_DEBUG
        WriteLog("TOM: Writing byte %02X at %06X", data, offset);
 #endif
@@ -1394,14 +1439,19 @@ void TOMWriteByte(uint32 offset, uint8 data, uint32 who/*=UNKNOWN*/)
                tomRam8[offset] = data, tomRam8[offset + 0x200] = data;
        }
 
-       tomRam8[offset & 0x3FFF] = data;
+//     tomRam8[offset & 0x3FFF] = data;
 }
 
+
 //
 // TOM word access (write)
 //
-void TOMWriteWord(uint32 offset, uint16 data, uint32 who/*=UNKNOWN*/)
+void TOMWriteWord(uint32_t offset, uint16_t data, uint32_t who/*=UNKNOWN*/)
 {
+       // Moved here tentatively, so we can see everything written to TOM.
+       tomRam8[(offset + 0) & 0x3FFF] = data >> 8;
+       tomRam8[(offset + 1) & 0x3FFF] = data & 0xFF;
+
 #ifdef TOM_DEBUG
        WriteLog("TOM: Writing byte %04X at %06X", data, offset);
 #endif
@@ -1429,7 +1479,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))
        {
@@ -1507,8 +1557,8 @@ if (offset >= 0xF02000 && offset <= 0xF020FF)
                data &= 0x03FF;                 // These are all 10-bit registers
 
 // Fix a lockup bug... :-P
-       TOMWriteByte(0xF00000 | offset, data >> 8, who);
-       TOMWriteByte(0xF00000 | (offset+1), data & 0xFF, who);
+//     TOMWriteByte(0xF00000 | offset, data >> 8, who);
+//     TOMWriteByte(0xF00000 | (offset+1), data & 0xFF, who);
 
 if (offset == MEMCON1)
        WriteLog("TOM: Memory Config 1 written by %s: $%04X\n", whoName[who], data);
@@ -1521,7 +1571,7 @@ if (offset == MEMCON2)
 //if (offset == OBF)
 //     WriteLog("TOM: Object Processor Flag written by %s: %u\n", whoName[who], data);
 if (offset == VMODE)
-       WriteLog("TOM: Video Mode written by %s: %04X. PWIDTH = %u, MODE = %s, flags:%s%s (VC = %u)\n", whoName[who], data, ((data >> 9) & 0x07) + 1, videoMode_to_str[(data & MODE) >> 1], (data & BGEN ? " BGEN" : ""), (data & VARMOD ? " VARMOD" : ""), GET16(tomRam8, VC));
+       WriteLog("TOM: Video Mode written by %s: %04X. PWIDTH = %u, MODE = %s, flags:%s%s (VC = %u) (M68K PC = %06X)\n", whoName[who], data, ((data >> 9) & 0x07) + 1, videoMode_to_str[(data & MODE) >> 1], (data & BGEN ? " BGEN" : ""), (data & VARMOD ? " VARMOD" : ""), GET16(tomRam8, VC), m68k_get_reg(NULL, M68K_REG_PC));
 if (offset == BORD1)
        WriteLog("TOM: Border 1 written by %s: $%04X\n", whoName[who], data);
 if (offset == BORD2)
@@ -1577,11 +1627,12 @@ if (offset == HEQ)
 // TOM Shouldn't be mucking around with this, it's up to the host system to properly
 // handle this kind of crap.
 // NOTE: This is needed somehow, need to get rid of the dependency on this crap.
+//       N.B.: It's used in the rendering functions... So...
 #warning "!!! Need to get rid of this dependency !!!"
 #if 1
        if ((offset >= 0x28) && (offset <= 0x4F))
        {
-               uint32 width = TOMGetVideoModeWidth(), height = TOMGetVideoModeHeight();
+               uint32_t width = TOMGetVideoModeWidth(), height = TOMGetVideoModeHeight();
 
                if ((width != tomWidth) || (height != tomHeight))
                {
@@ -1596,6 +1647,7 @@ if (offset == HEQ)
 #endif
 }
 
+
 int TOMIRQEnabled(int irq)
 {
        // This is the correct byte in big endian... D'oh!
@@ -1603,6 +1655,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 +1663,7 @@ int TOMIRQEnabled(int irq)
 
 void TOMPITCallback(void);
 
+
 void TOMResetPIT(void)
 {
 #ifndef NEW_TIMER_SYSTEM
@@ -1632,13 +1686,14 @@ void TOMResetPIT(void)
 #endif
 }
 
+
 //
 // TOM Programmable Interrupt Timer handler
 // NOTE: TOM's PIT is only enabled if the prescaler is != 0
 //
 //NOTE: This is only used by the old execution code... Safe to remove
 //      once the timer system is stable.
-void TOMExecPIT(uint32 cycles)
+void TOMExecPIT(uint32_t cycles)
 {
        if (tomTimerPrescaler)
        {
@@ -1657,6 +1712,7 @@ void TOMExecPIT(uint32 cycles)
        }
 }
 
+
 void TOMPITCallback(void)
 {
 //     INT1_RREG |= 0x08;                                                      // Set TOM PIT interrupt pending
@@ -1669,3 +1725,4 @@ void TOMPITCallback(void)
 
        TOMResetPIT();
 }
+