Skip to content

Conversation

Sai-teja573
Copy link
Contributor

This PR introduces a new test script gdsp_remoteproc.sh designed to validate the remoteproc functionality of gpdsp0 and gpdsp1 on the LeMans platform. It also includes a detailed README.md to document the test's purpose, usage, and expected output.

What's Included
1.gdsp_remoteproc.sh: Shell script that Identifies the correct remoteproc entries for gpdsp0 and gpdsp1
2.Verifies initial running state
3.Performs stop and restart operations
4.Confirms successful return to the running state
5. Logs results in a standardized format
6. README.md

Target Platform
LeMans

Notes
This test is specific to LeMans and may require adaptation for other platforms.
Root access is required to control remoteproc state.

@vnarapar vnarapar requested review from smuppand and vnarapar June 18, 2025 15:29
@Sai-teja573 Sai-teja573 force-pushed the gdsp-remoteproc-script branch from d8cb6d6 to fcbe270 Compare June 20, 2025 06:27
log_info "=== Test Initialization ==="

# Loop through all remoteproc firmware entries to find gpdsp0 and gpdsp1
for gdsp_firmware in gpdsp0 gpdsp1; do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea to have a modular and reusable approach for remoteproc validation. It's better to move it to functestlib.sh files, allowing it to find the remoteproc path for a given firmware substring.

remoteproc_path=$(dirname "$fw_path")
log_info "Found $gdsp_firmware at $remoteproc_path"

state1=$(cat "$remoteproc_path/state")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same even for fetching state

exit 1
fi

echo stop > "$remoteproc_path/state"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for restart too. Stop/ Start.

Copy link
Contributor

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better to refactor the tests by moving common code to functestlib.sh for better modularization.

This commit introduces a test script to validate gpdsp0 and gpdsp1 remoteproc functionality on the Lemans platform. It includes start/stop/restart checks and logs results in a standardized format. A detailed README is also added to document usage, expected output, and prerequisites.

Signed-off-by: Sai-teja573 <[email protected]>
@Sai-teja573 Sai-teja573 force-pushed the gdsp-remoteproc-script branch from fcbe270 to 13f98aa Compare June 23, 2025 08:26
@Sai-teja573 Sai-teja573 deleted the gdsp-remoteproc-script branch June 23, 2025 09:22
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.

2 participants