RE: FreeRTOS+WDT+SDK17 problem

user86158 — Tue, 24 May 2022 00:11:38 GMT

[quote userid="6207" url="~/f/nordic-q-a/63998/freertos-wdt-sdk17-problem/263131#263131"]Thanks for letting us know. For your reference, our internal bug tracking number is NRFFOETT-2516[/quote]

Has this bug been fixed -- it still seems to be there in SDK 17.1.0

RE: FreeRTOS+WDT+SDK17 problem

user86158 — Thu, 12 May 2022 17:37:20 GMT

I am continuing to see this 'bug' in SDK v17.1.0.

What is the ETA for a fix?

One more comment:

The condition " if (nrf_wdt_started())" will only ever be true IF, and only IF, the bit 'RUNSTATUS' in the WDT status register is actually a '1'.
So then, why does this bit read back as a '1' BEFORE the firmware initializes the WDT module?
I do not see a #define in the 'sdk_config.h' as influencing the behavior of the pre-compiled SoftDevice or MBR so it is not clear to me what is setting this bit to a '1' after the SoC goes through a power up reset.

RVM

RE: FreeRTOS+WDT+SDK17 problem

user17468 — Wed, 05 Aug 2020 14:10:04 GMT

OK.Thanks.

RE: FreeRTOS+WDT+SDK17 problem

user6207 — Wed, 05 Aug 2020 08:14:32 GMT

Wow, Thanks for such a clear description.

The below line is a bug and should not be there.

    if (nrf_wdt_started())
    {
        m_clock_cb.lfclk_on = true;
    }

I will create a bug report internally to let the team know about this.

Thanks for letting us know. For your reference, our internal bug tracking number is NRFFOETT-2516

RE: FreeRTOS+WDT+SDK17 problem

user17468 — Tue, 04 Aug 2020 15:06:03 GMT

Target board is PCA10056(nRF52840).

/**
 * Copyright (c) 2015 - 2020, Nordic Semiconductor ASA
 *
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without modification,
 * are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice, this
 *    list of conditions and the following disclaimer.
 *
 * 2. Redistributions in binary form, except as embedded into a Nordic
 *    Semiconductor ASA integrated circuit in a product or a software update for
 *    such product, must reproduce the above copyright notice, this list of
 *    conditions and the following disclaimer in the documentation and/or other
 *    materials provided with the distribution.
 *
 * 3. Neither the name of Nordic Semiconductor ASA nor the names of its
 *    contributors may be used to endorse or promote products derived from this
 *    software without specific prior written permission.
 *
 * 4. This software, with or without modification, must only be used with a
 *    Nordic Semiconductor ASA integrated circuit.
 *
 * 5. Any software provided in binary form under this license must not be reverse
 *    engineered, decompiled, modified and/or disassembled.
 *
 * THIS SOFTWARE IS PROVIDED BY NORDIC SEMICONDUCTOR ASA "AS IS" AND ANY EXPRESS
 * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
 * OF MERCHANTABILITY, NONINFRINGEMENT, AND FITNESS FOR A PARTICULAR PURPOSE ARE
 * DISCLAIMED. IN NO EVENT SHALL NORDIC SEMICONDUCTOR ASA OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
 * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
 * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 */
/** @file
 * @defgroup blinky_example_main main.c
 * @{
 * @ingroup blinky_example_freertos
 *
 * @brief Blinky FreeRTOS Example Application main file.
 *
 * This file contains the source code for a sample application using FreeRTOS to blink LEDs.
 *
 */

#include <stdbool.h>
#include <stdint.h>

#include "FreeRTOS.h"
#include "task.h"
#include "timers.h"
#include "bsp.h"
#include "nordic_common.h"
#include "nrf_drv_clock.h"
#include "sdk_errors.h"
#include "app_error.h"
#include "nrf_drv_wdt.h"

#define	WDT_ENABLE			1

#if LEDS_NUMBER <= 2
#error "Board is not equipped with enough amount of LEDs"
#endif

#define TASK_DELAY        200           /**< Task delay. Delays a LED0 task for 200 ms */
#define TIMER_PERIOD      1000          /**< Timer period. LED1 timer will expire after 1000 ms */

TaskHandle_t  led_toggle_task_handle;   /**< Reference to LED0 toggling FreeRTOS task. */
TimerHandle_t led_toggle_timer_handle;  /**< Reference to LED1 toggling FreeRTOS timer. */

/**@brief LED0 task entry function.
 *
 * @param[in] pvParameter   Pointer that will be used as the parameter for the task.
 */
