Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cyclone and DrZ80 #765

Closed
duduke opened this issue Dec 31, 2019 · 6 comments · Fixed by #766
Closed

Cyclone and DrZ80 #765

duduke opened this issue Dec 31, 2019 · 6 comments · Fixed by #766

Comments

@duduke
Copy link

duduke commented Dec 31, 2019

Can those commits:
libretro/mame2000-libretro@ad07ac3
libretro/mame2000-libretro@1e09c86

Be ported to the mame2003-plus core? Can they be enabled for Pi0 as well?

@arcadez2003
Copy link
Collaborator

I've never heard of the Cyclone or the DrZ80, certainly unlike MAME2000 this core does not
support either of them, so i cant port the above commits into this core for obvious reasons

Regards

@frangarcj
Copy link
Contributor

I'm porting those but I'm trying to figure out how to change an already inited driver

@arcadez2003
Copy link
Collaborator

what do you mean by inited driver.??

@frangarcj
Copy link
Contributor

libretro/mame2000-libretro@1e09c86#diff-c9ef03e11cee1e52c6ef5fc5cad11184R821

I got an error of wrong number of cpus later

@frangarcj
Copy link
Contributor

static data32_t cpu_readmem24bew_dword(offs_t address)
{
  89bd88:       e24dd014        sub     sp, sp, #20
  89bd8c:       e58d0004        str     r0, [sp, #4]
        data32_t result = cpu_readmem24bew_word(address) << 16;
  89bd90:       e59d0004        ldr     r0, [sp, #4]
  89bd94:       ebe44bfb        bl      1aed88 <cpu_readmem24bew_word>
!!!             e1a03000        mov     r3, r0 !!!
  89bd9c:       e1a03803        lsl     r3, r3, #16
  89bda0:       e58d300c        str     r3, [sp, #12]
        return result | cpu_readmem24bew_word(address + 2);
  89bda4:       e59d3004        ldr     r3, [sp, #4]

Some advance!!!

@frangarcj
Copy link
Contributor

It works, I'll make a PR later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants