feat: add MuziWorks Base bootloader support - #38
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 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. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 46e643456e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - Heltec T114 / HT-nRF5262 | ||
| - LilyGO T-Echo | ||
| - Minewsemi MX25LE01 | ||
| - MuziWorks Base Uno / Base Duo / Superbase |
There was a problem hiding this comment.
Require the new board's build check
Adding this board directory makes the set-matrix step in .github/workflows/githubci.yml create a new build (muzi_base) context, but master protection uses literal context names. Until a human adds this context, a future PR can merge even when the Muzi build fails; coordinate that protection update with this addition and refresh the documented board/check counts.
AGENTS.md reference: AGENTS.md:L127-L130
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Addressed the documentation portion in commits 97752ad and 2bf1783: the repository now records 18 boards, 19 required contexts (set-matrix + 18 board builds), and 4 of 18 boards with board.cmake. The new workflow run correctly includes the build (muzi_base) context but is currently awaiting maintainer approval (action_required). Adding that literal context to master branch protection requires a repository administrator. Hardware validation was also extended in the PR description: a complete BLE OTA application update succeeded on a physical MuziWorks Superbase, including transfer, flash, reboot, and application startup.
Checklist
tools/build_all.pypasses: 18 succeeded, 0 failed with ARM GCC 12.3.Rel1UF2_BOARD_IDis preserved asmuzi-Base-Board, matching the shipping device'sINFO_UF2.TXTDescription of Change
Adds OTAFIX bootloader support for MuziWorks Base Uno, Base Duo, and Superbase hardware.
The board definition preserves the manufacturer's original hardware contract:
239A:0081muzi-Basemuzi-Base-Board0xADA52840MUZI_DFUThe generated Muzi artifacts were also checked for approved flash ranges, UICR bootloader/MBR parameter addresses, combined HEX contents, SoftDevice presence, and serial recovery ZIP integrity.
Hardware validation
The shipping device reported bootloader
0.9.2-37-gf7fad14, modelmuzi Base, board IDmuzi-Base-Board, and S140 6.1.1. The OTAFIX build was installed by UF2 on a MuziWorks Superbase; installation completed and the device restarted into the existing Meshtastic application without errors.After installation, a complete Meshtastic application update was performed over BLE OTA on the same Superbase. The transfer completed, the image was written, and the node rebooted into the updated firmware successfully.