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.

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

feat: improvements according to application note AN5289 #135

Merged
merged 6 commits into from
May 30, 2023

Conversation

gabrielsantosphilips
Copy link
Contributor

Updates according to AN5289 document:

6.8 - 2 Mbps

  • Constructor sets the default PHY to 2 Mbps.

6.9 - Update connection parameters

  • The function aci_gap_start_connection_update should be invoked only if a parameter update is requested, as we don't have an interface defined to do that, it is pointless keeping this function on code base.

7.6.9 - Maximize data throughput

  • Connection parameters:
    • Connection interface: 7.5 ms (document suggests 400 ms, but this slows down the entire execution flow).
  • Function hci_le_set_data_length kept, after the connection is established, data length is configured to 251 octets.

@github-actions
Copy link
Contributor

github-actions bot commented May 23, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.08s
⚠️ MARKDOWN markdownlint 20 14 2 1.66s
⚠️ MARKDOWN markdown-link-check 20 13 37.76s
✅ MARKDOWN markdown-table-formatter 20 14 0 1.02s
✅ REPOSITORY checkov yes no 21.94s
✅ REPOSITORY dustilock yes no 0.04s
✅ REPOSITORY gitleaks yes no 94.8s
✅ REPOSITORY git_diff yes no 0.73s
✅ REPOSITORY secretlint yes no 29.26s
✅ REPOSITORY syft yes no 0.64s
✅ REPOSITORY trivy yes no 5.55s
✅ SPELL misspell 44 2 0 0.86s
✅ XML xmllint 17 2 0 0.84s
✅ YAML prettier 7 1 0 1.47s
✅ YAML v8r 7 0 6.43s
✅ YAML yamllint 7 0 0.24s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@gabrielsantosphilips gabrielsantosphilips marked this pull request as ready for review May 23, 2023 12:25
@gabrielsantosphilips gabrielsantosphilips requested a review from a team as a code owner May 23, 2023 12:25
@gabrielsantosphilips gabrielsantosphilips merged commit bd95621 into main May 30, 2023
@gabrielsantosphilips gabrielsantosphilips deleted the feature/stm32wb55_ble_performance branch May 30, 2023 14:01
@gabrielsantosphilips gabrielsantosphilips restored the feature/stm32wb55_ble_performance branch October 23, 2023 06:51
@gabrielsantosphilips gabrielsantosphilips deleted the feature/stm32wb55_ble_performance branch October 23, 2023 06:55
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.

2 participants