Skip to content

Conversation

feeph
Copy link
Owner

@feeph feeph commented Jul 20, 2024

This is a different way to implement the same functionality as read_device_registers() and write_device_register() when scheduling multiple read/write operations on the same device.

  • In addition it's now possible to perform read and write operations while holding the same lock. This was not possible with read_device_registers() and write_device_register() and is expected to improve usability and robustness.
  • However it is not possible to use feeph.i2c.BurstHandler() to schedule multiple read/write operations on different devices. This regression was a conscious choice in favor of a more convenient interface.

This is a different way to implement the same functionality as
`read_device_registers()` and `write_device_register()` when
scheduling multiple read/write operations _on the same device_.

- In addition it's now possible to perform read _and_ write operations
  while holding the same lock. This was not possible with
  `read_device_registers()` and `write_device_register()` and is expected
  to improve usability and robustness.
- However it is not possible to use `feeph.i2c.BurstHandler()` to schedule
  multiple read/write operations on different devices. This regression was
  a conscious choice in favor of a more convenient interface.
@feeph feeph added the enhancement New feature or request label Jul 20, 2024
@feeph feeph self-assigned this Jul 20, 2024
@feeph feeph merged commit a728a8b into master Jul 20, 2024
@feeph feeph deleted the fb-context-manager branch July 20, 2024 21:05
feeph added a commit that referenced this pull request Jul 20, 2024
##
[0.5.0](v0.4.1...v0.5.0)
(2024-07-20)


### Features

* provide a context manager for read/write operations
([#20](#20))
([a728a8b](a728a8b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant