Skip to content

audioreach-driver: fix const-correctness and header rewrite - #53

Merged
quic-aditrath merged 1 commit into
AudioReach:masterfrom
aiyapate:const_func_ptr_fix
Mar 4, 2026
Merged

quic-aditrath merged 1 commit into
AudioReach:masterfrom
aiyapate:const_func_ptr_fix

Conversation

@aiyapate

@aiyapate aiyapate commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

The GPR core updated the callback API to use a const-qualified packet type. Update the q6apm and q6prm callbacks to match the new signature.

The earlier code modified fields in the inbound GPR packet header, which is now const. Copy the packet buffer and apply port remapping only to the local copy.

@aiyapate
aiyapate requested review from a team March 4, 2026 05:40
The GPR core updated the callback API to use a const-qualified packet
type. Update the q6apm and q6prm callbacks to match the new signature.

The earlier code modified fields in the inbound GPR packet header,
which is now const. Copy the packet buffer and apply port remapping
only to the local copy.

Signed-off-by: Aiyaz Patel <aiyapate@qti.qualcomm.com>
@aiyapate
aiyapate force-pushed the const_func_ptr_fix branch from 29784c7 to 0c683e6 Compare March 4, 2026 05:49

@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

@quic-aditrath
quic-aditrath merged commit b0a33b7 into AudioReach:master Mar 4, 2026
11 of 12 checks passed
@qswat-orbit-external

Copy link
Copy Markdown

Merge Check Failed: No CR Numbers Found

Error: No Change Request numbers were found.

Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests.

1 similar comment
@qswat-orbit-external

Copy link
Copy Markdown

Merge Check Failed: No CR Numbers Found

Error: No Change Request numbers were found.

Please add Change Request numbers to your pull request description in the format CRs-Fixed: 12345 or link GitHub issues that are associated with Change Requests.

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