Skip to content

Tools: Testbench: Fix IPC4 PGA controls load#9591

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
singalsu:testbench4_pga_load_fix
Oct 21, 2024
Merged

Tools: Testbench: Fix IPC4 PGA controls load#9591
lgirdwood merged 1 commit intothesofproject:mainfrom
singalsu:testbench4_pga_load_fix

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 <seppo.ingalsuo@linux.intel.com>
@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
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