nRF9160 Timer issue

RE: nRF9160 Timer issue

Håkon Alseth — Thu, 17 Sep 2020 09:24:15 GMT

Hi,

Could you try starting the timer instance after initializing io_init and pwm_init?

Kind regards,

Håkon

RE: nRF9160 Timer issue

ephraim — Wed, 16 Sep 2020 09:19:29 GMT

Hi.

I stripped all unnecessary code .

Still the same error ( reset over and over.)

Please Advise

/*
 * Copyright (c) 2018 Nordic Semiconductor ASA
 *
 * SPDX-License-Identifier: LicenseRef-BSD-5-Clause-Nordic
 */

#include <zephyr.h>
#include <stdio.h>
#include <drivers/uart.h>
#include <string.h>
#include <stdlib.h>
#include <main.h>

#include <drivers/adc.h>
#include <drivers/spi.h>
#include <drivers/i2c.h>
#include <drivers/gpio.h>
#include <drivers/pwm.h>
//#include <drivers/timer.h>

#include <net/mqtt.h>
#include <net/socket.h>
#include <lte_lc.h>
#if defined(CONFIG_LWM2M_CARRIER)
#include <lwm2m_carrier.h>
#endif

#include <kernel.h>
#include <c:/zephyr/v1.2.0/zephyr/include/power/reboot.h>


#include <MT25QL128ABA.h>
#include <afe4900.h>
#include <adp536x.h>
#include <lis2dh12.h>
#include <init.h>

#include <hal/nrf_saadc.h>
#include <stdarg.h>
 

#define TIEMR_INTERVAL_SEC 1
#define TIEMR_INTERVAL_mSEC 10
 
struct k_timer bz_timer;

 
struct device * pwm_dev; 



#define MY_STACK_SIZE 512
#define MY_PRIORITY 5
K_THREAD_STACK_DEFINE(my_stack_area, MY_STACK_SIZE);
struct k_work_q my_work_q;

struct device_info {
    struct k_work work;
    char name[16]
} my_device;

  
  

  

void bz_func(struct k_timer *timer_id) 
{
     k_work_submit(&my_device.work);
}


void activate_buzzer(struct k_work *item)
{
     unsigned char k;
     static unsigned int time = 0;

    struct device_info *the_device =
        CONTAINER_OF(item, struct device_info, work);
   
   
   
     if(time == 0)
     {
       printk("Fisrt PWM set \n" );
       pwm_pin_set_usec(pwm_dev,PWM_IO, 260, 130,0);
       SetLed(BLUE,ON); 
       SetLed(GREEN,OFF); 
     }
    if(time == 10)
    {
       printk("Second PWM set \n" );
       pwm_pin_set_usec(pwm_dev,PWM_IO, 400, 200,0);
       SetLed(BLUE,OFF); 
       SetLed(GREEN,ON); 
     }
    if(++time >= 20)
       time = 0;
}



void main(void)
{
      int err;
      static unsigned int i = 0;
 

 
  //    initStructures();
  
  //     _AFE4900_AQUIRE_MODE.aquire_mode =  afe4900_aquire_sample;

      k_timer_init(&bz_timer, bz_func, NULL);
      k_timer_start(&bz_timer, K_SECONDS(1), K_MSEC(100));

      k_work_q_start(&my_work_q, my_stack_area,
              K_THREAD_STACK_SIZEOF(my_stack_area), MY_PRIORITY); 

      strcpy(my_device.name, "FOO_dev");
      k_work_init(&my_device.work, activate_buzzer);



      io_init();
      pwm_init();
      

 
      while(1) {

          k_sleep(K_MSEC(2000));
    
      };
}

RE: nRF9160 Timer issue

Håkon Alseth — Wed, 16 Sep 2020 08:53:54 GMT

Hi,

In this case, my_isr() is equivalent to your k_timer handler (bz_func).

Kind regards,

Håkon

RE: nRF9160 Timer issue

ephraim — Wed, 16 Sep 2020 08:36:26 GMT

Hi.

One more thing

/* install my_isr() as interrupt handler for the device (not shown) */
...

How do I set thid handler ?.

Thanks

RE: nRF9160 Timer issue

Håkon Alseth — Wed, 16 Sep 2020 06:49:12 GMT

You should not call functions that take long time or rely on other interrupts from a k_timer handler, as its also in interrupt context.

If you use a work queue, you schedule the function to another context to avoid this scenario, just as described in the work queues documentation:

https://docs.zephyrproject.org/1.9.0/kernel/threads/workqueues.html

Kind regards,

Håkon

RE: nRF9160 Timer issue

ephraim — Wed, 16 Sep 2020 06:42:28 GMT

Hi.

I am sorry , I am not following you.

Weather I use the workqueue , I will need to use the Timer.

Is it possible to produce interrupts without the timer ?.

if(time == 10)
{
// if(pwm_pin_set_usec(pwm_dev,PWM_IO, 400, 200,0) == 0)
{
SetLed(BLUE,OFF);
SetLed(GREEN,ON);
}

It is the pwm_pin_set_usec(pwm_dev,PWM_IO, 400, 200,0) that make the reset .

RE: nRF9160 Timer issue

Håkon Alseth — Tue, 15 Sep 2020 07:32:22 GMT

Hi,

The pwm zephyr wrapper for nRF devices is found in pwm_nrfx.c.

I cannot say what is wrong in your case, but the k_timer is inherited from the RTC interrupt handler, and the PWM module also relies on interrupts/events when setting the pin.

Can you try to use a workqueue instead of handling the PWM calls in the k_timer event handler?

You can read about workqueues here:

https://docs.zephyrproject.org/1.9.0/kernel/threads/workqueues.html

Kind regards,

Håkon

RE: nRF9160 Timer issue

ephraim — Mon, 14 Sep 2020 10:54:03 GMT

Hi.

I found the culprit ,

void bz_func(struct k_timer *timer_id) 
{
   // k_work_submit(&work_bz);       
    unsigned char k;
     static unsigned int time = 0;
  //    SetLed(BLUE,ON);  
     k = _BZ_SM.bz_state;
     printk("Buzzer \n");
     // Siren
     if(time == 0)
     {
//      if(pwm_pin_set_usec(pwm_dev,PWM_IO, 260, 130,0) == 0)
      {
        SetLed(BLUE,ON); 
        SetLed(GREEN,OFF);
      }
//      else
//      {
//        SetLed(RED,ON);
//        printk("Buzzer Error\n");
//      }
     }


     if(time == 10)
     {
//      if(pwm_pin_set_usec(pwm_dev,PWM_IO, 400, 200,0) == 0)
      {
        SetLed(BLUE,OFF); 
       SetLed(GREEN,ON); 
     }
//      else
//      {
//        SetLed(RED,ON);
//        printk("Buzzer Error\n");
//      }
     }
     if(++time >= 20)
       time = 0;

}

The Timer works fine without the update of the PWM .

When the PWM is re commented , the program reset itself ( see pic ).

The same code works fine in another small project .

Please Advise

RE: nRF9160 Timer issue

Håkon Alseth — Mon, 14 Sep 2020 10:04:21 GMT

Hi,

[quote user=""]In another larger project , I insert the Timer code , and it behaves erratically .[/quote]

What do you mean by this? Does it stop working after a while? Or does it not update the pwm value?

I see that you're not checking the return of the call to pwm_pin_set_usec. Please implement a check on the return code to ensure that the value has been set properly.

Kind regards,

Håkon