nrfx_saadc advanced mode oversampling

RE: nrfx_saadc advanced mode oversampling

picatostas — Wed, 10 Nov 2021 10:43:11 GMT

Hi Edvin,

I have been using the example https://github.com/NordicSemiconductor/nrfx/wiki/SAADC-Advanced-mode-with-IRQs

but I am struggling t get it working when I want to stop and continue the conversion at some point.

my init funcition is the same, with some difference based on my application, 500 samples instead of 1000 and my sampling freq is 10kHz:

static void adc_init(void) {
    ret_code_t err_code;
    nrfx_saadc_channel_t tuner_ch   = NRFX_SAADC_DEFAULT_CHANNEL_SE(GUITAR_IN_ADC_CH, 0);
    nrfx_saadc_adv_config_t adc_cfg = {
        .oversampling      = NRF_SAADC_OVERSAMPLE_DISABLED, // Disable oversample.
        .burst             = false,                         // Disable burst.
        .internal_timer_cc = 16000000UL/ADC_SAMPLE_RATE,    // Sample at 10kHz.
        .start_on_end      = false,                         // Latch next buffer at END event in interrupt.
    };
    err_code = nrfx_saadc_init(NRFX_SAADC_CONFIG_IRQ_PRIORITY);
    APP_ERROR_CHECK(err_code);
    err_code = nrfx_saadc_channels_config(&tuner_ch, 1);
    APP_ERROR_CHECK(err_code);
    err_code = nrfx_saadc_advanced_mode_set((1<<0), NRF_SAADC_RESOLUTION_10BIT, &adc_cfg, adc_evt_handler);
    APP_ERROR_CHECK(err_code);
    err_code = nrfx_saadc_buffer_set(adc_buf[buf_to_load % 2], SAMPLES_NUMBER);
    APP_ERROR_CHECK(err_code);
    buf_to_load++;
}

and whenever I want to get a set of samples i do:

my event handler looks like this, I modified it a bit for the sake of my application.

static void adc_evt_handler(nrfx_saadc_evt_t const * p_event) {
    switch (p_event->type) {
        case NRFX_SAADC_EVT_DONE: {
            // Buffer with data is available here: p_event->data.done.p_buffer
            adc_samples = p_event->data.done.size;
            for (uint32_t i = 0; i < adc_samples; i++) {
                guitar_samples[i] = p_event->data.done.p_buffer[i];
            }
            eventq_add(EVENT_TUNER_ADC_READY, 0, NULL);
        }
        break;
        case NRFX_SAADC_EVT_BUF_REQ:
            nrfx_saadc_buffer_set(adc_buf[buf_to_load % 2], SAMPLES_NUMBER);
            buf_to_load++;
            break;
        default:
            break;
    }
}

The first time I get the samples correctly. From what I understand since I configured the adc

with :

        .start_on_end      = false,                         // Latch next buffer at END event in interrupt.

Once the event :

NRFX_SAADC_EVT_DONE

is published ADC block wont convert anymore.

If call again

 ret_code_t err_code = nrfx_saadc_mode_trigger();
 APP_ERROR_CHECK(err_code);

error_code is 8.

What is the proper way of utilizing the adc to start stop samples on demand without having the ADC running non_stop ?

RE: nrfx_saadc advanced mode oversampling

picatostas — Tue, 09 Nov 2021 21:15:30 GMT

The mentioned example worked for me, thanks

RE: nrfx_saadc advanced mode oversampling

mafi — Thu, 27 Aug 2020 12:25:21 GMT

Hi Edvin,

I have been using the double buffering implementation quite succesfull, however, I am now trying to run the ISR on a lower priority level since it is conflicting with some other system interrupts (radio).

Whenever I lower the priority my initial problem is popping up again: weird outliers. Do you have any idea what's the cause?

Best regards,

Mattia

RE: nrfx_saadc advanced mode oversampling

mafi — Mon, 06 Jul 2020 14:21:09 GMT

Hi Edvin,

Many thanks for your answer, I could not find any example code. I have not yet tried this but I am certain this will resolve my question, I have verified your answer.

Best regards,

Mattia

RE: nrfx_saadc advanced mode oversampling

Edvin — Wed, 01 Jul 2020 13:29:05 GMT

Hello Mattia,

I am sorry for the late reply.

I didn't realize the saadc drivers from NCS were so different from the nRF5 SDK drivers.

If you look at the advanced mode example implementation found here, you can see that there are some differences in the event handler. Look at how they handle the p_event->data.done.p_buffer[].

Also, they don't set the buffer in NRFX_SAADC_EVT_DONE, but in the NRFX_SAADC_EVT_BUF_REQ event.

They also use the double buffer. sample0 and sample1, both containing SAMPLES_NUMBER (1000) elements.

What does your saadc configuration look like?

nrfx_saadc_adv_config_t config = ?

I suggest you try to use double buffering, like it is done in the example from the link. You don't need to for-loop in the event handler. It can be done elsewhere to free up the buffer (in case you are working on your buffer, and another ADC_DONE event occurs before you are done.

BR,
Edvin

RE: nrfx_saadc advanced mode oversampling

mafi — Fri, 26 Jun 2020 14:38:51 GMT

Hi Edvin,

Thank you for the response I will try with proper double buffers I think here lies the problem.

Regarding the implementation that I'm using: it is a fork of the official HAL from Nordic here https://github.com/NordicSemiconductor/nrfx/tree/v2.2.0 .

This HAL does not have the function nrf_drv_saadc_buffer_convert, which is why I'm using the buffer_init inside my IRQ in stead.

Is this then the correct approach? Or what should I replace nrf_drv_saadc_buffer_convert with based on this HAL?

Best regards,

Mattia

RE: nrfx_saadc advanced mode oversampling

Edvin — Fri, 26 Jun 2020 14:11:41 GMT

Hello,

If you compare your implementation with the saadc from the nRF5 SDK example:

SDK\examples\peripheral\saadc\

which uses a double buffer.

In fact, I think you should take a look at this example. I don't understand why you use nrf_drv_saadc_buffer_convert() in your NRFX_SAADC_EVT_DONE event. What you should use is nrf_drv_saadc_buffer_convert().

Did you find this implementation somewhere? I am sorry if that is a stupid question. I have not worked much with NCS (the zephyr SDK) yet. Check out the implementation in the saadc example from the nRF5 SDK.

Even though your implementation is working, it may be that when you set the single buffer in each interrupts, it may act a bit weird if it doesn't have time to do this on time (perhaps it is blocked by a BLE event some times?). The values that you say that you see sometimes looks like multiples of ~3785

Try to use the double buffers, and use nrf_drv_saadc_buffer_convert() in the interrupt, and see if it still glitches like this.

Best regards,

Edvin

RE: nrfx_saadc advanced mode oversampling

mafi — Fri, 26 Jun 2020 10:40:24 GMT

Sample data that I get:

With an internal ADC reference and 1/6 Gain, the DC offset is approx 0.81Volt, however, the outlier of 15112 I can't explain as this would mean a peak of 3.5 Volt which is highly unlikely.