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

Read two registers #57

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

htsmith11
Copy link

@htsmith11 htsmith11 commented Jan 28, 2025

added support to have a 2 register I2C read

- Reading 2x registers in one I2C read will improve speed
- Reads 2x registers in one I2C transaction
@Letme
Copy link
Member

Letme commented Jan 28, 2025

Nice that you combined 2 reads into one, but we already had a problem with endianness of 16 bit read because of different CPU architectures, and this now just adds to the confusion for some users. We gain some time with i2c read, but was there any problem with speed of read even at fastest 0.5s readout time?

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