-
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
Samples: Bluetooth: ISO time sync fixes #17646
base: main
Are you sure you want to change the base?
Samples: Bluetooth: ISO time sync fixes #17646
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 1ebb4c84aab369654e52229c3bf858b7c3342b39 more detailssdk-nrf:
Github labels
List of changed files detected by CI (4)
Outputs:ToolchainVersion: 9583beca34 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
71b6e56
to
becb41e
Compare
This commit fixes the following: * Fix a bug in the initialization of `init_val` in the GPIOTE task used for the LED, where it would always be initialized to `NRF_GPIOTE_INITIAL_VALUE_HIGH` and not take the dt_flags into account. This made the LED toggle with inverted polarity on platforms where the LEDs are active high. * Change the LED used when `CONFIG_LED_TOGGLE_IMMEDIATELY_ON_SEND_OR_RECEIVE` is enabled to not conflict with the LED used to display synchronized toggling. * Update references to LED numbers in the README to match the actual sample behavior. Signed-off-by: Ludvig Jordet <[email protected]>
becb41e
to
1ebb4c8
Compare
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. |
This commit fixes the following:
Sample now uses absolute pin for GPIOTE, to make the LED work on platforms where the LEDs are not on port 0 (such as nRF54L15).init_val
in the GPIOTE task used for the LED, where it would always be initialized toNRF_GPIOTE_INITIAL_VALUE_HIGH
and not take the dt_flags into account. This made the LED toggle with inverted polarity on platforms where the LEDs are active high.CONFIG_LED_TOGGLE_IMMEDIATELY_ON_SEND_OR_RECEIVE
is enabled to not conflict with the LED used to display synchronized toggling.