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

elfloader/cmake-tool support for RPi4 #153

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoonasOnatsu
Copy link

@JoonasOnatsu JoonasOnatsu commented Oct 27, 2022

This PR adds SMP support for RPi4 to elfloader.

@axel-h axel-h added the hw-build enable all sel4test hardware builds label Oct 27, 2022
@Ivan-Velickovic
Copy link
Contributor

Ivan-Velickovic commented Nov 8, 2022

I am able to run seL4test on a RPi4 without any changes to the elfloader using the latest U-Boot, maybe older U-Boot versions did not support EFI images?

Cherry-pick 'smp.c' and 'smp_head.S' from
ARM Research IceCap repository for BCM2711 platform.
Commit ID: b2b92939bf94443e7bf13a15801561ef7caf565d
URL: https://gitlab.com/icecap-project/seL4_tools/-/commit/<SHA>

UART support files are not selected from the commit,
as kernel correctly generates header(s) for the
'bcm2835-aux-uart', which is the same as on RPi3, and
the common 'bcm-uart.c' driver can be used for RPi4 too.

Signed-off-by: Joonas Onatsu <joonasx@ssrc.tii.ae>
@JoonasOnatsu
Copy link
Author

I am able to run seL4test on a RPi4 without any changes to the elfloader using the latest U-Boot, maybe older U-Boot versions did not support EFI images?

Hi Ivan, thanks for your comment. I also tested with the latest U-Boot, and it seems that you are correct and the image can be booted with 'bootefi'. I think I had something misconfigured before, and I couldn't boot the image.

I removed the commit for changing the image type.


extern void core_entry_head(unsigned long stack);

unsigned long core_stack;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this holds an address, then we should simply say so.

Suggested change
unsigned long core_stack;
void *core_stack;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hw-build enable all sel4test hardware builds
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants