03832: monymony, jackrabt: Missing speech

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
03832	Sound	Minor	Always	May 6, 2010, 20:38	May 25, 2010, 17:54

Tester	M.A.S.H.	View Status	Public	Platform	MAME (Self-compiled)
Assigned To	couriersud	Resolution	Fixed	OS	Windows XP (32-bit)
Status [?]	Resolved			Driver	zaccaria/zaccaria.cpp
Version	0.137u4	Fixed in Version	0.138u1	Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	03832: monymony, jackrabt: Missing speech
Description	The speech after INSERT COIN is missing in Money Money and Jack Rabbit. In MAME 0.137u3 you can hear in Money Money: "You can go and look for money" and in Jack Rabbit: "Hi there, i'm Jack Rabbit... I love carrots and i'm very hungry". The TMS5200 update in 0.137u4 is the problem!
Steps To Reproduce
Additional Information
Github Commit

Flags
Regression Version	0.137u4
Affected Sets / Systems	monymony, jackrabt

Attached Files

No.06150 Lord Nightmare Developer May 21, 2010, 04:41	Known issue (see whatsnew, I even mentioned it there). Try playing 3 or 4 games in either game, the speech will start working. It is known to work properly from start on a real pcb, see &fmt=18 I'm working on the issue, which is either a PIA bug or a bug with the 5220 read timing when other operations are pending completion (in this case, probably a reset command has executed and /READY is pulled inactive for 200ms, then the game does /RS and is waiting for /READY to go active, but the reset command hasn't finished yet; however the /RS logic causes /READY to go active too early. the fix in 0.137u4 was to fix a legitamate bug: /READY went active TOO LATE after /RS was activated, which I have logic analyzer logs to prove, but that is during normal speak-external fifo writes)
No.06179 Lord Nightmare Developer May 25, 2010, 17:53	couriersud fixed this by adding a proper /READY callback to the tms52xx core and hooking it to the PIA in zaccaria.c

No.06150

Lord Nightmare

Developer

May 21, 2010, 04:41

Known issue (see whatsnew, I even mentioned it there). Try playing 3 or 4 games in either game, the speech will start working. It is known to work properly from start on a real pcb, see

&fmt=18
I'm working on the issue, which is either a PIA bug or a bug with the 5220 read timing when other operations are pending completion (in this case, probably a reset command has executed and /READY is pulled inactive for 200ms, then the game does /RS and is waiting for /READY to go active, but the reset command hasn't finished yet; however the /RS logic causes /READY to go active too early. the fix in 0.137u4 was to fix a legitamate bug: /READY went active TOO LATE after /RS was activated, which I have logic analyzer logs to prove, but that is during normal speak-external fifo writes)

No.06179

Lord Nightmare

Developer

May 25, 2010, 17:53

couriersud fixed this by adding a proper /READY callback to the tms52xx core and hooking it to the PIA in zaccaria.c