Skip to content

Conversation

vkraleti
Copy link
Contributor

@vkraleti vkraleti commented Oct 2, 2025

IQ8275-EVK & IQ9075-EVK platforms use the same DSP binaries that of existing platforms. To avoid duplication, update config.txt to link those to the existing paths as appropriate.

Copy link
Collaborator

@lumag lumag left a comment

Choose a reason for hiding this comment

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

Obviously running make check isn't required when sending a PR. But please fix it anyway.

@vkraleti
Copy link
Contributor Author

vkraleti commented Oct 7, 2025

Obviously running make check isn't required when sending a PR. But please fix it anyway.

Done.

@vkraleti vkraleti requested a review from lumag October 7, 2025 05:29
@vkraleti vkraleti force-pushed the evk-support branch 2 times, most recently from 2674b86 to 2e0dcfb Compare October 7, 2025 07:33

if dst_path.endswith("/"):
sys.stderr.write("config.txt: %d: trailing '/' in %s\n" % (lineno, dst_path))
ret = False
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also please check that the source for symlink exists

Both "Install" & "Link" are valid config actions. Update check.py script
to parse 'Link' entries as well in addition to existing 'Install' entries.

Signed-off-by: Viswanath Kraleti <[email protected]>
Both IQ8275-EVK and QCS8300-RIDE platforms use the same DSP binaries.
To avoid duplication, update `config.txt` to link IQ8275-EVK DSP binary
paths to the existing QCS8300-RIDE binaries.

Signed-off-by: Viswanath Kraleti <[email protected]>
Both IQ9075-EVK and SA8775P-RIDE platforms use the same DSP binaries.
To avoid duplication, update `config.txt` to link IQ9075-EVK DSP binary
paths to the existing SA8775P-RIDE binaries.

Signed-off-by: Viswanath Kraleti <[email protected]>
@lumag lumag merged commit 89cd582 into linux-msm:trunk Oct 8, 2025
2 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.

2 participants