-
Notifications
You must be signed in to change notification settings - Fork 941
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
WIP: for feedback from tinygo folks #1027
base: dev
Are you sure you want to change the base?
Conversation
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). |
ee8de80
to
781dc40
Compare
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?) |
24f4ad6
to
345c90a
Compare
@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. |
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
@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. |
@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: