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

added some better comments to ADC example #1

Open
wants to merge 2 commits into
base: stm32f3
Choose a base branch
from

Conversation

rewolff
Copy link

@rewolff rewolff commented Jun 27, 2013

I'm testing your stm32f3 examples and I think users should contribute (small useful changes) more when they use open source things. So here goes.

It took me some time to figure out that the code was not using the internal temperature sensor, although the temperature sensor was mentioned in the code. (I didn't see the variable get overwritten the next line. :-( )

@rewolff
Copy link
Author

rewolff commented Jun 29, 2013

I've just tested the temp sensor ADC channel, and somehow it still doesn't work.
I might look into it later on, but no guarantees.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants