RE: LOCOMP function failed when reference level being changed periodically

Avadacadabara — Fri, 18 Feb 2022 11:05:00 GMT

Thanks Jared,

Your code worked.

Meanwhile I figured out the problem of my code: "nrf_lpcomp_event_check(NRF_LPCOMP_EVENT_READY)" is not actually working, I still need to insert delay manually everytime I re-init the LPCOMP before sampling.

Thanks!

RE: LOCOMP function failed when reference level being changed periodically

Jared — Thu, 17 Feb 2022 13:54:11 GMT

Hi,

I wasn't able to reproduce this on my side, but I tried modifying the lp-comp example by adding your level_set() and see if it was able to change the reference voltage, which it did. Here is the code I used:

#define WAVE_ON_PIN_NUMBER 2

static volatile uint32_t voltage_falls_detected = 0;
static volatile uint32_t voltage_falls_total    = 0;

/**
 * @brief LPCOMP event handler is called when LPCOMP detects voltage drop.
 *
 * This function is called from interrupt context so it is very important
 * to return quickly. Don't put busy loops or any other CPU intensive actions here.
 * It is also not allowed to call soft device functions from it (if LPCOMP IRQ
 * priority is set to APP_IRQ_PRIORITY_HIGH).
 */
static void lpcomp_event_handler(nrf_lpcomp_event_t event)
{
    if (event == NRF_LPCOMP_EVENT_DOWN)
    {
        bsp_board_led_invert(BSP_BOARD_LED_0); // just change state of first LED
        voltage_falls_detected++;
        voltage_falls_total++;
    }
}


/**
 * @brief Print out detection statistics.
 */
static void print_statistics(void)
{
    while (voltage_falls_detected)
    {
        voltage_falls_detected--;
        NRF_LOG_INFO("#%d fall detected", (int)voltage_falls_total);
    }
}


/**
 * @brief Initialize LPCOMP driver.
 */
static void lpcomp_init(void)
{
    uint32_t                err_code;

    nrf_drv_lpcomp_config_t config = NRF_DRV_LPCOMP_DEFAULT_CONFIG;
    config.input = NRF_LPCOMP_INPUT_2;
    // initialize LPCOMP driver, from this point LPCOMP will be active and provided
    // event handler will be executed when defined action is detected
    err_code = nrf_drv_lpcomp_init(&config, lpcomp_event_handler);
    APP_ERROR_CHECK(err_code);
    nrf_drv_lpcomp_enable();
}




static void level_set(nrf_lpcomp_ref_t ref, nrf_lpcomp_detect_t det)	//NRF_LPCOMP_REF_SUPPLY_4_8  NRF_LPCOMP_DETECT_UP
{
    uint32_t	err_code;

    nrf_drv_lpcomp_config_t config = NRF_DRV_LPCOMP_DEFAULT_CONFIG;
    config.input = NRF_LPCOMP_INPUT_2;
		config.hal.hyst = NRF_LPCOMP_HYST_50mV;
		config.hal.reference = ref;
		config.hal.detection = det;

    // initialize LPCOMP driver, from this point LPCOMP will be active and provided
    // event handler will be executed when defined action is detected
    err_code = nrf_drv_lpcomp_init(&config, lpcomp_event_handler);
    APP_ERROR_CHECK(err_code);
		nrf_drv_lpcomp_enable();
}

int main(void)
{
    bsp_board_init(BSP_INIT_LEDS);

    nrf_gpio_cfg_output(WAVE_ON_PIN_NUMBER); // on this pin 2Hz wave will be generated

#ifdef BSP_BUTTON_0
    // configure pull-up on first button
    nrf_gpio_cfg_input(BSP_BUTTON_0, NRF_GPIO_PIN_PULLUP);
#endif

    uint32_t err_code = NRF_LOG_INIT(NULL);
    APP_ERROR_CHECK(err_code);

    NRF_LOG_DEFAULT_BACKENDS_INIT();

    lpcomp_init();

    NRF_LOG_INFO("LPCOMP driver usage example started.");

    int i = 0;

    while (true)
    {

      if(i == 10)
        { 
          nrf_drv_lpcomp_uninit();
          level_set(NRF_LPCOMP_REF_SUPPLY_13_16, NRF_LPCOMP_DETECT_DOWN);
          NRF_LOG_INFO("New reference voltage set to 13_16");
        
        }
        
        if(i == 20)
        { 
          i = 0;
          nrf_drv_lpcomp_uninit();
          level_set(NRF_LPCOMP_REF_SUPPLY_9_16, NRF_LPCOMP_DETECT_DOWN);
          NRF_LOG_INFO("New reference voltage set to 9_16");
        
        }

        i++;
        print_statistics();
        bsp_board_led_on(BSP_BOARD_LED_1);
        NRF_GPIO->OUTCLR = (1 << WAVE_ON_PIN_NUMBER);
        nrf_delay_ms(100); // generate 100 ms pulse on selected pin
        print_statistics();
        bsp_board_led_off(BSP_BOARD_LED_1);
        NRF_GPIO->OUTSET = (1 << WAVE_ON_PIN_NUMBER);
        nrf_delay_ms(400);
        NRF_LOG_FLUSH();
    }
}

regards

Jared