02792: Missing HARDDIP settings in 'STANDARD_DIPS' for DIP's 4,5 and 6 ("COMM Setting")

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
02792	DIP/Input	Trivial	Always	Dec 31, 2008, 23:15	Jan 23, 2009, 15:50

Tester	Johnboy	View Status	Public	Platform
Assigned To		Resolution	Fixed	OS
Status [?]	Resolved			Driver	neogeo/neogeo.cpp
Version	0.128u7	Fixed in Version	0.129u3	Build
		Fixed in Git Commit		Github Pull Request #

Summary	02792: Missing HARDDIP settings in 'STANDARD_DIPS' for DIP's 4,5 and 6 ("COMM Setting")
Description	What: Missing HARDDIP settings in 'STANDARD_DIPS' for DIP's 4,5 and 6 ("COMM Setting") Possible in Mame: DIP 4 ON (COMM Setting 1) = OFF DIP 4+5 ON (COMM Setting 2) = OFF DIP 4+6 ON (COMM Setting 3) = ON [No 2] DIP 4+5+6 ON (COMM Setting 4) = ON [No 4] Missing in Mame: DIP 5+6 ON = ON [No 3] DIP 6 ON = ON [No 1] Verified on real hardware (NEO-MVH MV2F), see DIP5+6ON.JPG and DIP6ON.JPG . -------------------------------------------------------------------------------- In a service manual for a NEO-MVH MV4 there is also a setting for DIP 5 ON (see MV4.JPG) -------------------------------------------------------------------------------- Solution: ========= It should be possible to set HARDDIP's 4,5 and 6 DIP by DIP and not grouped like it is now. Attached: 1x DIP5+6ON.JPG 1x DIP6ON.JPG 1x MV4.JPG
Steps To Reproduce
Additional Information	Accepted as bug report via PM at MAMEWORLD.INFO
Github Commit

Flags
Regression Version
Affected Sets / Systems

Attached Files	DIP5+6ON.JPG (31,414 bytes) Dec 31, 2008, 23:15
	DIP6ON.JPG (28,687 bytes) Dec 31, 2008, 23:16
	MV4.JPG (37,862 bytes) Dec 31, 2008, 23:16

No.03449 Haze Senior Tester Jan 1, 2009, 12:18	ths issue is in neogeo.c PORT_DIPNAME( 0x0038, 0x0038, "COMM Setting" ) PORT_DIPLOCATION("SW:4,5,6") \ PORT_DIPSETTING( 0x0038, DEF_STR( Off ) ) \ PORT_DIPSETTING( 0x0030, "1" ) \ PORT_DIPSETTING( 0x0020, "2" ) \ PORT_DIPSETTING( 0x0010, "3" ) \ PORT_DIPSETTING( 0x0000, "4" ) \ there are no entries for 0x28, 0x18, or 0x08 defined. ALL possible setting should be defined, even if they do nothing.
No.03628 etabeta Developer Jan 23, 2009, 15:50	I've sent a patch which implements in MAME the settings from the manual (check mv4.jpg), i.e. it splits sw 4,5 from sw 6. as a result all the settings are now possible as in the real thing setting as closed since I think the change is so small it can hardly be rejected :)

No.03449

Haze

Senior Tester

Jan 1, 2009, 12:18

ths issue is in neogeo.c

PORT_DIPNAME( 0x0038, 0x0038, "COMM Setting" ) PORT_DIPLOCATION("SW:4,5,6") \
PORT_DIPSETTING( 0x0038, DEF_STR( Off ) ) \
PORT_DIPSETTING( 0x0030, "1" ) \
PORT_DIPSETTING( 0x0020, "2" ) \
PORT_DIPSETTING( 0x0010, "3" ) \
PORT_DIPSETTING( 0x0000, "4" ) \

there are no entries for 0x28, 0x18, or 0x08 defined.

ALL possible setting should be defined, even if they do nothing.

No.03628

etabeta

Developer

Jan 23, 2009, 15:50

I've sent a patch which implements in MAME the settings from the manual (check mv4.jpg), i.e. it splits sw 4,5 from sw 6. as a result all the settings are now possible as in the real thing

setting as closed since I think the change is so small it can hardly be rejected :)