Issue sending data from UART over mesh

RE: Issue sending data from UART over mesh

Arij — Wed, 04 Jul 2018 15:00:54 GMT

Here's what happen when I call send_my_message(), there is a softdevice assert and a warning: mesh_softdevice_init.c, 104, sd_ble_enable: app_ram_base should be adjusted to 0x200032C8

For the setup __LOG, I'am not sure if you talk about this

or the setup in sdk_config.h, actually I added the setup_config.h of the uart_example and add some defines from sdk_config.h of client example because I delete it from the project but in both files there is no defines for rtt or uart log enable, there is only nrf_log_enable which is set to 0

If I comment out printf, I don't receive any data on the other node and still have the same problem

RE: Issue sending data from UART over mesh

Hung Bui — Wed, 04 Jul 2018 13:38:07 GMT

Can you check what when wrong when you call send_my_message() ? any assertion ?

Did the application run normally after that or it crashed ?

How did you setup __LOG ? via RTT or UART ?
If you printing out log via UART you will have a conflict with your printf.

If you comment out printf do you receive the data on the other node in mesh ?

RE: Issue sending data from UART over mesh

Arij — Wed, 04 Jul 2018 12:58:56 GMT

Hi again,

I want the data that I receive on UART (string data) to send it over mesh, I tested with array of bytes as you said and it works, but in my case I want to get any data sent by pc terminal(termite) to send it over mesh, I did a lot of tests and nothing wrong with mesh, now I switch using this function to get data from UART:

void uart_event_handle(app_uart_evt_t * p_event)
{
    if (p_event->evt_type == APP_UART_DATA_READY)
    {
      while (app_uart_get(&cr) != NRF_SUCCESS);
        s[Index++] =  cr;
        s[Index] = 0;
        if(cr == '\n')
        {
            printf("RX end : %s\r\n", s);   //show nothing     
            send_my_message(s); //fail here 
            Index = 0;
        }
     }
    
    if (p_event->evt_type == APP_UART_COMMUNICATION_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_communication);
    }
    else if(p_event->evt_type == APP_UART_FIFO_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_code);
    }
}

So this function works fine when I don't add send_my_message() function as shown in line 11 above in code, but when add it, it stuck and the function shows nothing

here is result without send_my_message() function

and here with send_my_message() function

send_my_function() is mentioned in my previous comment, please help

Thank you

RE: Issue sending data from UART over mesh

Hung Bui — Thu, 28 Jun 2018 11:26:46 GMT

Hi Arji,

Please try testing with sending an array of bytes. Don't play with string first.

You can try to send something like

uint8_t data_array = {1,2,3,4,5}

And check on the other side if you receive 1 2 3 4 5 array or not. To print an array out , you can use a loop and print each element out.

Main different between a string and byte array is the terminate character. In C it's the NULL character (0). But I don't think this char sent over mesh because of the length.

RE: Issue sending data from UART over mesh

Arij — Wed, 27 Jun 2018 16:32:55 GMT

Now this function __LOG(LOG_SRC_ACCESS, LOG_LEVEL_DBG1, "TX: Msg %s\n", p_tx_message->p_buffer) work for printing the message, I don't know why it doesn't at first when I added it, but now my problem is when I send data from pc (termite terminal) to my board and send it over mesh, It will be sent like this

this is the log of the sending side

here what I receive on the othe side

and here is a snippet of my code

for (;;)
    {
        (void)sd_app_evt_wait();
       // uint8_t cr;
        char s[5];

      uint32_t status = NRF_SUCCESS;

      // while (app_uart_get(&cr) != NRF_SUCCESS);
      // while (app_uart_put(cr) != NRF_SUCCESS);

     uart_getstring(s);  
     uart_putstring(s);

     printf(" message : %s \n ", s);
      
     send_my_message(s); 

    }

This is my send_my_message() function

void send_my_message (uint8_t* s) 
{
    uint32_t status=0;
    uint8_t buffer[5];
    strcpy(buffer, s);                         
    uint8_t length;
    access_message_tx_t msg;
    length= sizeof(buffer);
             if (length)
                { 
                  msg.opcode.opcode = simple_message_OPCODE_SEND;
                  msg.opcode.company_id = 0x0059; // Nordic's company ID
                  msg.p_buffer = (const uint8_t *) &buffer[0];
                  msg.length =length;
                  
                 status= simple_message_client_set_unreliable(&m_clients[0], buffer, length);
                 __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Status : %u \n", status);
                        
                
                  if (status == NRF_ERROR_INVALID_STATE ||
                  status == NRF_ERROR_BUSY||status == NRF_ERROR_NO_MEM)
                   {
                     __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Cannot send. Device is busy.\n");
                      hal_led_blink_ms(LEDS_MASK, 50, 4);
                   }
                   else
                   {
                         ERROR_CHECK(status);
                   }
                }
}

