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.

Sign up for GitHub

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

Jump to bottom

ADIS16470 fixes. #40

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

ADIS16470 fixes. #40

wants to merge 1 commit into from

Conversation

meetgandhi-dev
Copy link

  1. Fixed Adis16470::read_register return value on success.
  2. Fixed Adis16470::update_burst read return value check (24 instead of 30)
  3. Fixed Adis16470::update_burst accl value updates according to the datasheet (Table 25. 16-Bit Accelerometer Data Format Examples)
  4. Fixed Adis16470::update_burst temerature offset handling (should be 17 instead of 16)
  5. Adis16470::set_bias_estimation_time use fprintf instead of ROSINFO to avoid ROS dependency in non-ROS source.

@meetgandhi-dev
ADIS16470 fixes.
d5822b5 
1. Fixed Adis16470::read_register return value on success.
2. Fixed Adis16470::update_burst read return value check (24 instead of 30)
3. Fixed Adis16470::update_burst accl value updates according to the datasheet (Table 25. 16-Bit Accelerometer Data Format Examples)
4. Fixed Adis16470::update_burst temerature offset handling (should be 17 instead of 16)
5. Adis16470::set_bias_estimation_time use fprintf instead of ROSINFO to avoid ROS dependency in non-ROS source.

Signed-off-by: Meet Gandhi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@meetgandhi-dev