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

config: runtime: boot: add support for barebox bootloader #2782

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

Conversation

hnez
Copy link

@hnez hnez commented Jan 23, 2025

We are currently in the process of reviving the Pengutronix LAVA lab for use with the new KernelCI infrastsructure.

As a first step we need Barebox bootloader support, since it is used by all boards in the Pengutronix lab.

This PR replaces #2600 by @pawiecz and includes the following suggestion by @a-wai #2600 (comment):

This doesn't take into account the case where boot_commands == nfs; I'd rather suggest to just copy the u-boot template and s/u-boot/barebox/ there, as it seems they should be otherwise identical.

The barebox template is indeed a copy of the u-boot template with the method set to barebox.

Barebox is used by platforms in Pengutronix' LAVA laboratory.

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
Copy link
Contributor

@pawiecz pawiecz left a comment

Choose a reason for hiding this comment

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

LGTM

Older nfsroot might not be able to run all the currently enabled tests but it can be updated later

@@ -0,0 +1,48 @@
{%- if boot_commands == 'nfs' and nfsroot is not defined %}
{%- set nfsroot = 'http://storage.kernelci.org/images/rootfs/debian/bookworm-kselftest/20240221.0/' + debarch %}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any specific reason to use 20240221.0 images? There were later updates to this rootfs

Copy link
Author

Choose a reason for hiding this comment

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

That's just what's currently used in u-boot.jinja2. I could add a commit that updates both to e.g. 20250117.0 if that makes sense.

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