05550: coco,cocoe, coco2, coco2b: Emulation will not start. "mc6847 not found"

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
05550	Core	Critical (emulation)	Always	Apr 26, 2014, 13:14	Apr 26, 2014, 16:56

Tester	Robert Gault	View Status	Public	Platform	MESS (Self-compiled)
Assigned To		Resolution	Invalid report	OS	Windows XP
Status [?]	Closed			Driver	trs/coco12.cpp
Version	N/A	Fixed in Version		Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	05550: coco, cocoe, coco2, coco2b: Emulation will not start. "mc6847 not found"
Description	This is related to but not the same as bug 5541 for 0.153b. For less than one day after 5541 was fixed, the above emulation ran correctly. As of svn30011, MESS reports, for example: mess coco2 mc6847 NOT FOUND (tried in coco2 coco mc6847_ntsc) FATALERROR: Required files are missing, the system cannot be run. This error is generated for coco, cocoe, coco2, and coco2b. As far as I can tell the video hardware initialization in mess/drivers/coco12.c is correct. // video hardware MCFG_SCREEN_MC6847_NTSC_ADD(SCREEN_TAG, VDG_TAG) MCFG_MC6847_ADD(VDG_TAG, MC6847_NTSC, XTAL_3_579545MHz, coco12_state::mc6847_config) MCFG_MC6847_HSYNC_CALLBACK(WRITELINE(coco12_state, horizontal_sync)) MCFG_MC6847_FSYNC_CALLBACK(WRITELINE(coco12_state, field_sync)) Likewise this is correctly inherited by the other versions and changed correctly. static MACHINE_CONFIG_DERIVED( coco2b, coco2 ) MCFG_MC6847_REMOVE(VDG_TAG) MCFG_MC6847_ADD(VDG_TAG, MC6847T1_NTSC, XTAL_3_579545MHz, coco12_state::mc6847_config) MCFG_MC6847_HSYNC_CALLBACK(WRITELINE(coco12_state, horizontal_sync)) MCFG_MC6847_FSYNC_CALLBACK(WRITELINE(coco12_state, field_sync)) MACHINE_CONFIG_END I don't see any mismatches with the defines in emu/video/mc6847.h but don't pretend to understand the MESS source code.
Steps To Reproduce
Additional Information
Github Commit

Flags
Regression Version
Affected Sets / Systems	coco,cocoe, coco2, coco2b

Attached Files

No.10629 Tafoid Administrator Apr 26, 2014, 16:56	This is a report for an SVN version. MameTesters is only for logging and fixing of release versions bugs. Anything discovered outside of this should be discussed in a proper forum (mameworld, emuversal) or at #messdev. The error is correct. There was a romset added for the chargen ROM for the coco2. Your sets will need this additional piece to function. https://github.com/startaq/mame/commit/281b78400195a5ac345b1b2848664116e78fabe1 This is not a bug nor, if it were a bug, does it qualify to be reported here. Closing.

No.10629

Tafoid

Administrator

Apr 26, 2014, 16:56

This is a report for an SVN version.
MameTesters is only for logging and fixing of release versions bugs. Anything discovered outside of this should be discussed in a proper forum (mameworld, emuversal) or at #messdev.

The error is correct. There was a romset added for the chargen ROM for the coco2. Your sets will need this additional piece to function.
https://github.com/startaq/mame/commit/281b78400195a5ac345b1b2848664116e78fabe1

This is not a bug nor, if it were a bug, does it qualify to be reported here.
Closing.