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

minor changes to pademu #228

Merged
merged 2 commits into from Oct 18, 2019
Merged

minor changes to pademu #228

merged 2 commits into from Oct 18, 2019

Conversation

belek666
Copy link
Contributor

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK
  • Requires update of the gsKit
  • Others (please specify below)

Pull Request description

Some small corrections and code clean-ups.

@J013k
Copy link
Contributor

J013k commented Oct 11, 2019

Any test version before merging it?

Update opl from main repo
@belek666
Copy link
Contributor Author

belek666 commented Oct 12, 2019

OPNPS2LD.zip
Commit doesn't change much but ds4 should be less delayed now.

@KrahJohlito
Copy link
Member

@J013k
You already tested this build yea?
I did a quick test and it seemed all good, no regression and changes do what are intended.

@J013k
Copy link
Contributor

J013k commented Oct 16, 2019

No, I don't have any additional regression with DS3.

Unfortunately I don't have DS4, so I cannot test recent updates to notice if delay has been reduced.

@carl0sjt
Copy link

carl0sjt commented Oct 16, 2019

I've tested with Dualshock 4. Controller is well responsive. Seems okay with most games. Some issues still remains:

  • controller vibrates on cutscenes, when game is paused (resident evil 4)
  • Real PS2 controller still vibrates when using PAD emu even when PADEMU is set to port 1/2 and REAL controller is connected to the port that correspond virtual pad. eg. If PADEMU port 1 vibrates, the real controller connect to controller port 1 will vibrate too.
  • Midnight Club 3 crashes at boot.
  • Its confusing to pair the controller. Some times it connects to other devices instead of PS2.

No regressions on this PR. :)

@KrahJohlito
Copy link
Member

Since this PR does as intended without breaking anything, can this be merged please?

It’s not about fixing all current issues with PADEMU but I’m sure it’s still helpful to see the current remaining issues.

Best regards

@ElPatas1 ElPatas1 merged commit 4ee8fcd into ps2homebrew:master Oct 18, 2019
@ElPatas1
Copy link
Contributor

Hello,

merged, although it would be nice to fix the remaining issues.

Best regards.

@KrahJohlito
Copy link
Member

Yes I agree, sorry to seem impatient we just needed the changes merged before submitting more changes for approval.

Thank you

AKuHAK pushed a commit that referenced this pull request Sep 30, 2021
citronalco pushed a commit to citronalco/OPL-Daily-Builds that referenced this pull request Sep 10, 2023
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

6 participants