Skip to content

audioreach-driver: q6apm_audio_pkt: remap APM_EVENT_MODULE_TO_CLIENT … - #71

Merged
quic-aditrath merged 1 commit into
AudioReach:masterfrom
kvijayas:kvijayas/enable-sva-voiceui
Sep 9, 2026
Merged

quic-aditrath merged 1 commit into
AudioReach:masterfrom
kvijayas:kvijayas/enable-sva-voiceui

Conversation

@kvijayas

@kvijayas kvijayas commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

When a GPR response's token isn't found in the audio-packet port map, module-to-client events (APM_EVENT_MODULE_TO_CLIENT) destined for the APM module IID were being dropped instead of delivered.

Detect this case and remap the destination port to the SVA client port (8208), preserving the original source port, so these events reach the intended client instead of being silently discarded.

CRs-Fixed: 4656702

@kvijayas
kvijayas requested review from a team August 25, 2026 11:34
…dest_port

When a GPR response's token isn't found in the audio-packet port map,
module-to-client events (APM_EVENT_MODULE_TO_CLIENT) destined for the
APM module IID were being dropped instead of delivered.

Detect this case and remap the destination port to the SVA client
port (8208), preserving the original source port, so these events
reach the intended client instead of being silently discarded.

Signed-off-by: Kiruthika Vijayasekar <kvijayas@qti.qualcomm.com>
@kvijayas
kvijayas force-pushed the kvijayas/enable-sva-voiceui branch from 783efdf to 588c767 Compare August 27, 2026 15:36

@mohsRafi mohsRafi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mohsRafi

mohsRafi commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Please validate Basic audio test in any one of QLI target. make sure no ABI Break.

@kvijayas

kvijayas commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Please validate Basic audio test in any one of QLI target. make sure no ABI Break.

Validated on LeMans EVK (QCS9075):

  • Verified basic audio playback and capture streams.
  • Verified SVA keyword detection event routing and LAB buffer drain.
  • Confirmed no ABI break (no UAPI header or struct changes; internal packet routing only).

@kvijayas
kvijayas requested a review from mohsRafi September 9, 2026 13:45
@quic-aditrath
quic-aditrath merged commit ddc4515 into AudioReach:master Sep 9, 2026
14 of 16 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.

5 participants