-
Notifications
You must be signed in to change notification settings - Fork 22
Change the full system emulation tutorial to use bare-metal instead of linux + busybox #16
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
Conversation
|
@stffrdhrn I cannot add reviewers for some reason, but can you review this PR when you get a chance? |
stffrdhrn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ok to me.
platform/qemu.md
Outdated
| ``` | ||
| 3. Go inside the root directory and run configure | ||
| ``` | ||
| cd qemu-9.2.2 && ./configure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to limit it to openrisc? For example: --target-list="o1rk-linux-user or1k-softmmu. Also consider others we enable here: https://github.com/stffrdhrn/or1k-utils/blob/master/qemu/config.qemu
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added target-list and softmmu for now. I didn't know about this, where can I read and learn more about these options?
|
Hi @anngineery I would like to merge this did you see my latest comments? |
|
Hey @stffrdhrn, I didn't get a chance to look at it until now. working on it! |
|
@stffrdhrn if you are happy with it, feel free to merge it |
Continuation of #14