Open
Conversation
niaow reviewed Apr 6, 2020
aykevl reviewed Apr 6, 2020
Member
| Note: I have changed the PR from master to dev, as that's what we're working on (and dev has many changes now not included in master). |
iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 6, 2020
ee8de80 to 781dc40 Compare iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 6, 2020
aykevl reviewed Apr 12, 2020
Member
| There are a number of conflicts, which you can fix by rebasing on top of the dev branch (although it appears like you already did some of that in #1048?) |
iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 12, 2020
iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 12, 2020
24f4ad6 to 345c90a Compare Author
| @aykevl It appears that the problem with picolib is affecting circle-ci as well. Check the errors in this build. This is the same trouble I had on my mac. |
iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 13, 2020
iansmith added a commit to iansmith/tinygo that referenced this pull request Apr 13, 2020
345c90a to ef5548e Compare This commit changes several files related to the raspberry pi. It assumes that if you want to change the configuration of the linkerscript or other parameters, you will override the one provided with your own local .json file. (Thanks ayke.) There is a sample program to use with these changes that can added to tinygo-zoo once these changes are merged and deployed.
ef5548e to db25b40 Compare Author
| @aykevl @jaddr2line I implemented all the suggestions from Ayke on overriding and machine generation of the hardware MMIO registers. Close the other PRs that were confusing the issue. |
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
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.
@aykevl @jaddr2line
This PR is to confirm that I have the files setup "right" to be a good citizen in tinygo.
The test program (note this does UART synchronously, not yet to the point of interrupts):
If you want to test locally:
Assuming you have Qemu 4.2 you can run with: