Jump to content
Search In
  • More options...
Find results that contain...
Find results in...
Sign in to follow this  
Robert

ARCADE 0.204

Recommended Posts

Change this:

 

map(0x1fec00, 0x1fec01).mirror(0x3fe).rw(FUNC(h6280_device::timer_r), FUNC(h6280_device::timer_w));

 

to this

 

map(0x1fec00, 0x1fec01).mirror(0x3fe).r(FUNC(h6280_device::timer_r));

 

If your tests prove conclusive, this is the change I will do.

Share this post


Link to post
Share on other sites
diff -Nru src/devices/cpu/h6280/h6280.h src-fx/devices/cpu/h6280/h6280.h
--- src/devices/cpu/h6280/h6280.h	Wed Nov 28 06:07:13 2018
+++ src-fx/devices/cpu/h6280/h6280.h	Fri Nov 30 12:10:26 2018
@@ -354,6 +354,7 @@
 
 	// internal registers
 	void internal_map(address_map &map);
+public:    // MAMEFX 2018-08-12
 	DECLARE_READ8_MEMBER( irq_status_r );
 	DECLARE_WRITE8_MEMBER( irq_status_w );
 

This is still needed ?

Share this post


Link to post
Share on other sites

Can't remember... but you'll get a compile error if it's wrong. Try what MAME has first, and if it complains, add "public:" .

Share this post


Link to post
Share on other sites

It is not need I think - I didn't have any compile errors and games using h6280 are ok. It means of course that Dark Seal And Vapor Trail have sound ok. Almost all other Data East games has been tested as well:

Act Fancer

Caveman Ninja

Boggie Wing

Battle Rangers

Crude Buster

Diet to Go

Funky Jet

Mad Motor

Rohga

Thunder Zone

 

some games in dec0 and dec32 drivers (not all are using h6280)

 

I didn't tested PC Engine Link to groovymame 64 with this modification - https://mega.nz/#!o5FBVaiJ!T0Fs7sUIe617UqHalmpiTFvW_NAY5q2PZgQjLg8mmns

Edited by haynor666

Share this post


Link to post
Share on other sites

Alright, thanks for testing.

 

I'll add the modification now, so that it will be there for 0.205

Share this post


Link to post
Share on other sites

Just did some tests on my side here, the new patch is fine. I don't see any real breakage in games that uses this CPU.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

×
×
  • Create New...