-
Notifications
You must be signed in to change notification settings - Fork 191
esp-qcom-image: convert to OE-core uki.bbclass #722
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
Conversation
|
See qualcomm-linux/meta-qcom-hwe#138 for review and comment history |
11851e3 to
4f01b2b
Compare
ndechesne
left a comment
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 neat!
|
|
||
| UKI_CMDLINE="root=${QCOM_BOOTIMG_ROOTFS} rw rootwait" | ||
|
|
||
| # Remove leading 'qcom/' from KERNEL_DEVICEREE, we store the file in DEPLOY_DIR_IMAGE directly, not in the qcom/ subfolder |
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.
do we need to keep these 2 lines here?
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.
The bottom one can go for sure, the top on is still needed AFAICT.
|
@koenkooi I think this is more or less ready now. Could you please squash those commits? Let's merge it, unless there are any obstacles. |
|
@koenkooi ping, let's finish and merge this. |
I've squashed and rebased it |
I think it's now failing on the copyright checks. LGTM otherwise. |
I've added a copyright header, but I do require some feedback on that:
|
You could define the range, but if owned by Qualcomm just updating to the latest is fine.
Yes, MIT is fine, that is our main license here. |
This class uses the OE-core uki.bbclass infrastructure to assemble an ESP image with systemd-boot + UKI. Systemd-boot is installed through package management, the UKI is picked up from deploy. Signed-off-by: Koen Kooi <koen.kooi@oss.qualcomm.com>
|
Jobs failed to run https://github.com/qualcomm-linux/meta-qcom/actions/runs/14358395851 |
|
Should had removed my approval, this PR is not yet fully functional, as I pasted above it fails to boot now. |
This new class uses the OE-core uki.bbclass infrastructure to assemble an
ESP image with systemd-boot + UKI. It uses method #2 for config-less booting.
Systemd-boot is installed through package management, the UKI is picked
up from deploy.
Signed-off-by: Koen Kooi koen.kooi@oss.qualcomm.com
Fixes #707