Skip to content

applications: nrf5340_audio: Implement USB feedback#26997

Draft
gWacey wants to merge 1 commit intonrfconnect:mainfrom
gWacey:OCT-3572-Implement-feedback-mechanism-for-USB
Draft

applications: nrf5340_audio: Implement USB feedback#26997
gWacey wants to merge 1 commit intonrfconnect:mainfrom
gWacey:OCT-3572-Implement-feedback-mechanism-for-USB

Conversation

@gWacey
Copy link
Contributor

@gWacey gWacey commented Feb 12, 2026

OCT-3572

This is to control the rate of data received over the USB link, so as to prevent over- or under-runs within the audio system.

The feedback module calculates the error between the USB clock and the nRF5340 audio DK clock. This is then used to generate a Q10.24 or Q16.16 samples per 1ms value, depending on the speed (fast or full) of the USB connection.
There is also now an overflow net-buff, this gives an overflow value in terms of bytes. This overflow is compared to user defined high and low tide values so as to trigger the calculation.

The code in this DRAFT PR builds but fails when it attempts to initialize the USB endpoints.

This is to control the rate of data received over the
USB link, so as to prevent over- or under-runs within
the audio system.

Signed-off-by: Graham Wacey <graham.wacey@nordicsemi.no>
@gWacey gWacey added the DNM label Feb 12, 2026
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 12, 2026
@NordicBuilder
Copy link
Contributor

CI Information

To view the history of this post, click the 'edited' button above
Build number: 1

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

Note: This message is automatically posted and updated by the CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants