Systems promoted to working

---------------------------
Grand Master (Milton Bradley) [hap, Berger]

Author: happppp

src/devices/machine/gmboard.cpp

@@ -19,7 +19,7 @@ Hardware notes:
 
 Concept/design by Milton Bradley, for use in the Grand Master. Fidelity licensed
 or bought the design, and applied it nearly unchanged to Fidelity Phantom. Years
-later, the ex chief engineer of by then defunct Fidelity used the same technology
+later, the ex chief engineer of Fidelity (by then defunct) used the same technology
 while working for Excalibur.
 
 TODO:
@@ -99,8 +99,6 @@ void gmboard_device::device_start()
 void gmboard_device::device_reset()
 {
 	memset(m_motor_drift, 0, sizeof(m_motor_drift));
-	m_magnet = 0;
-
 	output_magnet_pos();
 }
 
@@ -207,7 +205,7 @@ void gmboard_device::magnet_w(int state)
 		return;
 	m_magnet = state;
 
-	if (m_piece_hand != 0)
+	if (m_piece_hand)
 		realign_magnet_pos();
 	output_magnet_pos();
 
@@ -229,8 +227,6 @@ void gmboard_device::magnet_w(int state)
 
 	if (state)
 	{
-		bool found = false;
-
 		if (valid_pos)
 		{
 			// pick up piece, unless it was picked up by the user
@@ -239,16 +235,15 @@ void gmboard_device::magnet_w(int state)
 			{
 				m_piece_hand = m_board->read_piece(x, y);
 
-				if (m_piece_hand != 0)
+				if (m_piece_hand)
 				{
-					found = true;
 					m_board->write_piece(x, y, 0);
 					m_board->refresh();
 				}
 			}
 		}
 
-		if (!found)
+		if (!m_piece_hand)
 		{
 			int count = 0;
 
@@ -273,6 +268,7 @@ void gmboard_device::magnet_w(int state)
 	{
 		LOGMASKED(LOG_MAGNET, "%s piece %2d @ %2d,%2d (%2d,%2d)\n", state ? "grab" : "drop", m_piece_hand, x, y, gx, gy);
 
+		// drop piece
 		if (!state)
 		{
 			if (valid_pos)

src/mame/fidelity/phantom.cpp

@@ -318,7 +318,7 @@ u8 phantom_state::motors_r(offs_t offset)
 {
 	u8 data = 0x3f;
 
-	// IRQ F/F output
+	// IRQ F/F Q
 	if (offset)
 	{
 		data |= m_irq_ff[1]->output_r() << 6;

src/mame/layout/fidel_cphantom.lay

@@ -8,7 +8,6 @@ authors:Sandro Ronco, hap
 <!-- define elements -->
 
 	<element name="blackb"><rect><color red="0" green="0" blue="0" /></rect></element>
-	<element name="greenb"><rect><color red="0.1" green="0.8" blue="0.35" /></rect></element>
 	<element name="blackbd"><disk><color red="0" green="0" blue="0" /></disk></element>
 	<element name="whited"><disk><color red="0.99" green="0.99" blue="0.99" /></disk></element>
 	<element name="lcd_bg"><rect><color red="0.54" green="0.57" blue="0.58" /></rect></element>
@@ -490,6 +489,17 @@ authors:Sandro Ronco, hap
 		</rect>
 	</element>
 
+	<element name="outline3">
+		<rect>
+			<bounds xc="0" yc="0" width="6" height="4.5" />
+			<color red="0.1" green="0.8" blue="0.35" />
+		</rect>
+		<rect>
+			<bounds xc="0" yc="0" width="5.6" height="4.1" />
+			<color red="0" green="0" blue="0" />
+		</rect>
+	</element>
+
 	<element name="text_u1"><text string="VERIFY"><color red="0.01" green="0.01" blue="0.01" /></text></element>
 	<element name="text_u2"><text string="OPTION"><color red="0.01" green="0.01" blue="0.01" /></text></element>
 	<element name="text_u3"><text string="LEVEL"><color red="0.01" green="0.01" blue="0.01" /></text></element>
@@ -533,9 +543,9 @@ authors:Sandro Ronco, hap
 	<group name="panel">
 		<bounds x="-3" y="0" width="125" height="17" />
 
-		<element ref="greenb"><bounds x="51" y="4" width="6" height="4.5" /></element>
-		<element ref="blackb"><bounds x="51.5" y="4.5" width="5" height="3.5" /></element>
+		<element ref="outline3"><bounds x="51" y="4" width="6" height="4.5" /></element>
 		<element name="0.8" ref="led"><bounds x="53.2" y="5.5" width="1.5" height="1.5" /></element>
+
 		<element ref="cwhite"><bounds x="51" y="9.5" width="6" height="4.5" /></element>
 		<element ref="text_shift"><bounds x="51" y="10.9" width="6" height="1.5" /></element>
 		<element ref="hlr" inputtag="IN.0" inputmask="0x100"><bounds x="51" y="9.5" width="6" height="4.5" /><color alpha="0.3" /></element>

src/mame/layout/fidel_phantom.lay

@@ -8,7 +8,6 @@ authors:Sandro Ronco, hap
 <!-- define elements -->
 
 	<element name="blackb"><rect><color red="0" green="0" blue="0" /></rect></element>
-	<element name="redb"><rect><color red="0.81" green="0.3" blue="0.29" /></rect></element>
 	<element name="blackbd"><disk><color red="0" green="0" blue="0" /></disk></element>
 	<element name="whited"><disk><color red="0.99" green="0.99" blue="0.99" /></disk></element>
 	<element name="lcd_bg"><rect><color red="0.54" green="0.57" blue="0.58" /></rect></element>
@@ -490,6 +489,17 @@ authors:Sandro Ronco, hap
 		</rect>
 	</element>
 
+	<element name="outline3">
+		<rect>
+			<bounds xc="0" yc="0" width="6" height="4.5" />
+			<color red="0.81" green="0.3" blue="0.29" />
+		</rect>
+		<rect>
+			<bounds xc="0" yc="0" width="5.6" height="4.1" />
+			<color red="0" green="0" blue="0" />
+		</rect>
+	</element>
+
 	<element name="text_u1"><text string="VERIFY"><color red="0.01" green="0.01" blue="0.01" /></text></element>
 	<element name="text_u2"><text string="OPTION"><color red="0.01" green="0.01" blue="0.01" /></text></element>
 	<element name="text_u3"><text string="LEVEL"><color red="0.01" green="0.01" blue="0.01" /></text></element>
@@ -533,9 +543,9 @@ authors:Sandro Ronco, hap
 	<group name="panel">
 		<bounds x="-3" y="0" width="125" height="17" />
 
-		<element ref="redb"><bounds x="51" y="4" width="6" height="4.5" /></element>
-		<element ref="blackb"><bounds x="51.5" y="4.5" width="5" height="3.5" /></element>
+		<element ref="outline3"><bounds x="51" y="4" width="6" height="4.5" /></element>
 		<element name="0.8" ref="led"><bounds x="53.2" y="5.5" width="1.5" height="1.5" /></element>
+
 		<element ref="cwhite"><bounds x="51" y="9.5" width="6" height="4.5" /></element>
 		<element ref="text_shift"><bounds x="51" y="10.9" width="6" height="1.5" /></element>
 		<element ref="hlr" inputtag="IN.0" inputmask="0x100"><bounds x="51" y="9.5" width="6" height="4.5" /><color alpha="0.3" /></element>