Fixes for STM32H5 demo. Fixes MQTT broker and improrves testing/docs. Expands CI test#73
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the STM32H563 demo to better support the wolfMQTT broker on bare-metal (wolfIP TLS + UART logging), expands documentation for broker usage, and adds a fuller CI job that boots the firmware in m33mu and integration-tests multiple services.
Changes:
- Configure wolfMQTT broker to use wolfIP-provided TLS context and route broker logs to UART via a custom
PRINTF. - Add
wolfmqtt_log()UART printf-style logger for WBLOG/broker logging. - Expand STM32H563 CI to build and integration-test echo/HTTPS/TLS-echo/MQTT/SSH in emulation.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/port/stm32h563/user_settings.h | Enables broker custom net/TLS integration and custom UART logging macros; raises broker log level. |
| src/port/stm32h563/main.c | Adds wolfmqtt_log() implementation using vsnprintf + UART output. |
| src/port/stm32h563/README.md | Updates build/flash instructions and adds detailed MQTT broker usage/testing docs. |
| .github/workflows/stm32h563-m33mu.yml | Adds a “full” CI job that emulates the firmware and performs network service checks incl. MQTT. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 21 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 21 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 22 out of 22 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 22 out of 22 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 23 out of 23 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Features
demo.sh) covering ping, TCP echo, HTTPS, SSH, and MQTTFixes
ssl.c,tls.c, etc.) assigned to an unused Makefile variableuser_settings.hnow guardsWOLFSSL_SP_ARM_CORTEX_M_ASMwith#ifndef WOLFSSL_SP_NO_ASM, preventingsp_c32.cfrom being compiled away in CI emulator buildstarget.ldexports_heap_limit = _estack - _Min_Stack_Size;_sbrk()enforces this upper boundstm32h563-m33mu-https-tls13.ymlintostm32h563-m33mu.yml; adds HTTPS, TLS echo, MQTT, and SSH banner test jobs