Skip to content

Conversation

@erhankur
Copy link
Collaborator

@erhankur erhankur commented Oct 30, 2025

Checklist

  • Component contains License
  • Component contains README.md
  • Component contains idf_component.yml file with url field defined
  • Component was added to upload job
  • Component was added to build job
  • Optional: Component contains unit tests
  • CI passing

Change description

  • Introduces esp_sysview as a managed component for ESP-IDF.
  • Extracted from components/app_trace/sysview in esp-idf.
  • Internal MR 43011

@erhankur erhankur changed the title Add esp_sysview (SEGGER SystemView) component Draft: Add esp_sysview (SEGGER SystemView) component Oct 30, 2025
@erhankur erhankur force-pushed the add_sysview branch 3 times, most recently from b778414 to bbb0d0e Compare October 30, 2025 13:52
@erhankur erhankur force-pushed the add_sysview branch 2 times, most recently from 56e70a0 to b709a78 Compare November 11, 2025 20:06
@erhankur erhankur force-pushed the add_sysview branch 2 times, most recently from 31ff1d8 to e25fe4a Compare November 14, 2025 09:25
@erhankur erhankur changed the title Draft: Add esp_sysview (SEGGER SystemView) component Add esp_sysview (SEGGER SystemView) component Nov 14, 2025
@erhankur erhankur requested a review from suda-morris November 14, 2025 22:53
@erhankur
Copy link
Collaborator Author

@suda-morris PTAL

Copy link
Contributor

Copilot AI left a 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.

@erhankur
Copy link
Collaborator Author

@suda-morris one more PTAL

@suda-morris suda-morris merged commit 866943c into espressif:master Nov 28, 2025
85 checks passed
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