Skip to content

khadas-vim1 / khadas-vim2: bump prebuilt u-boot blob to khadas-uboot 0.17.3#9994

Open
igorpecovnik wants to merge 2 commits into
mainfrom
khadas-vim2-uboot-blob-0.17.3
Open

khadas-vim1 / khadas-vim2: bump prebuilt u-boot blob to khadas-uboot 0.17.3#9994
igorpecovnik wants to merge 2 commits into
mainfrom
khadas-vim2-uboot-blob-0.17.3

Conversation

@igorpecovnik

@igorpecovnik igorpecovnik commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Updates the prebuilt SD u-boot blobs for Khadas VIM1 and Khadas VIM2 from the khadas-uboot 0.11 release to 0.17.3 (u-boot 2021.07+, dated 2025-09-19).

Changes

  • khadas-vim2: u-boot-vim2-sd.bin → 0.17.3 (VIM2.u-boot.sd.bin, md5 b52581b80040a56778e0cc0389308b58)
  • khadas-vim1: u-boot-vim1-sd.bin → 0.17.3 (VIM1.u-boot.sd.bin, md5 34cf7192baebc532ac9a2aead76461e4)
  • Both blobs verified against the release list.md5sum. On-disk layout is unchanged (1363968 bytes), so UBOOT_TARGET_MAP is untouched apart from the provenance comment (0.110.17.3).

Source: https://github.com/khadas/khadas-uboot/releases/tag/0.17.3

Depends on

#9995 (artifact-uboot: hash local blob content) — without it the artifact version doesn't change when a blob is bumped in place, so the build would reuse the cached .deb and the new blobs wouldn't be packaged. Merge #9995 first.

Testing

Both boards tested successfully booting from SD card and eMMC.

Summary by CodeRabbit

  • Chores
    • Updated bootloader configuration version for khadas-vim boards.

Update the VIM2 SD u-boot blob from the khadas-uboot 0.11 release to
0.17.3 (u-boot 2021.07+, dated 2025-09-19). Same on-disk layout
(1363968 bytes: 1147392 u-boot + env), so the existing UBOOT_TARGET_MAP
raw-write is unchanged; only the binary and its provenance comment move
from tag 0.11 to 0.17.3.

Blob: VIM2.u-boot.sd.bin, md5 b52581b80040a56778e0cc0389308b58
(verified against the release list.md5sum).
Source: https://github.com/khadas/khadas-uboot/releases/tag/0.17.3

Signed-off-by: Igor Pecovnik <igor@armbian.com>
@igorpecovnik igorpecovnik requested a review from a team as a code owner June 18, 2026 15:50
@igorpecovnik igorpecovnik requested review from PanderMusubi and matthijskooijman and removed request for a team June 18, 2026 15:50
@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The workaround comment version for the prebuilt U-Boot used on khadas-vim1 and khadas-vim2 boards in meson-gxl.conf is updated from 0.11 to 0.17.3. The UBOOT_TARGET_MAP assignments for both boards are unchanged.

Changes

Khadas Board U-Boot Comment Update

Layer / File(s) Summary
Workaround comment version bump
config/sources/families/meson-gxl.conf
Updates the "temporally workaround" comment version from 0.11 to 0.17.3 in both the khadas-vim1 and khadas-vim2 conditionals; UBOOT_TARGET_MAP assignments remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

Ready to merge, 05, Patches

Suggested reviewers

  • ColorfulRhino
  • HeyMeco
  • Tonymac32
  • amazingfate
  • catalinii
  • efectn
  • lanefu
  • paolosabatino
  • prahal
  • pyavitz

Poem

🐰 A comment once read "zero point eleven,"
Now "zero point seventeen point three" — closer to heaven!
The VIM boards still boot with their prebuilt U-Boot grace,
Just a version string hopped to its proper place.
This bunny approves with a wiggle and a hop! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: bumping the prebuilt U-Boot blob version for two specific boards.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch khadas-vim2-uboot-blob-0.17.3

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size/small PR with less then 50 lines 08 Milestone: Third quarter release Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Framework Framework components BSP Board Support Packages and removed 08 Milestone: Third quarter release size/small PR with less then 50 lines labels Jun 18, 2026
Same bump as khadas-vim2: update the VIM1 SD u-boot blob from the
khadas-uboot 0.11 release to 0.17.3 (u-boot 2021.07+, dated 2025-09-19).
On-disk layout unchanged (1363968 bytes), so UBOOT_TARGET_MAP is
untouched apart from the provenance comment.

Blob: VIM1.u-boot.sd.bin, md5 34cf7192baebc532ac9a2aead76461e4
(verified against the release list.md5sum). Requires the artifact-uboot
local-blob hashing fix (separate PR) for the bump to trigger a rebuild.
Source: https://github.com/khadas/khadas-uboot/releases/tag/0.17.3

Signed-off-by: Igor Pecovnik <igor@armbian.com>
@igorpecovnik igorpecovnik force-pushed the khadas-vim2-uboot-blob-0.17.3 branch from 534d5bf to 2ccb2fe Compare June 18, 2026 15:52
@github-actions github-actions Bot added 08 Milestone: Third quarter release size/small PR with less then 50 lines labels Jun 18, 2026
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge and removed Needs review Seeking for review labels Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

08 Milestone: Third quarter release BSP Board Support Packages Framework Framework components Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

1 participant