-
Notifications
You must be signed in to change notification settings - Fork 121
Add esp_sysview (SEGGER SystemView) component #600
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
esp_sysview/src/Sample/FreeRTOSV10.4/Config/esp/SEGGER_SYSVIEW_Config_FreeRTOS.c
Fixed
Show fixed
Hide fixed
b778414 to
bbb0d0e
Compare
bbb0d0e to
9ab2d43
Compare
esp_sysview/src/Sample/FreeRTOSV10.4/Config/esp/SEGGER_SYSVIEW_Config_FreeRTOS.c
Fixed
Show fixed
Hide fixed
esp_sysview/src/Sample/FreeRTOSV10.4/Config/esp/SEGGER_SYSVIEW_Config_FreeRTOS.c
Fixed
Show fixed
Hide fixed
esp_sysview/src/Sample/FreeRTOSV10.4/Config/esp/SEGGER_SYSVIEW_Config_FreeRTOS.c
Fixed
Show fixed
Hide fixed
56e70a0 to
b709a78
Compare
31ff1d8 to
e25fe4a
Compare
|
@suda-morris PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds the esp_sysview component to idf-extra-components, which provides SEGGER SystemView integration for ESP-IDF. The component has been extracted from ESP-IDF's internal app_trace/sysview module and is now being published as a managed component.
Key changes:
- Introduces SEGGER SystemView v3.56 as a standalone component
- Provides real-time tracing and visualization capabilities for ESP32 systems
- Includes adapter code to integrate SystemView with ESP-IDF's trace infrastructure
- Supports heap tracing through SystemView
Reviewed Changes
Copilot reviewed 31 out of 33 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| esp_sysview/src/include/esp_trace_freertos_impl.h | Header that includes SEGGER_SYSVIEW_FreeRTOS.h and undefines INLINE to avoid redefinition |
| esp_sysview/src/include/esp_sysview_heap_trace_module.h | Public API for heap tracing through SystemView |
| esp_sysview/src/ext/logging.c | Logging implementation using SystemView |
| esp_sysview/src/ext/heap_trace_tohost.c | Heap trace implementation that sends events to host via SystemView |
| esp_sysview/src/ext/heap_trace_module.c | SystemView heap tracing module registration and event handlers |
| esp_sysview/src/esp/SEGGER_RTT_esp.c | ESP-specific RTT (Real-Time Transfer) implementation |
| esp_sysview/src/adapter_encoder_sysview.* | Adapter layer connecting SystemView encoder to ESP trace infrastructure |
| esp_sysview/src/Sample/FreeRTOSV10.4/* | FreeRTOS integration for SystemView |
| esp_sysview/src/SEGGER/* | Core SEGGER SystemView library files (v3.56) |
| esp_sysview/src/Config/* | Configuration files for SystemView |
| esp_sysview/sdkconfig.rename | Config option mappings for backward compatibility |
| esp_sysview/linker.lf | Linker fragment placing SystemView code in RAM |
| esp_sysview/idf_component.yml | Component manifest |
| .idf_build_apps.toml | Build system integration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e25fe4a to
bc6f1eb
Compare
bc6f1eb to
845cbb8
Compare
|
@suda-morris one more PTAL |
Checklist
urlfield definedChange description
esp_sysviewas a managed component for ESP-IDF.components/app_trace/sysviewin esp-idf.