Skip to content
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

Tools: Testbench: Fix IPC4 PGA controls load #9591

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

singalsu
Copy link
Collaborator

This mistake causes the module id and instance number number to be garbage, so the control can't be set up later in widget setup. The issue fixes the valgrind step fail with new proposed IPC4 version of scripts/host-testbench.sh.

@singalsu singalsu marked this pull request as ready for review October 17, 2024 16:36
@singalsu singalsu requested a review from ranj063 as a code owner October 17, 2024 16:36
@singalsu singalsu force-pushed the testbench4_pga_load_fix branch from f1494c2 to 8590f58 Compare October 18, 2024 09:02
@singalsu
Copy link
Collaborator Author

I fixed the commit text duplicated word "number number", no other changes.

@lgirdwood
Copy link
Member

@wszypelt @lrudyX good to merge ? - not expecting this code to be checked with internal CI today. Thanks !

@wszypelt
Copy link

@lgirdwood Internal Intel CI, everything's fine.

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 21, 2024

@singalsu Can you rebase, it seems rebase-and-merge won't work anymore, this is now 22 patches behind mainline and some of your other testbench patches are conflicting.

This mistake causes the module id and instance number
to be garbage, so the control can't be set up later in
widget setup. The issue fixes the valgrind step fail
with new proposed IPC4 version of scripts/host-testbench.sh.

Signed-off-by: Seppo Ingalsuo <[email protected]>
@singalsu singalsu force-pushed the testbench4_pga_load_fix branch from 8590f58 to 0e2bd16 Compare October 21, 2024 13:09
@lgirdwood lgirdwood merged commit 8ba3528 into thesofproject:main Oct 21, 2024
42 of 47 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.

6 participants