External ADC Driver Implementation #20
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…start from scratch
Burke-Daniel
added
enhancement
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
First actual Node PR woot woot!
This contains the ADC driver I wrote for our specific 16-bit external ADC, the structure of this is similar to the external can driver that I found and brought into the repo. Some background info:
Datasheet can be found here
Our microcontroller (SAMD21...) interfaces with the external ADC through SPI, and so I was able to make use of the built in SPI functionality that arduino has. There are Enums defined with register addresses, commands, and configurations. These are used within the helper methods to abstract away a bit of the lower level stuff.
Still need to document this code a little bit and outline how this is to be set up and used in our main code.
Also note that this won't be merged until we can test it with the sensor board.