Skip to content

remove bg_2 backup routine / free up ram block - #396

Open
nn357 wants to merge 1 commit into
blkerby:mainfrom
nn357:bg_2_fix
Open

nn357 wants to merge 1 commit into
blkerby:mainfrom
nn357:bg_2_fix

Conversation

@nn357

@nn357 nn357 commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

hud_expansion_opaque already fixed the DMA transfer size but didn't remove the BG2 backup/restore routine (which is only there because of the vanilla transfer size being too large and overwriting the BG2 tilemap.)

Interestingly the hudfix copies 112 tiles over ($0E00) vs what should be 'correct' vanilla 128 tiles ($1000), with a note in the .asm saying to not get bottom row of screen?

I assume this was / is for some graphical corruption? I tried it with both and couldn't see any differences but left the vanilla_hudfix patch to the DMA transfer size in place and just removed the unnecessary backup/restore of the bg2 tiles on pause/unpause.

hud_expansion_opaque already fixed the DMA transfer size but didn't remove the BG2 backup/restore routine (which is only there because of the vanilla transfer size being too large)

Interestingly the hudfix copies 112 tiles over ($0E00) vs what should be correct vanilla 128 tiles ($1000), with a note in the .asm saying to not get bottom row of screen.

I assume this was / is for some graphical corruption? I tried it with both and couldn't see any differences but left the vanilla_hudfix patch to the DMA transfer size in place and just removed the unnecessary backup/restore of the bg2 tiles on pause/unpause.
@nn357 nn357 changed the title Initial remove bg_2 backup routine / free up ram block Oct 24, 2026

This branch has not been deployed

No deployments
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.

1 participant