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

Add a note to the HDMA timings about how many tiles you can maximal transfer during VBlank. #569

Merged
merged 4 commits into from
Sep 25, 2024

Conversation

daid
Copy link
Contributor

@daid daid commented Sep 25, 2024

Add a note to the HDMA timings about how many tiles you can maximal transfer during VBlank. This is one of those numbers you can figure out by going from M-Cycles to dots and then checking the amount of dots in VBlank, but it's a common enough thing you would want to know that I think it's good to have it here.

daid added 4 commits July 13, 2021 16:20
Add note about how much tiles we can transfer during VBlank. So people do not have to figure that out from M-cycles->dots->nr of dots in vblank.
Copy link
Member

@ISSOtm ISSOtm left a comment

Choose a reason for hiding this comment

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

Yes, this is good information to point out explicitly. “Saved you a click calculation.”

Thank you, Daid!

@ISSOtm ISSOtm merged commit ccaaaf7 into gbdev:master Sep 25, 2024
2 checks passed
@daid
Copy link
Contributor Author

daid commented Sep 25, 2024

I figured it's a easy leap from this to "oh, it's easy to transfer 128 tiles in a single vblank".

Note that in theory you could transfer a bit more, if you use the hblank of the last line and the OAM scan of the first line as well you can transfer about 10 tiles more. But getting that timing right would be quite complicated. And with this, I figured most people would go "let's just transfer 128 tiles per vblank"

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.

2 participants