Conversation
aabadie
left a comment
There was a problem hiding this comment.
The CI is reporting trailing whitespaces. Otherwise the changes look good (even if not verified with the datasheet but I tend to trust you @keestux :) ).
I won't be able to test this PR on my Sodaq boards before next week unfortunately. Maybe just provide some output ?
|
@aabadie the only thing I did was to use the For ADC I just used |
eba2937 to
413087c
Compare
Murdock found problems and there are indeed issues.
|
Murdock is reporting an issue with Looking a second time into this PR, it seems |
|
I'll have a look. My first impression is that this is a weird test to be part of RIOT. |
|
Also adding @smlng because he added this test. This test ( From the README of the test
It does not even say which Arduino, is it Arduino Zero? Can't we pin the test to just that board? |
Right now the gain and reference are hard coded. Until it is configurable at runtime prefer the same default as with Arduino Core. That is, GAIN_DIV2 and REFSEL_INTVCC1. This way we can properly measure the battery voltage.
The Sodaq boards support Arduino, but there are no ARDUINO_PIN_* defines.
3594a13 to
57463d7
Compare
|
@aabadie For now I have added my boards to the black list of that test. Murdock is happy. |
|
Thanks |
Contribution description
This PR adds the Arduino include files for 4 Sodaq boards (Autonomo, Explorer, One, Sara AFF).
As an extra this PR also adds the ADC configuration for Sodaq Autonomo. Furthermore it adjusts the ADC configurations for the other Sodaq boards so that each of them can properly reads it BATVOLT.
Testing procedure
Build and run
tests/periph_gpio_arduinoandtests/sys_arduino.