Beware that this post is related to an SDK in maintenance mode
More Info: Consider nRF Connect SDK for new designs
This post is older than 2 years and might not be relevant anymore
More Info: Consider searching for newer posts

Meaning of SAADC_CONFIG_LP_MODE. Misleading comment in nRF5_SDK_14.2.0_17b948a/config/sdk_config.h ?

Not really a question, just that I find the comment misleading. In the sdk_config.h example coming with the SDK there is this piece of code:

// <q> SAADC_CONFIG_LP_MODE - Enabling low power mode


#ifndef SAADC_CONFIG_LP_MODE
#define SAADC_CONFIG_LP_MODE 0
#endif

IMHO, comment would be improved as follows:

// <q> SAADC_CONFIG_LP_MODE - Enabling low power mode
// <0=> Disable
// <1=> Enable

#ifndef SAADC_CONFIG_LP_MODE
#define SAADC_CONFIG_LP_MODE 0
#endif

BTW, as far as I can understand, low power mode does not concern the SAADC HW but the way it interacts with PPI, TIMER, DMA, etc… for HW triggered acquisition. So if you make the acquisition in one-shot and triggered by SW, then what LP mode you set in the driver does not make any difference. Is that correct ?

Parents Reply Children
No Data
Related