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

PCRTC/Counters: Fix H-Blanks per frame #10199

Merged
merged 1 commit into from
Oct 29, 2023
Merged

Conversation

refractionpcsx2
Copy link
Member

Description of Changes

Fixes the H-Blanks per frame according to testing.
Also fix a bug in checking if it's NTSC

Rationale behind Changes

The numbers were slightly out, I retested them and they match pretty closely now.

Suggested Testing Steps

Smoke test some games, maybe try some games with incorrect field orders, might be better now..

cycles values are bus cycles/256


Progressive NTSC

PCSX2
bus cycles during vblank 842 vrender 8785 vblank 842
blanks during vblank 23 vrender 240 vblank 23

PS2
bus cycles during vblank 842 vrender 8785 vblank 841
blanks during vblank 23 vrender 240 vblank 23


Progressive PAL

PCSX2
bus cycles during vblank 959 vrender 10617 vblank 958
blanks during vblank 26 vrender 288 vblank 26

PS2
bus cycles during vblank 958 vrender 10616 vblank 958
blanks during vblank 26 vrender 288 vblank 26


Interlaced NTSC

PCSX2
bus cycles during vblank 824 vrender 8786 vblank 823
blanks during vblank 23 vrender 240 vblank 22

PS2
bus cycles during vblank 823 vrender 8785 vblank 823
blanks during vblank 23 vrender 240 vblank 22


Interlaced PAL

PCSX2
bus cycles during vblank 903 vrender 10617 vblank 903
blanks during vblank 25 vrender 288 vblank 24

PS2
bus cycles during vblank 903 vrender 10616 vblank 903
blanks during vblank 25 vrender 288 vblank 24

@refractionpcsx2 refractionpcsx2 merged commit 3850227 into master Oct 29, 2023
22 checks passed
@refractionpcsx2 refractionpcsx2 deleted the pcrtc_hblanksperframe branch October 29, 2023 22:20
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.

4 participants