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

General improvements #10

Merged
merged 3 commits into from
Mar 29, 2025
Merged

General improvements #10

merged 3 commits into from
Mar 29, 2025

Conversation

AntonioND
Copy link
Contributor

@AntonioND AntonioND commented Mar 27, 2025

This PR is a bit big, but individual commits are easier to understand.

I've fixed some chapter indices that I forgot to update in the last PR (I've also renamed the folders of the code examples).

I've also fixed a data cache flush when updating the OAM buffer. It was too broad of a flush, it's better to flush just the affected range.

@AntonioND AntonioND changed the title General improvements. Additions to backgrounds chapter General improvements Mar 28, 2025
Flushing all the data cache is a very costly operation, and it isn't needed
because we're only copying the shadow OAM buffer from main RAM to OAM.
Commit 3d43ca2 ("Turn passthrough chapter into an appendix") forgot
to update chapter numbers in many places. This commit fixes that.
Copy link
Owner

@Patater Patater left a comment

Choose a reason for hiding this comment

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

LGTM

@Patater Patater merged commit 4176550 into Patater:master Mar 29, 2025
2 of 3 checks passed
@AntonioND AntonioND deleted the for-upstream branch March 29, 2025 02:50
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