Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(BLE): Setting dtmRxSyncMs to 10. #1267

Merged
merged 1 commit into from
Nov 14, 2024
Merged

fix(BLE): Setting dtmRxSyncMs to 10. #1267

merged 1 commit into from
Nov 14, 2024

Conversation

kevin-gillespie
Copy link
Contributor

@kevin-gillespie kevin-gillespie commented Nov 12, 2024 •
edited
Loading

Description

This solves a Bluetooth test mode problem when receiving long packets. Increasing the sync time to 10 ms prevents the receiver from timing out while receiving packets from the tester.

Checklist Before Requesting Review

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.

@kevin-gillespie
Setting dtmRxSyncMs to 10.
fd87628 
This solves a problem with the RX test with long coded PHY packets.
@github-actions github-actions bot added the BLE Related to Bluetooth label Nov 12, 2024
leifangadi
leifangadi approved these changes Nov 14, 2024
View reviewed changes
Copy link

@leifangadi leifangadi left a comment

Choose a reason for hiding this comment

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

in DTM heatmap test, increasing this number can help on reducing single-packet loss. However, it cannot be too long, otherwise coded modes (S2 and S8) at low power region(< -95dBm) will have huge (>30%) PER spikes.

@kevin-gillespie kevin-gillespie merged commit f6f5edd into main Nov 14, 2024
12 of 14 checks passed
@kevin-gillespie kevin-gillespie deleted the fix/dtm_rx_sync_ms branch November 14, 2024 15:52
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Nov 14, 2024
@actions-user
Update Zephyr MSDK Hal based on MSDK PR: analogdevicesinc/msdk#1267
2a7b7b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leifangadi leifangadi leifangadi approved these changes

Assignees
No one assigned
Labels
BLE Related to Bluetooth
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kevin-gillespie @leifangadi