Skip to content

Conversation

Cyrillius
Copy link

@Cyrillius Cyrillius commented Oct 17, 2025

LE_AUDIO_SDU_SIZE_OCTETS is a define which is used in nrf aura config application inside the function bt_le_audio_tx_send (https://github.com/nrfconnect/sdk-nrf/blob/v3.1.0/applications/nrf5340_audio/src/bluetooth/bt_stream/bt_le_audio_tx/bt_le_audio_tx.c#L259) with a frame duration other than 10ms like 7.5ms.

The problem is that this define is using an hard coded value of the frame duration and will miscalculate from the bitrate the sdu size and fail to send data.

Another solution would be to stop using LE_AUDIO_SDU_SIZE_OCTETS and let it as it is for the nrf5340 audio demo application which is using hard coded configuration.

@Cyrillius Cyrillius requested a review from a team as a code owner October 17, 2025 12:26
@Cyrillius Cyrillius requested review from alexsven, gWacey, koffes and rick1082 and removed request for a team October 17, 2025 12:26
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 17, 2025
@NordicBuilder
Copy link
Contributor

Thank you for your contribution!
It seems you are not a member of the nrfconnect GitHub organization. External contributions are handled as follows:
Large contributions, affecting multiple subsystems for example, may be rejected if they are complex, may introduce regressions due to lack of test coverage, or if they are not consistent with the architecture of nRF Connect SDK.
PRs will be run in our continuous integration (CI) test system.
If CI passes, PRs will be tagged for review and merged on successful completion of review. You may be asked to make some modifications to your contribution during review.
If CI fails, PRs may be rejected or may be tagged for review and rework.
PRs that become outdated due to other changes in the repository may be rejected or rework requested.
External contributions will be prioritized for review based on the relevance to current development efforts in nRF Connect SDK. Bug fix PRs will be prioritized.
You may raise issues or ask for help from our Technical Support team by visiting https://devzone.nordicsemi.com/.

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

@NordicBuilder NordicBuilder added the external External contribution label Oct 17, 2025
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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. external External contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants