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

Update assembly examples for VERA v0.9 #115

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LRFLEW
Copy link
Contributor

@LRFLEW LRFLEW commented Apr 11, 2021

A little over a year ago, the VERA protocol changed. In particular, the bank $F was removed and all of its registers were moved either to the 6502's address space or the rail end of bank $1. This updates the code in /assembly to use the new layout, allowing it to work with both r38 and the master branch of the emulator.

Since more 6502 IO addresses were added, I had to update vera.inc to include the new addresses. The documentation in the x16-docs repo has names for them, but has them in all uppercase, as opposed to the existing all lowercase names. I opted to replace all the variables with all uppercase names prefixed with VERA_ to keep things consistent with the documentation. EDIT: I realized all these variables were defined in the ROM code here, so I copy-pasted that instead. There are some minor differences (eg VERA_ADDRx_L becomes VERA_ADDR_L), but this should keep things more consistent.

I think we should also rename the files in here. The names "mode 4" and "mode 7" refer to features of the VERA v0.8 specification. VERA used to have 8 video modes that defined the bit depth, tilemap/bitmap mode, and 16 or 256 color text. Now VERA separates the three into separate configuration values. I renamed the demos to "tilemap" and "bitmap" demo, as they better reflect what they are demos of in the new protocol.

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