comp or lpcomp for 2 analog inputs with different v-- (ref)

[ziv123]

i am working with nrf52832

i need to check 2 analog inputs and set my own v- (ref) 

i can have value read every few seconds so no fast readings needs to be done 

if i understand correctly i better use lpcomp if no fast readings is needed?

also,  is it possible, and if there are some code examples, to be reading 2 analog inputs with 2 different v-- (references) (one for each input pin) ?

best regards

Ziv

[Karl Ylvisaker]

Hello,

You will need to make sure that the sampling criterias in the SAADC documentation is satisfied, to ensure proper function of the SAADC. Mainly, this would be the continuous mode criteria:

 f_SAMPLE < 1/[t_ACQ + t_conv]

and the scan mode criteria: 

Total time < Sum(CH[x].tACQ+tCONV), x=0..enabled channels

As for the event that multiple triggers for the SAMPLE task happens at the same time, or in close succession I do not know what the output will be, since this is undefined behavior. My guess is that one of the triggers is ignored, which will cause you to loose a sample if you are going to only extract 1 of the samples each trigger.

ziv123 said:

also, if i use the 'app_timer' , does it changes the way i work with the ppi in the "least CPU intensive wat" asspect ?

I am not sure I understand what you are asking? The app timer module sets up the timer peripheral, which does not require CPU interaction after it is started. However, if it is operating in single_shot_mode, then you will need to have the CPU restart the timer every so often, for example.

ziv123 said:

if i am to use one saadc periferial with 3 channels (for 3 input pins), i wish to use 2 of them as high/low interrupts for the analog input and one that reads the values.. can you help with giving me direction or a close example?

Please see the "Limits event monitoring" section of the SAADC documentation.
As it is described there, you must set each channels limits, which will then trigger the events you mention when they are exceeded. You could use the set_limits function for this. 
Be advised that when having limits enabled and using scan mode, the limit event will trigger every time the channel is sampled and the limits are exceeded.

Best regards,
Karl

[ziv123]

well .. not sure i know where to go now... 

1. if i init sampeling (  nrf_drv_saadc_sample() ) via timer1 or flage in timer 1's handler, and uninit the saadc ( nrf_drv_saadc_uninit() ) in the saadc_callback function, i can risk calling for sampeling twice if i understood your answer correctly, which is something i don't want to do if it can cause problems

so there is no known safe solution to do the analog readings for each channel in a different period of time (for each channel, to be clear),  if i understand corrctly again

2. where this beeing my init:

"

ret_code_t err_code;

// set adc port for battery and level sensors
   nrf_drv_saadc_config_t saadc_config;
   nrf_saadc_channel_config_t channel_config0;

//Configure SAADC
   saadc_config.resolution = NRF_SAADC_RESOLUTION_10BIT; 
   saadc_config.oversample = NRF_SAADC_OVERSAMPLE_4X; 
   saadc_config.interrupt_priority = APP_IRQ_PRIORITY_LOW; //Set SAADC interrupt to low priority.
   saadc_config.low_power_mode = NRFX_SAADC_CONFIG_LP_MODE;

//Configure SAADC channel
   channel_config0.reference = NRF_SAADC_REFERENCE_INTERNAL; 
   channel_config0.gain = NRF_SAADC_GAIN1_6; 
   channel_config0.acq_time = NRF_SAADC_ACQTIME_10US; 
   channel_config0.mode = NRF_SAADC_MODE_SINGLE_ENDED; 
   channel_config0.pin_n = NRF_SAADC_INPUT_DISABLED; 
   channel_config0.resistor_p = NRF_SAADC_RESISTOR_DISABLED; 
   channel_config0.resistor_n = NRF_SAADC_RESISTOR_DISABLED;
   channel_config0.burst = NRF_SAADC_BURST_ENABLED;

//Initialize SAADC
   err_code = nrf_drv_saadc_init(&saadc_config, saadc_callback);
   APP_ERROR_CHECK(err_code);

   channel_config0.pin_p = IRRIGATION_LOW_LEVEL_ANALOG_SENS_PIN_IN;
   err_code = nrf_drv_saadc_channel_init(0, &channel_config0);
   APP_ERROR_CHECK(err_code);
   channel_config0.pin_p = EVACUATION_LOW_LEVEL_ANALOG_SENS_PIN_IN;
   err_code = nrf_drv_saadc_channel_init(1, &channel_config0);
   APP_ERROR_CHECK(err_code);

   channel_config0.pin_p = BATTERY_MILLI_WATT_PIN_IN;
   err_code = nrf_drv_saadc_channel_init(2, &channel_config0);
   APP_ERROR_CHECK(err_code);

// seting limits for the analog values read of the tanks:
   nrf_drv_saadc_limits_set(1, IRRI_TANK_LOW_V_LIMIT, IRRI_TANK_HIGH_V_LIMIT);
   nrf_drv_saadc_limits_set(2, EVAC_TANK_LOW_V_LIMIT, EVAC_TANK_HIGH_V_LIMIT);

   err_code = nrf_drv_saadc_buffer_convert(adc_read_buffer[0], NUMBER_OF_ADC_SAMPLES);
   APP_ERROR_CHECK(err_code);

   err_code = nrf_drv_saadc_buffer_convert(adc_read_buffer[1], NUMBER_OF_ADC_SAMPLES);
   APP_ERROR_CHECK(err_code);

