03653: crzrally, crzrallya, crzrallyg: Default high scores are incorrect

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
03653	Graphics	Minor	Always	Jan 9, 2010, 12:19	Jan 10, 2010, 20:38

Tester	Kold666	View Status	Public	Platform	MAME (Official Binary)
Assigned To	Reip	Resolution	Fixed	OS
Status [?]	Resolved			Driver	tecfri/holeland.cpp
Version	0.136	Fixed in Version	0.136u1	Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	03653: crzrally, crzrallya, crzrallyg: Default high scores are incorrect
Description	Mame displays the default hiscore all 0 but on the original pcb you can see names and points. Note the this pcb unlike the Tricky doc one which had the same problem doesn't have battery backed ram.
Steps To Reproduce
Additional Information
Github Commit

Flags	Verified with Original
Regression Version
Affected Sets / Systems	crzrally, crzrallya, crzrallyg

Attached Files	Immag083.jpg (317,052 bytes) Jan 9, 2010, 12:19

No.05470 Tafoid Administrator Jan 9, 2010, 12:58	Your picture appears to be from the "crzrallyg" set. I can confirm with that set assuming the others behave the same (have the same score list). It makes sense there generally are default scores to break.
No.05473 etabeta Developer Jan 10, 2010, 12:53	Fixed by Reip
No.05474 Haze Senior Tester Jan 10, 2010, 13:31	Reip's fix is actually wrong according to what's been posted here "Note the this pcb unlike the Tricky doc one which had the same problem doesn't have battery backed ram" He added NVRAM for it, but kold says the PCB has none (unless the battery is just missing on his PCB?). It clearly needs the RAM initializing to FF for the highscore table to appear correct tho.
No.05477 Kold666 Developer Jan 10, 2010, 20:09	I just looked at the pcb and it should have had the battery but no one ever soldered it! it's very difficult to spot but after Haze comment I looked carefully and there is a little circuit and a "BAT" mark! it's on the upper left of the pcb near the 2k ram! http://www.citylan.it/wiki/images/a/ad/119_PCB_component_side.jpg
No.05478 Haze Senior Tester Jan 10, 2010, 20:38 edited on: Jan 10, 2010, 20:40	Ok, the fix is correct then, although I'm unconvinced that the only option MAME offers to initialize RAM is if you specify it at battery backup as it's feasible that some other badly programmed games may assume RAM to default to either 0x00 or 0xff. I believe by default MAME assumes all RAM to be 0x00, but in this case (due to the fact it works on the real PCB without battery) the game code is correctly assuming that ram is initialized to 0xff on powerup.