New working software list items

-------------------------------
videopac: Service Test Cartridge (Europe, plus) [anonymous]

hash/videopac.xml

@@ -18,7 +18,6 @@ TODO:
 - add serials for radiola and jopacs
 
 Undumped (or dumped but not public):
-- G7400 service test cartridge
 - misc. homebrew cartridges
 
 Protos:
@@ -2108,7 +2107,7 @@ distinguished by the changed copyright from Magnavox to N.A.P. (North American P
 	</software>
 
 	<software name="testcart">
-		<!-- not a consumer product, factory test rom -->
+		<!-- not a consumer product, factory test cartridge -->
 		<description>Service Test Cartridge (Europe)</description>
 		<year>198?</year>
 		<publisher>Philips</publisher>
@@ -2550,6 +2549,21 @@ distinguished by the changed copyright from Magnavox to N.A.P. (North American P
 		</part>
 	</software>
 
+	<software name="testcartpl" cloneof="testcart" supported="partial">
+		<!-- not a consumer product, factory test cartridge -->
+		<description>Service Test Cartridge (Europe, plus)</description>
+		<year>1983</year>
+		<publisher>Philips</publisher>
+		<info name="usage" value="See testcart, and at end of normal tests, press both joystick buttons to continue." />
+		<part name="cart" interface="odyssey_cart">
+			<feature name="slot" value="o2_rally" />
+			<feature name="b_pin" value="1" />
+			<dataarea name="rom" size="0x2000">
+				<rom name="testcartpl.bin" size="0x2000" crc="b72b1494" sha1="fb281a5e4c7664209106f94e463d9017751712b5" />
+			</dataarea>
+		</part>
+	</software>
+
 	<software name="stonepl" cloneof="stone">
 		<description>Stone Sling (Europe, plus)</description>
 		<year>198?</year>
@@ -2564,7 +2578,7 @@ distinguished by the changed copyright from Magnavox to N.A.P. (North American P
 	</software>
 
 	<software name="syracusepl" cloneof="syracuse">
-		<description>Syracuse (France, plus, prototype)</description>
+		<description>Syracuse+ (France, plus, prototype)</description>
 		<year>198?</year>
 		<publisher>Brandt</publisher>
 		<part name="cart" interface="odyssey_cart">
@@ -2642,7 +2656,7 @@ distinguished by the changed copyright from Magnavox to N.A.P. (North American P
 		<description>Norseman (Europe)</description>
 		<year>1983</year>
 		<publisher>GST Video</publisher>
-		<info name="programmer" value="Andy Eltisand Jake Dowding" />
+		<info name="programmer" value="Andy Eltisand, Jake Dowding" />
 		<info name="serial" value="57" />
 		<part name="cart" interface="odyssey_cart">
 			<feature name="slot" value="o2_rom" />

src/devices/video/i8244.cpp

@@ -433,7 +433,7 @@ void i8244_device::write_cx(int x, bool cx)
 {
 	if (cx)
 	{
-		u8 colx = m_collision_map[x] & 0x3f;
+		u8 colx = m_collision_map[x];
 
 		// check if we collide with an already drawn source object
 		if (colx)
@@ -451,9 +451,7 @@ void i8244_device::write_cx(int x, bool cx)
 
 		// check if an already drawn object would collide with us
 		if (m_vdc.s.collision & 0x40)
-		{
 			m_collision_status |= colx;
-		}
 	}
 }
 

src/mame/philips/odyssey2.cpp

@@ -55,10 +55,15 @@ TODO:
 - backgamm doesn't draw all the chars/sprites, it does multiple screen updates
   and writes to the ptr/color registers, but does not increment the Y regs.
   Does it (ab)use an undocumented 8245 feature?
-- g7400 helicopt sometimes locks up at the sea level, timing or IRQ related?
+- G7400 helicopt sometimes locks up at the sea level, timing or IRQ related?
+- testcartpl is French instead of English on G7400. It's due to inaccurate mcs48
+  timer emulation. It sets up the timer to trigger an IRQ exactly 32 cycles later,
+  but MAME is 2 cycles (or 1 opcode) too early, and a language check fails. It
+  uses the internal timer, not the T1 timer, so it works reliably on the real
+  console. Set bp 41a in the debugger to see.
 - spaans has a keyboard debounce issue: if you push and hold a key after the game
   revealed the answer, it will be entered in the next input field. It's a prototype
-  so it wouldn't be surprising there are bugs, but this issue does not happen on
+  so it wouldn't be surprising if there are bugs, but this issue does not happen on
   a real Videopac or Odyssey 2. It's probably video timing related.
 - screen resolution is not strictly defined, height(243) is correct, but
   horizontal overscan differs depending on monitor/tv? see syracuse for overscan
@@ -78,8 +83,8 @@ TODO:
     to inaccurate PAL video timing. The game does mid-scanline video updates.
   * gtwallst turns the display on too soon, the middle scroller is partially
     visible when it's not supposed to (also a bit glitchy on NTSC but not as bad)
-- g7400 probably has different video timing too (not same as g7000)
+- G7400 probably has different video timing too (not same as G7000)
-- 4in1 and musician are not supposed to work on g7400, but work fine on MAME,
+- 4in1 and musician are not supposed to work on G7400, but work fine on MAME,
   caused by bus conflict or because they write to P2?
 - according to tests, 8244 does not have a sound interrupt, but the Philips
   service test cartridge for 8245 tests for it and fails if it did not get an irq
@@ -101,7 +106,7 @@ BTANB:
 - a lot of PAL games have problems on NTSC (the other way around, not so much)
   * most-common cause is due to shorter vblank, less time to prepare frame
   * characters are not rendered near upper border on 8244 (eg. tutank, chezmxme)
-- g7400 games don't look correct on odyssey3 and vice versa: ef934x graphics are
+- G7400 games don't look correct on odyssey3 and vice versa: ef934x graphics are
   placed lower on odyssey3
 - Blackjack (Videopac 5) does not work on G7400, caused by a removed BIOS routine
 - due to different XTAL ratio on Jopac JO7400, some games that do mid-screen video