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

Fox32 architecture documentation #2

Merged
merged 3 commits into from
Dec 31, 2022
Merged

Fox32 architecture documentation #2

merged 3 commits into from
Dec 31, 2022

Conversation

neuschaefer
Copy link
Contributor

Hi everyone,

I've spend some time documenting the Fox32 CPU based on how it's handed in the emulator. It's still a bit incomplete (I haven't written descriptions for the individual instructions), but I think it's a start.

The already existing encodings.md file is now a bit stranded, as I didn't know best to integrate it into the new document.

Feel free to merge just part of this PR, but I wanted to get it out for review now.

With docs/cpu.md, I aim to describe the CPU architecture as fully as
necessary to allow understanding it without having to look into emulator
source code anymore.

This currently leaves encoding.md a bit stranded, because I didn't quire
know how to integrate it into this new document.
@ry755
Copy link
Contributor

ry755 commented Dec 31, 2022

Woah this is incredibly detailed even though it's currently unfinished, thank you so much for doing this!! I'll merge the entire PR, I think every little bit of documentation helps. Really the only thing I would change in the future is the fact that "fox32" should be all lowercase, it's just a style consistency thing :P

@ry755 ry755 merged commit a1d1e34 into fox32-arch:main Dec 31, 2022
neuschaefer added a commit to neuschaefer/fox32 that referenced this pull request Jan 1, 2023
@neuschaefer neuschaefer deleted the doc branch January 1, 2023 02:34
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.

None yet

2 participants