-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
applications: nrf5340_audio: Handle PD=0 in broadcast_sink #18091
base: main
Are you sure you want to change the base?
applications: nrf5340_audio: Handle PD=0 in broadcast_sink #18091
Conversation
alexsven
commented
Oct 22, 2024
- If presentation delay is set to 0, use the default and print a warning.
- OCT-2952
- If presentation delay is set to 0, use the default and print a warning. - OCT-2952 Signed-off-by: Alexander Svensen <alexander.svensen@nordicsemi.no>
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 9425d860a47fb751ee00d8b55ca87c5c9ff9ac91 more detailssdk-nrf:
Github labels
List of changed files detected by CI (2)
Outputs:ToolchainVersion: 3dd8985b56 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
@@ -866,7 +866,8 @@ ZBUS_LISTENER_DEFINE(sdu_ref_msg_listen, audio_datapath_sdu_ref_update); | |||
int audio_datapath_pres_delay_us_set(uint32_t delay_us) | |||
{ | |||
if (!IN_RANGE(delay_us, CONFIG_AUDIO_MIN_PRES_DLY_US, CONFIG_AUDIO_MAX_PRES_DLY_US)) { | |||
LOG_WRN("Presentation delay not supported: %d", delay_us); | |||
LOG_WRN("Presentation delay not supported: %d us", delay_us); | |||
LOG_WRN("Keeping current value: %d us", ctrl_blk.pres_comp.pres_delay_us); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are only log changes in this PR. No handling required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct. Since the default is already used, and then you are not allowed to set PD=0 it will continue to use the default value. So I just removed the error print (as that would only be printed if someone attempts to set PD=0) and then printed what the default value is