Skip to content

Conversation

@ankuns
Copy link
Contributor

@ankuns ankuns commented Mar 6, 2025

This commit provides an extension to the i2c_nrfx_twim driver. It introduces possibility to acquire/release exclusive access to the i2c bus

The provided API is necessary for use cases when, i2c transfers must be started from interrupt context, while still allowing to share the i2c bus in multi-tasking environment.

Copy link
Contributor

@nordic-piks nordic-piks left a comment

Choose a reason for hiding this comment

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

Please add test/samples which showcase/prove that those uses cases works.

@ankuns ankuns force-pushed the nrfx_twim_async_api_cp branch from 6f2ce2c to 6121005 Compare March 20, 2025 10:00
@ankuns
Copy link
Contributor Author

ankuns commented Mar 20, 2025

@nordic-piks added.

@ankuns
Copy link
Contributor Author

ankuns commented Mar 20, 2025

It is not yet clear if this [nrf fromlist] can go in or the [nrf noup] version (#2624) will be allowed to go in.

This commit provides an extension to the i2c_nrfx_twim driver.
It introduces possibility to acquire/release exclusive access to the
i2c bus. While the exclusive access to the i2c bus is acquired
you can access the underlying nrfx_twim driver directly.

Signed-off-by: Andrzej Kuros <[email protected]>
(cherry picked from commit d15c9fe)
@ankuns ankuns force-pushed the nrfx_twim_async_api_cp branch from 6121005 to 3da2c59 Compare March 28, 2025 07:54
@ankuns ankuns changed the title [nrf fromlist] drivers: i2c_nrfx_twim: add asynchronous write and exlusive access API [nrf fromtree] drivers: i2c_nrfx_twim: add exlusive access API Mar 28, 2025
@ankuns
Copy link
Contributor Author

ankuns commented Mar 28, 2025

The upstream PR zephyrproject-rtos/zephyr#86721 is merged, cherry-picking as [nrf fromtree]

@ankuns ankuns added this to the ncs-3.0.0 milestone Mar 28, 2025
@ankuns
Copy link
Contributor Author

ankuns commented Mar 28, 2025

@anangl , waiting for approval.

@sonarqubecloud
Copy link

@rlubos rlubos merged commit db413b6 into nrfconnect:main Mar 28, 2025
22 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.

4 participants