drivers: spi: cc13xx_cc26xx: Allow clocks below 2 MHz #87400
Conversation
zephyrbot
added
area: SPI
|
Hello @dlim04, and thank you very much for your first pull request to the Zephyr project! |
Remove unnecessary check that the SPI clock is being set to a frequency above 2 MHz to allow devices running at common lower frequencies (i.e. 1.2 MHz and 400 kHz). Replace with check that the frequency is not below the minimum frequency supported by the chipset to prevent overflow error which can occur if the HAL sets a frequency too low resulting in a SPI clock much larger than expected. Fixes zephyrproject-rtos#69986 Signed-off-by: David Lim <dlim04@qub.ac.uk>
dlim04
force-pushed
the
69986_fix_cc13xx_cc26xx_spi
branch
from
75fb20f to
f2b0369
Compare
|
Hi @dlim04! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
Remove unnecessary check that the SPI clock is being set to a frequency above 2 MHz to allow devices running at common lower frequencies (i.e. 1.2 MHz and 400 kHz).
Replace with check that the frequency is not below the minimum frequency supported by the chipset to prevent overflow error which can occur if the HAL sets a frequency too low resulting in a SPI clock much larger than expected.
Fixes #69986