The string data is sent separately, please help

Thank you

RE: Issue sending data from UART over mesh

Hung Bui — Wed, 27 Jun 2018 15:25:49 GMT

How do you check on the other side if you are receiving correct data ?

The way you printing the message out is not very correct. It's a byte array, not a string. It's better to do a loop to print out every byte of the array instead of doing that. And please do the same with the RX on the other side. You may want to print the payload length out also.

RE: Issue sending data from UART over mesh

Arij — Tue, 26 Jun 2018 15:21:28 GMT

I added the call __LOG(LOG_SRC_ACCESS, LOG_LEVEL_DBG1, "TX: Msg %s \n", p_tx_message->p_buffer) to packet_tx() function in access.c, but it doesn't show my messages, I'm not sure if I'm doing it right,

I verified my mesh functions and they work normally, I'am able to send and receive my messages over mesh from server to client and vice versa when I press a button, but not the messages that I want to send from UART.

Yes I tested that example combining light switch example SDK v2.0.1 and ble_app_uart sdk v15 and I'am using it on the server side in order to get the data received from the phone and put it on mesh but I'm struggling on how to acheive that. Could you please check my code above and see if it's correct, thank you

RE: Issue sending data from UART over mesh

Hung Bui — Tue, 26 Jun 2018 13:20:16 GMT

Hi Arij,

I would suggest to set the logging debug level to LOG_LEVEL_DBG1 when init logging module, and in mesh_msg_handle() in access.c you print out more information in the call __LOG(LOG_SRC_ACCESS, LOG_LEVEL_DBG1, "RX: [aop: 0x%04x]\n", opcode.opcode); so that it not only printout opcode but also other information such as message src, dst, length, buffer, etc.

You can do the same inside packet_tx() as well.

By doing this you can monitor all message processed at the node.

Have you verified that you mesh functions normally ? You can control LED on and off from your client ?

How did you check that your send_my_message() sends random data ? Please try to send just one byte first.

We posted an example here combining light switch example SDK v2.0.1 and ble_app_uart for SDK v15 not sure if needed for you though.

RE: Issue sending data from UART over mesh

Arij — Tue, 26 Jun 2018 01:28:35 GMT

Update I used uart_event_handle function instead of uart_getstring and uart_putstring but still not able to send data over mesh

I only modified uart_error_handle() and commented uart_putstring and uart_getstring

void uart_error_handle(app_uart_evt_t * p_event)
{
    //added 
    if (p_event->evt_type == APP_UART_DATA_READY)
    {
      while (app_uart_get(&cr) != NRF_SUCCESS);
        s[Index++] =  cr;
        s[Index] = 0;
        if(cr == '\n')
        {
            printf("RX end : %s\r\n", s);
            
            send_my_message(s); //failed to pass s value to this function
           
            Index = 0;
        }
    }
    
    else if (p_event->evt_type == APP_UART_COMMUNICATION_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_communication);
    }
    else if(p_event->evt_type == APP_UART_FIFO_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_code);
    }
}

//send_my_message function

void send_my_message (uint8_t* s)                          
{
    uint32_t status=0;
    uint8_t buffer[10];
    strcpy(buffer, s);                          
    uint8_t length;
    uint16_t address;
    access_message_tx_t msg;
    length= sizeof(buffer);
               if (length)
                { 
                  msg.opcode.opcode = simple_message_OPCODE_SEND;
                  msg.opcode.company_id = 0x0059; // Nordic's company ID
                  msg.p_buffer = (const uint8_t *) &buffer[0];
                  msg.length =length;
                 status= simple_message_client_set_unreliable(&m_clients[0], buffer, length);
                 __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Status : %u \n", status);
                        

                  if (status == NRF_ERROR_INVALID_STATE ||
                  status == NRF_ERROR_BUSY||status == NRF_ERROR_NO_MEM)
                   {
                     __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Cannot send. Device is busy.\n");
                      hal_led_blink_ms(LEDS_MASK, 50, 4);
                   }
                   else
                   {
                         ERROR_CHECK(status);
                   }
                }
}