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

Update LTC2485.cpp #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

glitch911
Copy link

Fix bit operation to match datasheet description (only 6 bits are noise and should be removed, after logical shit left by one we need to remove 7 bits so /=128)

Fix bit operation to match datasheet description (only 6 bits are noise and should be removed, after logical shit left by one we need to remove 7  bits so /=128)
@thorenscientific
Copy link
Contributor

Hi @glitch911 , thanks for pointing this out (and I note that this PR is nearly a year old.) I've got an eval board on order to dig into this, but I'm planning on refactoring the LTC2485 driver to use the LTC24XX_general library. There are 18 parts in this family, with various quirks, and differences in data output formats. The LTC24XX_general library handles these differences, and all data transfer functions return data formatted as 32-bits, consistent with the SPI devices (I2C device's data is shifted left by two bits.)
Also the LTC24XX_diff_code_to_voltage handles a bug in the 2X speedup mode, where a digital output code corresponding to exactly zero is not formatted properly.
I also see that there was never an example sketch done for this part, so I'll try to add that as well.

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