Skip to content

boot: zephyr: Fix boot banner - #2843

Open
nordicjm wants to merge 2 commits into
mcu-tools:mainfrom
nordicjm:versionfix
Open

boot: zephyr: Fix boot banner#2843
nordicjm wants to merge 2 commits into
mcu-tools:mainfrom
nordicjm:versionfix

Conversation

@nordicjm

Copy link
Copy Markdown
Collaborator
Fixes the boot banner after changes in Zephyr

@rettichschnidi rettichschnidi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Works as expected:

*** Using Zephyr OS build v4.4.0-11893-g5d4fefb989e3 ***
*** Booting MCUboot v2.4.0-118-g7ad67106c325 ***
I: Starting bootloader
I: Image index: 0, Swap type: none
I: Image index: 0, Swap type: none
<snip>

Thanks a lot!

Nits:

  • MCU and Zephyr boot banners are now swapped. I guess this might trip some automated testing. Maybe add a hint to the release notes?
  • Would link to zephyrproject-rtos/zephyr#116947 in the commit message.

Comment thread docs/release-notes.d/zephyr-banner.md Outdated
@@ -0,0 +1,2 @@
- Zephyr MCUboot boot banner has been fixed after changes
made to the boot banned in Zephyr 4.4.

@rettichschnidi rettichschnidi Aug 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  1. Typo:
Suggested change
made to the boot banned in Zephyr 4.4.
made to the boot banner in Zephyr 4.4.
  1. The relevant change came in after Zephyr 4.4:
    $ git describe 9fe4cc20a2b5c7ef269206bfe80b8e606e813156
    v4.4.0-1380-g9fe4cc20a2b
    

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

Comment thread boot/zephyr/Kconfig Outdated
bool "Use MCUboot boot banner"
depends on BOOT_BANNER
depends on "$(APP_VERSION_EXTENDED_STRING)" != ""
default y

@rettichschnidi rettichschnidi Aug 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not sure if working well for everyone, but would suit my needs quite well:

Suggested change
default y
default y if BOOT_BANNER

(yep, PR feature creep 😄)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good idea, added

Fixes the boot banner after changes in Zephyr, as per issue:
zephyrproject-rtos/zephyr#116947

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Adds a note that this issue has been fixed

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>

@rettichschnidi rettichschnidi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks a lot!

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