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

Add ODroid C4 support #95

Merged
merged 1 commit into from
Jul 9, 2021
Merged

Add ODroid C4 support #95

merged 1 commit into from
Jul 9, 2021

Conversation

xurtis
Copy link
Contributor

@xurtis xurtis commented Jul 8, 2021

Adds the ODroid C4 as a valid device.

Adds the ODroid C4 as a valid device.

Signed-off-by: Curtis Millar <curtis@curtism.me>
reset_cntvoff();
}

#if CONFIG_MAX_NUM_NODES > 1
Copy link
Member

Choose a reason for hiding this comment

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

There is the alias ENABLE_SMP_SUPPORT that is also in use, is there a policy when to use what?

Copy link
Member

Choose a reason for hiding this comment

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

On a 2nd look, seems this is only used in the kernel, but not in the ELF loader. So ignore that comment above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think ENABLE_SMP_SUPPORT is only defined inside the kernel.

Copy link
Member

Choose a reason for hiding this comment

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

It used to be, but then the kernel's private config file was moved into libsel4: https://github.com/seL4/seL4/blob/master/libsel4/include/sel4/config.h#L96

Copy link
Member

Choose a reason for hiding this comment

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

Could be a thing for later then: #96

@nomadeel
Copy link
Contributor

nomadeel commented Jul 9, 2021

Looks good to me, the CONFIG_MAX_NUM_NODES could be resolved in a different PR instead.

@xurtis xurtis merged commit 83c1891 into seL4:master Jul 9, 2021
@xurtis xurtis deleted the odroidc4 branch July 9, 2021 04:05
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.

None yet

4 participants