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

identify_accessory() identifies controller pak as rumble pak #536

Open
fhoedemakers opened this issue May 4, 2024 · 6 comments
Open

identify_accessory() identifies controller pak as rumble pak #536

fhoedemakers opened this issue May 4, 2024 · 6 comments
Labels
fixed-in-preview Fixed in the preview branch

Comments

@fhoedemakers
Copy link

fhoedemakers commented May 4, 2024

WIth controller pak inserted in controller 0:

When trying cpak-utest and cpaktest from the examples, the Identify_accessory(0) call returns ACCESSORY_RUMBLEPAK

Tried with different controllers and controller paks on an NTSC Nintendo 64 with Everdrive 64 X7

@fhoedemakers fhoedemakers changed the title identify_accessory() identifies controller pak as rumble pack identify_accessory() identifies controller pak as rumble pak May 4, 2024
@networkfusion
Copy link
Contributor

Is it a first party pak or a third party pak? If third party, what model?

@fhoedemakers
Copy link
Author

First party NUS-004. Official games recognize the pak without any problem. (Doom 64, Turok)

@networkfusion
Copy link
Contributor

networkfusion commented May 5, 2024

It this point I am guessing you are using the trunk branch. I am pretty sure this is fixed on the preview and unstable branches which implement the new joybus API.

@fhoedemakers
Copy link
Author

Ok thanks, will try the preview branch.

@fhoedemakers
Copy link
Author

Hi, I tested with the preview branch and now the controller pak is recognized without a problem. Thanks!

@rasky rasky added the fixed-in-preview Fixed in the preview branch label May 6, 2024
@rasky
Copy link
Collaborator

rasky commented May 6, 2024

Thanks I'll mark this as fixed in preview, so that others can find it. I'll close the issue once the new library is merged to trunk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-preview Fixed in the preview branch
Projects
None yet
Development

No branches or pull requests

3 participants