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

change the order of z1013 joysticks #2115

Closed
wants to merge 1 commit into from

Conversation

Hojoe42
Copy link
Contributor

@Hojoe42 Hojoe42 commented Oct 25, 2022

This pull request changes the order of the joystick of target z1013.

Please look at the picture:
z1023Joystik-2
The input of joystick 1 goes to joystick 2.

The code at http://z1013.mrboot.de/software-database/db/5591df18d0d6145646e91d54f4d0637a-joystick_demo/index.html says 0x20 is for joystick 2 and 0x40 is for joystick 1

asm("ld a,#0x20"); //select JOYSTICK 2
...
asm("ld a,#0x40"); //select JOYSTICK 1

@suborb
Copy link
Member

suborb commented Oct 25, 2022

I'm just trying to understand how things got mixed up....

The source for the original information was here: https://hc-ddr.hucki.net/wiki/doku.php/z1013/joystick

And my copy of jkcemu source (0.9.8.2) has this in Z1013.java:

    if( (joySel & 0x20) != 0 ) {                // Joystick 1 selektiert
      actionMask |= this.joy0ActionMask;
    }
    if( (joySel & 0x40) != 0 ) {                // Joystick 2 selektiert
      actionMask |= this.joy1ActionMask;
    }

@Hojoe42
Copy link
Contributor Author

Hojoe42 commented Oct 26, 2022

I'm just trying to understand how things got mixed up....

The source for the original information was here: https://hc-ddr.hucki.net/wiki/doku.php/z1013/joystick

And my copy of jkcemu source (0.9.8.2) has this in Z1013.java:

Interesting, I will check this.

@Hojoe42
Copy link
Contributor Author

Hojoe42 commented Oct 27, 2022

I close this request. I think it's a bug in JKCEmu.

@Hojoe42 Hojoe42 closed this Oct 27, 2022
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 this pull request may close these issues.

None yet

2 participants