STM32 processor does not have analog reference available, constant DEFAULT not defined

[mitchmitchell]

In Node.h, the statement uint8_t _analog_reference = DEFAULT; should be protected by an #ifdef CHIP_AVR / #endif pair as the code which uses it is in Node.cpp since the constant DEFAULT is not defined for non AVR processors (or at least not for STM32 processors)

Node.cpp is:

ifdef CHIP_AVR

// nothing to do
if (value == _analog_reference) return;
// change the analog reference
analogReference(value);
// wait a bit 
wait(200);
// perform some reading before actually reading the value
if (pin > -1) {
    for (int i = 0; i < 5; i++) {
        analogRead(pin);
        wait(50);
    }
}

endif

Node.h SHOULD be as below:

ifdef CHIP_AVR

uint8_t _analog_reference = DEFAULT;
#endif

Reference regarding ADC accuracy: https://www.st.com/content/ccc/resource/technical/document/application_note/group0/3f/4c/a4/82/bd/63/4e/92/CD00211314/files/CD00211314.pdf/jcr:content/translations/en.CD00211314.pdf

[user2684]

For my own reference, related issue and a few other things to fix for this chipset https://forum.mysensors.org/topic/10169/compile-nodemanager-for-stm32f103c8-but-i-get-error/2

[user2684]

Fixed by #512 (also those fixes from the MySensors forum)