Skip to content

added coresight helper(utils) and sink-source-test#389

Open
Rohan-in-Qualcomm wants to merge 1 commit intoqualcomm-linux:mainfrom
Rohan-in-Qualcomm:debug-smmu
Open

added coresight helper(utils) and sink-source-test#389
Rohan-in-Qualcomm wants to merge 1 commit intoqualcomm-linux:mainfrom
Rohan-in-Qualcomm:debug-smmu

Conversation

@Rohan-in-Qualcomm
Copy link
Copy Markdown

@Rohan-in-Qualcomm Rohan-in-Qualcomm commented Apr 7, 2026

  • Coresight-Sink-Source-Test: Validate trace capture across all CoreSight source and sink combinations by enabling sources, routing to sinks, and verifying collected trace data.

Also a common library helper function coresight_helper.sh is added to prevent duplication of code blocks.

@smuppand
Copy link
Copy Markdown
Contributor

smuppand commented Apr 7, 2026

@Rohan-in-Qualcomm This PR is too large for a clean review in its current form.

It is 1 commit / 19 files and bundles a shared helper plus 6 different CoreSight suites. It also mixes repeated YAML/orchestration issues with runner-specific correctness bugs, which makes it hard to review and hard to rework safely.

Please split this into smaller PRs. Suggested grouping:

  1. Runner/utils/coresight_helper.sh + Coresight-Sink-Source-Test
  2. Ftrace-Dump-ETF-Base
  3. Reset-All-Sinks-Base + Reset-Random-Sinks-Base
  4. Single-Sink-Reset-Base + Single-Sink-Reset-Connected-Base

Also please update the PR title/body to match the actual scope. This is a CoreSight debug-suite PR, not an SMMU PR.

After the split, the file-level issues will be much easier to review cleanly.

@Rohan-in-Qualcomm Rohan-in-Qualcomm force-pushed the debug-smmu branch 2 times, most recently from fc727ca to 61c81a2 Compare April 8, 2026 05:41
@Rohan-in-Qualcomm Rohan-in-Qualcomm changed the title Added SMMU Support for Embedded Trace Router scripts added coresight helper(utils) and sink-source-test Apr 8, 2026
Signed-off-by: Rohan Dutta <rohadutt@qti.qualcomm.com>
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