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

Enable to support USB boot #1175

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

harunobukurokawa
Copy link
Contributor

@harunobukurokawa harunobukurokawa commented May 5, 2023

Raspberry Pi 4 board can boot the image from USB memory. But, current meta-raspberrypi supports only SD boot.

This commit allows switching SD boot to other device. For example, if you want to use USB boot, please define the following parameters in your conf/local.conf file.
CMDLINE_ROOT_PARTITION = "/dev/sda2"
BOOT_MEDIA = "usb"

- What I did
I checked
- How I did it
This commit is a proposal then I wrote some description in commit message

Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

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

Thanks for this, @harunobukurokawa. I've left some comments and we also need a documentation update on this.

recipes-bsp/bootfiles/rpi-cmdline.bb Outdated Show resolved Hide resolved
recipes-bsp/rpi-u-boot-scr/rpi-u-boot-scr.bb Outdated Show resolved Hide resolved
@harunobukurokawa harunobukurokawa force-pushed the sandbox/support-usbboot branch from 50909af to daf7e0c Compare June 13, 2023 14:16
recipes-bsp/rpi-u-boot-scr/files/boot.cmd.in Outdated Show resolved Hide resolved
recipes-bsp/bootfiles/rpi-cmdline.bb Show resolved Hide resolved
@harunobukurokawa harunobukurokawa force-pushed the sandbox/support-usbboot branch 2 times, most recently from 3532001 to fedae88 Compare August 20, 2023 11:48
Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

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

This looks good to me now. Thanks.

docs/extra-build-config.md Show resolved Hide resolved
Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

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

@harunobukurokawa, this looks good to me now. Could you rebase your commits and update the PR description to match the current changes?

@harunobukurokawa harunobukurokawa changed the title [RFC] Enable to support USB boot Enable to support USB boot Nov 16, 2023
The Raspberry Pi 4 board can boot the image from USB memory.
But current meta-raspberrypi supports only SD boot.

This commit allows switching SD boot to other device.
For example, if you want to use USB boot, please define
the following parameters in your conf/local.conf file.
CMDLINE_ROOT_PARTITION = "/dev/sda2"
BOOT_MEDIA = "usb"

v4: Update extra-build-config.md to explain more detail for new variable.

v3: Changed variable name to "BOOT_MEDIA" (was "BOOT_DEVTIPE").

v2: Set existing microSD parameter as default. User can overwrite
some parameters.

Signed-off-by: Harunobu Kurokawa <[email protected]>
@harunobukurokawa
Copy link
Contributor Author

@agherzan I rebased and push my commit.

Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

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

Looks good to me. Kicked off the CI.

@agherzan agherzan requested a review from kraj November 23, 2023 12:23
Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

This looks ok to me, however I think if we can have one knob instead of two, it might be better from usability point of view.

@agherzan
Copy link
Owner

@kraj I requested that so that we don't have to maintain values from uboot/kernel perspective. Indeed, a UX compromise. I'll leave it as such for now. We could have a better variable in the future if needed.

@agherzan agherzan merged commit 4c033eb into agherzan:master Dec 13, 2023
18 checks passed
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.

3 participants