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

Skylander - Change portal version #10061

Merged
merged 3 commits into from Apr 15, 2021
Merged

Skylander - Change portal version #10061

merged 3 commits into from Apr 15, 2021

Conversation

Desterly
Copy link
Contributor

@Desterly Desterly commented Apr 2, 2021

Switch emulation to the Traptanium portal. This portal is universal
and works with all games.

Reset response updated to send portal version information.

Switch emulation to the Traptanium portal.   This portal is universal
and works with all games.
@@ -272,7 +272,13 @@ void usb_device_skylander::control_transfer(u8 bmRequestType, u8 bRequest, u16 w
// Shutdowns the portal
ensure(buf_size == 2 || buf_size == 32);
q_result = {
0x52, 0x02, 0x0A, 0x03, 0x02, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
0x53, 0x00, 0x00, 0x00, 0x00, 0xc4, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0x53 packets are sent when there is a query but no pending data to get status of figures. Also you're using static values for the packet number which is definitely incorrect, see

void sky_portal::get_status(u8* reply_buf)
.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing that out. Initially it appeared the status packets were required but after further testing it appears it was just a timing issue reading the portal. The only real change needed is the version info in byte 2/3 from 0A03 to 1800.

That being said, there's a few additional query types I'm tracking down to see if they require any real response. It doesn't appear so but I'd rather be sure.

I'll update the PR tonight once I find those.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont understand this at all

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please tell me where I need to put that code if u can please beacuse my traps arent working in game when I use the emulator portal

- Remove "Status" query responses from reset request
- Add additional request types for newer portal
@saculll
Copy link

saculll commented Apr 14, 2021

so i don't really know how programming or github works, but is there some time i can expect this to be fixed, or is it already fixed??

@Megamouse Megamouse merged commit 99d6f9c into RPCS3:master Apr 15, 2021
@Healthpotion133
Copy link

So where do I place this new code?

@EvansXGames
Copy link

Please help where I am going to put that code if someone knows please help me because my traps arent working correctly in skylanders trap team @RipleyTom @Desterly

@YuriNator557
Copy link

Posting messages on a closed PR isn't going to do anything, go to the Discord server if you want help

@EvansXGames
Copy link

OK can you sent a link to the discord server @YuriNator557

@FreezingNose7
Copy link

hey is this fixed yet?

@FreezingNose7
Copy link

Switch emulation to the Traptanium portal. This portal is universal and works with all games.

Reset response updated to send portal version information.

is this fixed?? can i play trap team now without my portal refrasching ?

@Darkhost1999
Copy link
Contributor

This PR has been merged and acheives the results that it was set out to acheive.
You can load traps.
This PR does not fix any exisiting issues discovered from enabling the ability to use traps.
The issue you are discussing may have been exposed by this PR but shouldn't be caused by it. And supposedly has not been fixed yet to answer your question
All this PR does is enable the built into rpcs3 portal to be used in all skylanders games and it was successful.

@EvansXGames
Copy link

EvansXGames commented Oct 27, 2022 via email

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

Successfully merging this pull request may close these issues.

None yet

10 participants