-
Notifications
You must be signed in to change notification settings - Fork 133
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
RISCV32 Fixes #267
RISCV32 Fixes #267
Conversation
Signed-off-by: Khem Raj <raj.khem@gmail.com>
These recipes have gotten fixed overtime Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Use -x jitter to fix the cpu hog Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
# EXTRA_IMAGEDEPENDS += "u-boot" | ||
# UBOOT_MACHINE = "qemu-riscv32_smode_defconfig" | ||
# UBOOT_ELF = "u-boot" | ||
EXTRA_IMAGEDEPENDS += "u-boot" | ||
UBOOT_MACHINE = "qemu-riscv32_smode_defconfig" | ||
UBOOT_ELF = "u-boot" |
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.
Woohoo!
Subject: [PATCH] riscv32: Use double-float ABI for rv32 | ||
|
||
So it can use libgcc built with OE toolchain | ||
Fixes | ||
error: "can't link hard-float modules with soft-float modules" | ||
|
||
Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
--- |
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.
Awesome!
Can this go upstream instead of here? Once the Linux-yocto build is working again I wanted to upstream RV32 support.
FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:" | ||
|
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.
Can this go to OE-Core as well?
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.
perhaps yes but it should go along with the series to enable rv32, otherwise there will be questions as its purely rv32 patch
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.
Looks good! It would be great to start sending these to OE-Core instead though
No description provided.