static void led_toggle_task_function (void * pvParameter)
{
    UNUSED_PARAMETER(pvParameter);
    while (true)
    {
#if WDT_ENABLE
    	nrf_drv_wdt_feed();
#endif
        bsp_board_led_invert(BSP_BOARD_LED_0);

        /* Delay a task for a given number of ticks */
        vTaskDelay(TASK_DELAY);

        /* Tasks must be implemented to never return... */
    }
}

/**@brief The function to call when the LED1 FreeRTOS timer expires.
 *
 * @param[in] pvParameter   Pointer that will be used as the parameter for the timer.
 */
static void led_toggle_timer_callback (void * pvParameter)
{
    UNUSED_PARAMETER(pvParameter);
    bsp_board_led_invert(BSP_BOARD_LED_1);
}

#if WDT_ENABLE
static nrf_drv_wdt_channel_id m_channel_id;

static void wdt_event_handler(void)
{
}
#endif

static void bsp_evt_handler(bsp_event_t evt)
{
    switch (evt)
    {
        case BSP_EVENT_KEY_0:
    	NVIC_SystemReset(); /* Software Reset */
            break;
    	
    default:
    		break;
    }
}

int main(void)
{
    ret_code_t err_code;

    /* Initialize clock driver for better time accuracy in FREERTOS */
    err_code = nrf_drv_clock_init();
    APP_ERROR_CHECK(err_code);

    /* Configure LED-pins as outputs */
    err_code = bsp_init(BSP_INIT_LEDS | BSP_INIT_BUTTONS, bsp_evt_handler);
    APP_ERROR_CHECK(err_code);

    /* Create task for LED0 blinking with priority set to 2 */
    UNUSED_VARIABLE(xTaskCreate(led_toggle_task_function, "LED0", configMINIMAL_STACK_SIZE + 200, NULL, 2, &led_toggle_task_handle));

    /* Start timer for LED1 blinking */
    led_toggle_timer_handle = xTimerCreate( "LED1", TIMER_PERIOD, pdTRUE, NULL, led_toggle_timer_callback);
    UNUSED_VARIABLE(xTimerStart(led_toggle_timer_handle, 0));

    /* Activate deep sleep mode */
    SCB->SCR |= SCB_SCR_SLEEPDEEP_Msk;

#if WDT_ENABLE
    nrf_drv_wdt_config_t config = NRF_DRV_WDT_DEAFULT_CONFIG;
    err_code = nrf_drv_wdt_init(&config, wdt_event_handler);
    APP_ERROR_CHECK(err_code);
    err_code = nrf_drv_wdt_channel_alloc(&m_channel_id);
    APP_ERROR_CHECK(err_code);
    nrf_drv_wdt_enable();
#endif
	/* Start FreeRTOS scheduler. */
    vTaskStartScheduler();

    while (true)
    {
        /* FreeRTOS should not be here... FreeRTOS goes back to the start of stack
         * in vTaskStartScheduler function. */
    }
}

/**
 *@}
 **/

And sdk_config.h modified
#define NRFX_WDT_CONFIG_RELOAD_VALUE 10000

If #define WDT_ENABLE 1. (WDT enabled)
1. When PoweOnReset, LED is blinking.
2. When Button 1 pushed(execute SoftwareReset), LED blinking is stopped.
3. After 10 seconds, LED blinking is started.

If #define WDT_ENABLE 0. (WDT disabled)
1. When PoweOnReset, LED is blinking.
2. When Button 1 pushed(execute SoftwareReset), LED blinking is restarted quickly.

RE: FreeRTOS+WDT+SDK17 problem

user6207 — Tue, 04 Aug 2020 11:48:45 GMT

[quote user="loquat"]The problem is that if a software reset occurs while WtchDog is running, FreeRTOS will not work properly.
And the cause is that LFCLK is not initialized properly.[/quote]

i cannot see this behavior in my quick test on the desk. No software resets have happened in my test. Can you please attach your project so that i can test this.

RE: FreeRTOS+WDT+SDK17 problem

user17468 — Tue, 04 Aug 2020 11:43:19 GMT

I add feeding the watchdog in led_toggle_task_function.

But the problem is not there.
The problem is that if a software reset occurs while WtchDog is running, FreeRTOS will not work properly.
And the cause is that LFCLK is not initialized properly.

RE: FreeRTOS+WDT+SDK17 problem

user6207 — Tue, 04 Aug 2020 10:11:25 GMT

Where are you feeding the watchdog? (where is your application calling nrf_drv_wdt_channel_feed?)

For me it is working fine if i feed the wdt in the bsp_event_callback with a button press. the wdt reset happens if i do not press the button.

RE: FreeRTOS+WDT+SDK17 problem

user6207 — Tue, 04 Aug 2020 09:34:57 GMT

Syuji,

Sorry for late response, we started coming back from summer holidays. I will take a look at this problem today, will come back to you soon.