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

Wlan channel Fix - Ridge Racer 2 & Warriors Orochi 2 #13401

Merged
merged 3 commits into from
Sep 8, 2020

Conversation

anr2me
Copy link
Collaborator

@anr2me anr2me commented Sep 8, 2020

Based on JPCSP with prx files.
Multiplayer using automatic channel are now working properly on Warriors Orochi 2 and Ridge Racer 2 #10698

@anr2me anr2me force-pushed the wlan_channel branch 2 times, most recently from d377a78 to 55c9e48 Compare September 8, 2020 03:32
@hrydgard hrydgard merged commit b3d7864 into hrydgard:master Sep 8, 2020
@unknownbrackets unknownbrackets added this to the v1.11.0 milestone Sep 8, 2020
@unknownbrackets
Copy link
Collaborator

Wouldn't these probably depend on the PSP system setting and country?

https://en.wikipedia.org/wiki/List_of_WLAN_channels#2.4_GHz_(802.11b/g/n/ax)
(the PSP only supports 802.11b.)

My PSP had a Network Settings menu where I could chose the Ad Hoc port (but it defaulted to 'Automatic'.)

-[Unknown]

@anr2me
Copy link
Collaborator Author

anr2me commented Sep 8, 2020

The main key in this bug fix is the returned undocumented error code, regarding the channel it self i'm not sure what to be returned (probably unchanged due to error code being returned)

Regarding the automatic channel, yes, normally it's region dependent but PSP seems to only support channel 1,6, and 11 based on the constant, and as i remembered based on @adenovan test automatic channel defaulted to channel 11 internally, so i'm using it.

@anr2me anr2me deleted the wlan_channel branch September 9, 2020 06:40
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.

3 participants