-
Notifications
You must be signed in to change notification settings - Fork 17
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
Some improvements to the reverse engineered source and improved firmware #2
Open
dmantione
wants to merge
45
commits into
mist64:master
Choose a base branch
from
dmantione:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…properly comment the freezer code.
…with the bank 3 source code.
…code to disk. I now view part of it as junk that, while it calls into the actual directory code, is junk that should be viewed as unused space. * Move the directory write-back code to a new file desktop_helper2.s * Better commenting of directory write-back code. * Change some absolute adresses in screenshot.s to symbols
+ Add symbol for KERNAL PRT routine * _load_ae_rom_hidden add underscore in name to indicate it is inside persistent block
* Improve symbolization and comments of freezer menu
…ery code). + Finalization of source code for bank 3, the code is complete and all of bank 3 can now be built from source.
* Tweak label names
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello Michael,
Based on your work on the FC3 reverse engineering, I have created an improved firmware for the FC3:
http://www.freepascal.org/~daniel/fc3/fc3101percent.crt
http://www.freepascal.org/~daniel/fc3/changes.txt
I am developing in this tree:
https://github.com/dmantione/final_cartridge/tree/development
While doing the work, I did some general improvements to the reverse engineered source (i.e. better commenting and labeling code) that I would like to merge back. These changes don't modify the generated code.
I have been working for a long time on this and progress was slow. I'd like to note that it is impossible to just modify bank 0 in the FC3 like suggest in the README.md. There is some hardcoded jumping between bank 0 and bank 3 going on in the firmware, so if you just modify bank 0 you will break some freezer functions. In my modified firmware, this is for now patched, but I am also working on a source code reconstruction for bank 3 (not yet in the Github tree).
Best regards,
Daniël Mantione