Skip to content

Conversation

@carlosperate
Copy link
Collaborator

So that the ADC input range is configured from 0 to Vdd instead of the current 0 to 3.6V, as discussed in #142.

So that the ADC input range is configured from 0 to Vdd instead of the
current 0 to 3.6V.
Copy link
Owner

@sandeepmistry sandeepmistry left a comment

Choose a reason for hiding this comment

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

Should we do the same for nRF52?

@carlosperate
Copy link
Collaborator Author

Probably a good idea as well. It looks like the internal reference is 0.6v so no easy way to do 0 to 3.3V with that.
Setting Vdd/4 as reference and keeping a gain of 4 would do the trick.
I could add that to the PR, but I don't really have an nRF52 board to test it.

@sandeepmistry sandeepmistry merged commit 98444be into sandeepmistry:master May 7, 2017
@sandeepmistry
Copy link
Owner

I'll merge this for now, we'll have to tackle the nRF52 side later

@sandeepmistry
Copy link
Owner

@carlosperate thanks for the contribution!

@carlosperate
Copy link
Collaborator Author

carlosperate commented May 8, 2017

Sounds good, thanks! Hope you don't mind I've created issue #158 to not forget about this.

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