Skip to content
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

QEMU: Add support of ARCv2 processors #208

Merged
merged 1 commit into from
May 9, 2020

Conversation

abrodkin
Copy link
Collaborator

@abrodkin abrodkin commented May 8, 2020

QEMU port for ARCv2 is almost ready and we're working on opening all our work for starters via Synopsys GitHub repo https://github.com/foss-for-synopsys-dwc-arc-processors/qemu and later as a part of upstream project.

For now we submit changes for Zephyr SDK in for of a huge cumulative patch.

If of any interest existing open repository with our changes is available here https://github.com/abrodkin/qemu/tree/zephyr.

Corresponding platform for Zephyr itself is posted for review here: zephyrproject-rtos/zephyr#25100

@galak
Copy link
Contributor

galak commented May 8, 2020

In the patch, this looks fishy:

diff --git a/include/sysemu/arch_init.h b/include/sysemu/arch_init.h
index 62c6fe4cf1..4935b00a2d 100644
--- a/include/sysemu/arch_init.h
+++ b/include/sysemu/arch_init.h
@@ -24,6 +24,7 @@ enum {
QEMU_ARCH_NIOS2 = (1 << 17),
QEMU_ARCH_HPPA = (1 << 18),
QEMU_ARCH_RISCV = (1 << 19),

  • QEMU_ARCH_ARC = (1 << 19),
    };

Should that be 20 instead?

@abrodkin
Copy link
Collaborator Author

abrodkin commented May 8, 2020

What a catch! It was there from the very first commit in our tree :)
I'll do a re-spin as this is an obvious fail :(

QEMU port for ARCv2 is almost ready and we're working on
opening all our work for starters via Synopsys GitHub repo
https://github.com/foss-for-synopsys-dwc-arc-processors/qemu
and later as a part of upstream project.

For now we submit changes for Zephyr SDK in for of a huge
cumulative patch.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
@galak galak merged commit 098dd43 into zephyrproject-rtos:master May 9, 2020
@galak
Copy link
Contributor

galak commented May 9, 2020

Please look to have a similar pr based on qemu5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants