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

drivers/radio/nrf24l01: Increase startup delay. #953

Merged
merged 1 commit into from
Apr 10, 2025

Conversation

marcsello
Copy link
Contributor

According to the datasheet of the NRF240L1 chip, 150 μs startup time is only acceptable when the chip is clocked externally. Most modules use crystal, which require 1.5 ms to settle. I think it's okay to wait more in both cases, for a reliable startup.


ref: "6.1.7 Timing Information" in nRF24L01+ datasheet rev. 1.0

Without this patch I couldn't make my module to transmit reliably.

Copy link
Member

@dpgeorge dpgeorge left a comment

Choose a reason for hiding this comment

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

This looks good, thank you.

According to the datasheet of the NRF240L1 chip, 150 μs startup time is
only acceptable when the chip is clocked externally.  Most modules use a
crystal, which require 1.5 ms to settle.  It should be okay to wait more in
both cases, for a reliable startup.

Signed-off-by: Marcell Pünkösd <punkosdmarcell@rocketmail.com>
@dpgeorge dpgeorge force-pushed the fix-nrf24l01-startup-delay branch from b9d9036 to 3e859d2 Compare April 10, 2025 12:27
@dpgeorge dpgeorge merged commit 3e859d2 into micropython:master Apr 10, 2025
4 checks passed
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