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

DPS310 I2C Pressure Sensor Interface #24

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

Conversation

Burke-Daniel
Copy link
Member

This PR contains the I2C interface code for the DPS310 pressure sensor being used in Wind Tunnel testing. This code came from the vendor, but was refactored to remove inheritance and virtual function calls, to be more compliant with typical embedded software standards.

If you see anything that may need to be written/refactored/improved, let me know.

@Burke-Daniel Burke-Daniel changed the title Feature/nd/pressure sensor i2 c interface DPS310 I2C Pressure Sensor Interface Oct 3, 2020
@Burke-Daniel
Copy link
Member Author

Copy link
Member

@colton-smith colton-smith left a comment

Choose a reason for hiding this comment

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

Missing newlines at EOF then GTM

uint8_t length;
} RegBlock_t;

#endif
Copy link
Member

Choose a reason for hiding this comment

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

No newline at EOF


const RegBlock_t coeffBlock = {0x10, 18};
} // namespace dps310
#endif
Copy link
Member

Choose a reason for hiding this comment

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

No newline at EOF

};

} // namespace dps
#endif /* DPS_CONSTS_H_ */
Copy link
Member

Choose a reason for hiding this comment

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

No newline at EOF

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