Skip to content

zephyr: update to zephyr 4.4.0#19157

Open
dpgeorge wants to merge 5 commits into
micropython:masterfrom
dpgeorge:zephyr-update-4-4-0
Open

zephyr: update to zephyr 4.4.0#19157
dpgeorge wants to merge 5 commits into
micropython:masterfrom
dpgeorge:zephyr-update-4-4-0

Conversation

@dpgeorge
Copy link
Copy Markdown
Member

Summary

This PR updates the zephyr port to work with zephyr 4.4.0, which was released two weeks ago.

Testing

Tested on frdm_k64f and nucleo_wb55rg using all possible tests in the test suite, include BLE tests on the WB55. There are no regressions.

(Note that the WB55 needs the latest HCI WS firmware to work correctly.)

Will also be tested under CI.

Generative AI

I did not use generative AI tools when creating this PR.

Changed in zephyr commit e24d3b92a3a2b016d932d075fad0b2d9e1922f22, the only
option for connectable advertising is now BT_LE_ADV_OPT_CONN, whose value
is _BT_LE_ADV_OPT_CONNECTABLE|_BT_LE_ADV_OPT_ONE_TIME.

Signed-off-by: Damien George <damien@micropython.org>
It was removed in zephyr commit b348fd4d7a6ba92aa45ae8d96ad018c0e0cf11db.

Signed-off-by: Damien George <damien@micropython.org>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.46%. Comparing base (8a56be6) to head (6fee5cf).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #19157   +/-   ##
=======================================
  Coverage   98.46%   98.46%           
=======================================
  Files         176      176           
  Lines       22811    22811           
=======================================
  Hits        22460    22460           
  Misses        351      351           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Otherwise there is a compiler error.  Likely due to the update to use
zephyr SDK 1.0.1.

Signed-off-by: Damien George <damien@micropython.org>
To be consistent with the setting in `prj.conf`, so that the function
`uart_console_in_debug_hook_install()` is enabled.

Signed-off-by: Damien George <damien@micropython.org>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

Code size report:

Reference:  extmod/moductypes: Be more defensive with uctypes_struct_agg_size args. [8a56be6]
Comparison: zephyr: Upgrade to Zephyr v4.4.0. [merge of 6fee5cf]
  mpy-cross:    +0 +0.000% 
   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
      esp32:    +0 +0.000% ESP32_GENERIC
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS
  qemu rv32:    +0 +0.000% VIRT_RV32

@dpgeorge dpgeorge force-pushed the zephyr-update-4-4-0 branch from 0c4b872 to 74f21df Compare April 27, 2026 02:46
Updates the Zephyr port build instructions and CI to use the latest Zephyr
release tag.

Tested on frdm_k64f and nucleo_wb55rg.

Signed-off-by: Damien George <damien@micropython.org>
@dpgeorge dpgeorge force-pushed the zephyr-update-4-4-0 branch from 74f21df to 6fee5cf Compare April 27, 2026 03:11
@dpgeorge
Copy link
Copy Markdown
Member Author

dpgeorge commented May 1, 2026

@VynDragon maybe you would like to review this PR? Are you using zephyr 4.4.0?

@VynDragon
Copy link
Copy Markdown
Contributor

More of a mainline/ bleeding edge user. What do you need validated? Just see if it builds and runs with some random mcus? I do not use QEMU otherwise.

@dpgeorge
Copy link
Copy Markdown
Member Author

dpgeorge commented May 1, 2026

What do you need validated

I tested on hardware mentioned above. Also CI runs the qemu build over the test suite. So there's nothing much extra to do in that regard.

Just if you could take a look at the changes and see if anything jumps out at you as being wrong or could be improved.

@VynDragon
Copy link
Copy Markdown
Contributor

I don't see anything wrong, but I am not qualified on the changes pertaining to QEMU, STM, and BLE (but switching to BT_LE_ADV_OPT_CONN looks correct).

@dpgeorge
Copy link
Copy Markdown
Member Author

dpgeorge commented May 1, 2026

OK, thanks for looking!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants