-
Notifications
You must be signed in to change notification settings - Fork 192
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
pcf8591: add ADC only implementation for I2C ADC/DAC #690
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except for the small parts I pointed out, it looked good to me
Thanks for the quick feedback @sago35 changes made to this PR, please take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK except for the following
Signed-off-by: deadprogram <ron@hybridgroup.com>
Thanks for the proofreading @sago35 changes made and pushed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds an ADC only implementation for the pcf8591 I2C ADC/DAC.
A future PR can add the DAC feature.