No code changes, just to get everything in sync. #86
Conversation
This file contains hidden or 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
Fold main back into dev.
Catch up Dev merges
- Do not clear button bit flags after warm_sleep_cycle(). - Move RX_IRFaces() to run before button state is reset.
Fix button click detection after sleep.
Closes #100. More info there.
Clean up `blinklib.h` comments. Remove unneeded tools in package file.
Oh man that took so many hours. The IDE was somehow using lines from the JSON installed platform.txt and mixing them in with the manually installed one. What a mess. Solution was to completely remove the JSON one and restart and then it got very confused and forgot everything... but then used everything from the correct platform.txt. Yuck. Seems to work and program right, we still need to fix the BlinkBIOS code to know where the active and built-in games are.
added 30 commits
May 16, 2021 10:54
...even when the CRC errorMode(3) line is changed. Hmmmm...
…ame address as the one on 168 The BIOS makes a duplicate vector table at 0x3800 on the 328 so that there is always a table here om both parts (on the 168 the table at 0x3800 is the real one inside the bootloader). This change makes games compiled for the 328 target use this fake table so that if there ever get run on a 168 (because of IR download) then they will still find a table at this address.
Still need to update BIOS to be able to share big games to make this actually useful.
Still debug code in MAX just in case, we will get rid of it soon.
It seems that the linker sometimes overflows .text prematurely. We give it a little extra room here so that it does not bail. This will make a few games that are right on the edge be able to compile and they do end up being the right size in the end. For those that actually end up being too large to fix, we will catch those with the final size check in the Arduino IDE build process (uses `upload.maximum_size` in `boards.txt`)
You really need the `.data` section or any program that has initialized data in it (nost!) will not work because the data will not be inside the binary image. Note that you still do need to *remove* the `.bss` section since that really should not be in there (it is not even the correct kind of memory since bss in in SRAM), so change the command line to *exclude* bss rather than *include* data since if we only include `.text` (yep, the command is "include *only*" so you can not explicitly include multiple sections. the we do not get the `.data` section. So we just got lucky that there is only one section we need to exclude.
Add latest blinkbios that supports working NFC .
Includes all the new NFC stuff as well can corrected flash size for MAX blinks.
Shows a hack to get brighter RED or GREEN.
Stupid Arduino.
Clean up.
Bring `dev` in line with with `main` to prepare for merge. Just updates the board manager URL.
Merge `dev` to prepare for Arduino board manager release
To save lots of space.
Would rather figure out what is really work, but can no reproduce on my platform so I guess try this?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.