"

and this beeing my saadc callback (and for the moment all reading as i understand are in the same time :

"

static void saadc_callback(nrf_drv_saadc_evt_t const * p_event)
{
  if (p_event->type == NRF_DRV_SAADC_EVT_DONE)
  {
       // here i need to read the battery value likewise
       battery_val = (uint16_t)(((uint32_t)(p_event->data.done.p_buffer[0])*3600)/1024);
  }
  if (p_event->type == NRFX_SAADC_EVT_LIMIT)
  {
      // i need to know somehow wich pin triggered the event and if it was high limit or low limit ??? how can           i do that ??
  }
}

"

i need to know somehow wich pin triggered the event and if it was high limit or low limit ??? how can i do that ?? 

3. you wrote: "Be advised that when having limits enabled and using scan mode, the limit event will trigger every time the channel is sampled and the limits are exceeded."

does that means that when limits enable then the saadc works all the time, or just when triggered via some timer for example? 

will each limits event will raise the NRFX_SAADC_EVT_LIMIT event type event followed by NRF_DRV_SAADC_EVT_DONE event type ?

also, if i want to be reading all 3 channels at the same time like that, the first channel will generate only the NRF_DRV_SAADC_EVT_DONE  event, so can i miss an event of type NRFX_SAADC_EVT_LIMIT  ? 

sorry for so many questions but i just don't get the right way to do 1 analog value read for one pin and 2 analog limits or compares for 2 other pins .. maybe it will be better to use LPCOMP afterall, and find out how to switch the pins in each read which means CPU intervention but i don't know what other option there is

4. can using this function

and this 

or this

help to configure an event for each channel ?

best regards

Ziv

[Karl Ylvisaker]

Hello Ziv,

Do I understand you correctly that you are intending to use the one SAADC channel to read the battery voltage of the device, and the two others to compare the pin levels to a reference?
If this is the case, you should see the Shared Resources documentation, and you might also want to have a look at this ticket(NB its for the nRF51 series).

ziv123 said:

1. if i init sampeling (  nrf_drv_saadc_sample() ) via timer1 or flage in timer 1's handler, and uninit the saadc ( nrf_drv_saadc_uninit() ) in the saadc_callback function, i can risk calling for sampeling twice if i understood your answer correctly, which is something i don't want to do if it can cause problems

What I said in my last comment was that double sampling(having two PPI events trigger the SAMPLE TASK at the same time) is undefined behavior - meaning it is not specified what happens if this occurs. You could test it out and see, my guess is that one of the trigger events will be ignored(only 1 sample will occur).

ziv123 said:

2. where this beeing my init:

Please use the "Insert->Code" option, to drastically increase readability.
At first glance you code looks alright to me, but I notice that you are configuring your channels are single ended inputs - instead of double ended? From earlier I think you said you wanted to use double ended(with an external reference?).
I also wonder whether you intend for all the 3 sampling channels to have the same configurations?
I also note that your saadc callback seems very sparse.
The limit event will contain both channel that triggered the event, and the type of limit that was exceeded, as can be seen here in the limit event documentation. 
For an example on how to set up and use the battery level service(if you are intending to do so), you can see it demonstrated in the ble_peripheral proximity example.

ziv123 said:

i need to know somehow wich pin triggered the event and if it was high limit or low limit ??? how can i do that ?? 

 As stated in my initial comment, LPCOMP is intended for use on one pin - which means you will know which pin that triggered the event every time it happens. The challenge with using LPCOMP for multiple pins is the constant reconfiguration. Please see this ticket for further detail.

ziv123 said:

does that means that when limits enable then the saadc works all the time, or just when triggered via some timer for example? 

 The sampling will still only occur when the SAMPLE TASK is triggered. The limit events will trigger based on the value of the acquired sample.

ziv123 said:

will each limits event will raise the NRFX_SAADC_EVT_LIMIT event type event followed by NRF_DRV_SAADC_EVT_DONE event type ?

The DONE event will occur once the sampling completes. The LIMIT events may trigger whenever a new sample is gathered, that exceeds the limits. So the order would be DONE then LIMIT. You will get an event for each channel, if there are multiple LIMIT events.

ziv123 said:

sorry for so many questions

 No problem at all! I am happy to help.

ziv123 said:

help to configure an event for each channel ?

I would not recommend mixing HAL and driver function calls in your application, since this may unintentionally put the driver into an invalid state. What happens when you build and run the code you included above?

Best regards,
Karl

[ziv123]

Karl Ylvisaker said:

Do I understand you correctly that you are intending to use the one SAADC channel to read the battery voltage of the device, and the two others to compare the pin levels to a reference?

to be clear: 

i want to use ont pin to read analog values of battery every few seconds. 

on 2 other pins i have 2 optical sensors that gives analog inputs and i need to do something when each of them goes down from a ref v and up again.

Karl Ylvisaker said:

What I said in my last comment was that double sampling(having two PPI events trigger the SAMPLE TASK at the same time) is undefined behavior - meaning it is not specified what happens if this occurs. You could test it out and see, my guess is that one of the trigger events will be ignored(only 1 sample will occur).

i can not take the risk of some event allwys or sometimes ignored, so this is not a vlide solution for me.

Karl Ylvisaker said:

you are configuring your channels are single ended inputs - instead of double ended? From earlier I think you said you wanted to use double ended(with an external reference?).

i am planing to use some internal referance (some value i will hard code) not external.

 

Karl Ylvisaker said:

The DONE event will occur once the sampling completes. The LIMIT events may trigger whenever a new sample is gathered, that exceeds the limits. So the order would be DONE then LIMIT. You will get an event for each channel, if there are multiple LIMIT events.

strange .. i read somewherer in this forom that the DONE event is triggered after data has been writen to RAM via easyDMA and the LIMIT event is triggered once the sempeled value exceeds a value in a register and this comparison happens before data is passed to the RAM hanse the LIMIT event will be triggered first.. is this wrong ?

also, now when i use BURST and i read 3 channels is it a new event for each channel because it seems to be one call of the event handler for all 3 ? 

another question is - is it possible to disable a certain limit event in the handler?  .. because after, for example, i got high limit event i don't want to get it again every read but i will want to enable it again once i get low limit event ?

Karl Ylvisaker said:

I would not recommend mixing HAL and driver function calls in your application, since this may unintentionally put the driver into an invalid state. What happens when you build and run the code you included above?

not sure i allways know which functions are HAL and which drv 

anyway and maybe it is related.. can i use the app_timer in my app and also the '

nrf_drv_timer_t' timer (all exampels i saw that use ppi use the ' nrf_drv_timer_t ' but on other modules in my program i use the app_timer - which i guess picks the channel itself and i don't know which channel and can it be interfered with the ' NRF_DRV_TIMER_INSTANCE(channel); ' ?

or is there any example on how to use ppi with app_timer ? 

p.s. if i understand correctly from what you wrote and what i have read in the forom it is not a good idea to use the LPCOMP with 2 changing channels, sounless i missunderstood i am working on saadc only now (3 channels as i mentioned iand i guess i have no choice but to read them all at the same time ) 

best regards

Ziv

[ziv123]

well i am trying to set a timer for saadc triggered via pi, i used app_timer in another module in my program but app_timer has no API to get saadc samle event address so ..

i read the RTC0 is used by the softdevice and RTC1 is used by app_timer so i can use RTC2 for the 

' static const nrf_drv_timer_t m_timer = NRF_DRV_TIMER_INSTANCE(2); '

but i get a compilation error - 

'NRFX_TIMER2_INST_IDX' undeclared here (not in a function); did you mean 'NRFX_TIMER_INSTANCE'?

/** @brief Macro for creating a timer driver instance. */
#define NRFX_TIMER_INSTANCE(id)                                   \
{                                                                 \
    .p_reg            = NRFX_CONCAT_2(NRF_TIMER, id),             \
    .instance_id      = NRFX_CONCAT_3(NRFX_TIMER, id, _INST_IDX), \
    .cc_channel_count = NRF_TIMER_CC_CHANNEL_COUNT(id),           \
}

#ifndef __NRFX_DOXYGEN__
enum {
#if NRFX_CHECK(NRFX_TIMER0_ENABLED)
    NRFX_TIMER0_INST_IDX,
#endif
#if NRFX_CHECK(NRFX_TIMER1_ENABLED)
    NRFX_TIMER1_INST_IDX,
#endif
#if NRFX_CHECK(NRFX_TIMER2_ENABLED)
    NRFX_TIMER2_INST_IDX,
#endif
#if NRFX_CHECK(NRFX_TIMER3_ENABLED)
    NRFX_TIMER3_INST_IDX,
#endif
#if NRFX_CHECK(NRFX_TIMER4_ENABLED)
    NRFX_TIMER4_INST_IDX,
#endif
    NRFX_TIMER_ENABLED_COUNT
};
#endif

i wonder why ???

.. i have tried to edit the sdk_config file enableing 

// <e> NRFX_TIMER_ENABLED - nrfx_timer - TIMER periperal driver
//==========================================================
#ifndef NRFX_TIMER_ENABLED
#define NRFX_TIMER_ENABLED 1
#endif
// <q> NRFX_TIMER0_ENABLED  - Enable TIMER0 instance
 

#ifndef NRFX_TIMER0_ENABLED
#define NRFX_TIMER0_ENABLED 0
#endif

// <q> NRFX_TIMER1_ENABLED  - Enable TIMER1 instance
 

#ifndef NRFX_TIMER1_ENABLED
#define NRFX_TIMER1_ENABLED 0
#endif

// <q> NRFX_TIMER2_ENABLED  - Enable TIMER2 instance


#ifndef NRFX_TIMER2_ENABLED
#define NRFX_TIMER2_ENABLED 1
#endif

Best regards

Ziv