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

Evel Knievel GBC Game - distorted characters issue #22

Closed
ViiMaster opened this issue Sep 4, 2020 · 2 comments
Closed

Evel Knievel GBC Game - distorted characters issue #22

ViiMaster opened this issue Sep 4, 2020 · 2 comments

Comments

@ViiMaster
Copy link

ViiMaster commented Sep 4, 2020

Hi,
First of all, thank you so much for your hard work! Now on the issue I've encountered...
On the game: Evel Knievel for the GBC which is backwards compatible, I've noticed that as soon as I reach the level selection screen, the characters appear distorted and there are general character issues with the background... I'm not sure if it is an acual hardware bug of SGB or not though... Gameplay wise the game appears fine!

@redacted173
Copy link
Owner

This game is using the OAM stat interrupt to avoid writing VRAM state when the PPU is actively reading it. But the interrupt was missing from line 0 which caused some of the VRAM tile map writes to be dropped. Should be fixed in the following commit:

1dfbe87

I tested some other problematic games to make sure they didn't break, however I don't have time to test anything else. It's also a relatively minor bug which doesn't warrant a new release on its own. If you'd like to try the fix you can copy the bit file contents of the attached zip file over a sgb07 install.

ek_missing_oam_stat_int.zip

@ViiMaster
Copy link
Author

ViiMaster commented Sep 6, 2020

Thank you very much for attending the issue! Of course I'll apply the fix since you went through the trouble of finding and correcting it! :) I understand a seperate release is not necessary for now since the core is solid as it is but it's nice that a small issue like that was corrected! Also thank you for the explanation you provided on what the problem was...

Edit:
Just tested it with the patch you've provided! Works great now! Thanks 👍 🥇 :)

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

No branches or pull requests

2 participants