nRF51822 Processing of received UART data

RE: nRF51822 Processing of received UART data

Rafaello — Tue, 16 Jul 2019 20:51:40 GMT

I got the solution!
My GSM module had enabled echo function. When I send AT command it repeat after me then send answer.
When I disable echo function everything work fine.

RE: nRF51822 Processing of received UART data

Rafaello — Mon, 15 Jul 2019 08:12:47 GMT

Hi, Overbekk.

Yes, you understand it correct. I can find the string but was thinking if is any better way than mine.

And the main problem is with with data.

Your way to clean buffer is much better so thanks for it.

Nevertheless problem is still here.

I checked it with "simple LED debuger"

I was sending "AT" command in the loop. GSM module should respond with OK. I set:

If get OK toggle 1st LED.

else toggle 2nd LED.

And i found something like that:

After first AT command nRF toggled second LED (something different than OK), at second AT command toggled 1 LED (OK) . And it was repeated. First 2nd LED then 1st LED and again 2nd LED again 1st LED and again and again.

It looks like something is wrong with command sending or with reciving function.

To send command I used:

app_uart_put('A');

app_uart_put('T');

app_uart_put('\r');

app_uart_put('\n');

So sending should be fine.

Thanks in advance

Rafaello

RE: nRF51822 Processing of received UART data

ovrebekk — Fri, 12 Jul 2019 11:55:39 GMT

Hi Rafaello

If I understand you correctly the issue is not to find the +CGNSPWR=0 string, but that the data in the finale buffer is not correct after reception?

The way you 'clean' the buffer seems overly complicated in my opinion. It should be sufficient either to just clear all the 100 bytes before you copy data to finale, or to clear the remaining bytes of finale after you have copied the data. Also, I would expect you to want to clean the bytes that you don't overwrite with data, there is no point clearing the bytes that you are about to overwrite anyway.

I would expect it to look something like this:

for (uint32_t i = 0; i < count; i++)
{
finale[i] = data[i];
}

for(uint32_t i = count; i < 100; i++)
{
finale[i] = 0;
}

And if you still have issues I would suggest putting some logging into the function so that you can see exactly which bytes are received at any one time.

Best regards
Torbjørn