-
Notifications
You must be signed in to change notification settings - Fork 417
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
Enable to support USB boot #1175
Conversation
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.
Thanks for this, @harunobukurokawa. I've left some comments and we also need a documentation update on this.
50909af
to
daf7e0c
Compare
3532001
to
fedae88
Compare
fedae88
to
e36c072
Compare
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.
This looks good to me now. Thanks.
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.
@harunobukurokawa, this looks good to me now. Could you rebase your commits and update the PR description to match the current changes?
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]>
00c7b0f
to
b522aba
Compare
@agherzan I rebased and push my commit. |
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 to me. Kicked off the CI.
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.
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.
@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. |
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