01794: tm5k: Coin inputs incorrectly mapped

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
01794	DIP/Input	Minor	Always	May 10, 2008, 02:56	May 23, 2008, 03:29

Tester	Luigi30	View Status	Public	Platform	MAME (Official Binary)
Assigned To	Luigi30	Resolution	Fixed	OS	Windows XP/Vista 32-bit
Status [?]	Resolved			Driver	midway/tmaster.cpp
Version	0.125	Fixed in Version	0.125u2	Build
		Fixed in Git Commit		Github Pull Request #

Summary	01794: tm5k: Coin inputs incorrectly mapped
Description	tm5k should have two coin inputs and a bill validator, but they're both mapped to Coin 1 and there's no validator input.
Steps To Reproduce
Additional Information	whatsnew.txt mentions this fix in 0.125u1, but fix is still not implemented.
Github Commit

Flags
Regression Version
Affected Sets / Systems	tm5k

Attached Files

No.00934 Tafoid Administrator May 10, 2008, 14:18	The TODO list in the driver itself mentions "Coin optics" are not implemented yet. This includes the coins and DBV (dollar bill validator). Looking around, too, the DBV was not always included in the base unit and often needed to be purchased as an upgrade. I'll leave acknowledged for now - pending Dev input.
No.00959 Luigi30 Tester May 15, 2008, 13:22	I fixed this in u1
No.00960 Fujix Administrator May 15, 2008, 13:45	Thanks, updated.
No.00971 Luigi30 Tester May 15, 2008, 19:38	No, not really. The whatsnew says "Fixed coin inputs, added validator input, and added notes on inputs to the tmaster driver." but they yanked all of it out. This should be unresolved since nothing was actually fixed.
No.01059 Luigi30 Tester May 23, 2008, 03:13	And they actually added the fix in u2.