-
Notifications
You must be signed in to change notification settings - Fork 7.4k
Add I2C support to TI cc23x0 SoC #84526
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
base: main
Are you sure you want to change the base?
Add I2C support to TI cc23x0 SoC #84526
Conversation
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
38e32dc
to
e79aada
Compare
e79aada
to
bdb7220
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why constantly wait on interrupts to then swap back to the calling thread to then update a few registers? This is a lot of extra work that really isn't needed in this driver.
Make the driver a simple state machine that reads/writes out of a current transfer and wait for completion of the whole thing? any particular reason to have a k_sem_take in a for loop wait on byte at a time transfers?
bdb7220
to
af9c62b
Compare
Add support for I2C to cc23x0 SoC. Only controller mode is implemented. Signed-off-by: Stoyan Bogdanov <sbogdanov@baylibre.com>
Add support for I2C to cc23x0 SoC. Signed-off-by: Stoyan Bogdanov <sbogdanov@baylibre.com>
Enable I2C for TI lp_em_cc2340r5. Signed-off-by: Stoyan Bogdanov <sbogdanov@baylibre.com>
Add lp_em_cc2340r5 overlay in bme280 sensors sample. Signed-off-by: Stoyan Bogdanov <sbogdanov@baylibre.com>
af9c62b
to
c7a53a6
Compare
Hi @teburd I try to make driver work without IRQ, I was not able to get it working stable enough. Most likely because I am trying to fetch status reg all the time. Instead of Just rebased again on latest main , because of merged PR for the same platform. |
|
This series adds I2C support to TI cc23x0 SoC.
Datasheet: https://www.ti.com/lit/ds/symlink/cc2340r5.pdf