RE: UART behaviour changed automatically, numbers sent over RX Characteristic are not received the same on other side.

Kenneth — Wed, 10 Mar 2021 18:11:11 GMT

It could be code optimization that have changed, I suspect you will need to check with an embedded C expert if you wanted to dig into the details, I could at least find that the comparison you did had some issues; that both int10 and int100 and int1000 are all equal int10 if the comparison check the first 5 ascii characters.

Kenneth

RE: UART behaviour changed automatically, numbers sent over RX Characteristic are not received the same on other side.

IMFROST — Wed, 10 Mar 2021 16:05:55 GMT

I think the issue is that the codes that I am sending from nrF connect app are being sent in HEX to the device instead of string. So the zeroes at the end of each code mess up the outcome.

So I did a workaround by adding alphabets to the end of each command and now it works.

static void nus_data_handler(ble_nus_evt_t * p_evt)       
{

    if (p_evt->type == BLE_NUS_EVT_RX_DATA)
    {
        NRF_LOG_DEBUG("Received data from BLE NUS.");
        NRF_LOG_HEXDUMP_INFO(p_evt->params.rx_data.p_data, p_evt->params.rx_data.length);

        if((memcmp(p_evt->params.rx_data.p_data,"dfu", p_evt->params.rx_data.length) == 0))
        {
        enter_dfu_mode();
		    }

        if((memcmp(p_evt->params.rx_data.p_data,"name", p_evt->params.rx_data.length) == 0))
        {
        nflag = 1;
		    }


        if((memcmp(p_evt->params.rx_data.p_data,"tx0", p_evt->params.rx_data.length) == 0))
        {
        
	TX_POWER_LEVEL = (0);
            }
        if((memcmp(p_evt->params.rx_data.p_data,"tx4", p_evt->params.rx_data.length) == 0))
        {
        TX_POWER_LEVEL = (+4);
        cflag = 1;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"txm20", p_evt->params.rx_data.length) == 0))
        {
        TX_POWER_LEVEL = (-20);
        cflag = 1;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"txm12", p_evt->params.rx_data.length) == 0))
        {
        TX_POWER_LEVEL = (-12);
        cflag = 1;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"txm8", p_evt->params.rx_data.length) == 0))
        {
        TX_POWER_LEVEL = (-8);
        cflag = 1;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"txm4", p_evt->params.rx_data.length) == 0))
        {
        TX_POWER_LEVEL = (-4);
        cflag = 1;
		    }


        if((memcmp(p_evt->params.rx_data.p_data,"int100ms", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 160;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int300ms", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 480;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int500ms", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 800;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int800ms", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 1280;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int1sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL= 1600;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int1.5sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 2400;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int2sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL= 3200;
        }
        if((memcmp(p_evt->params.rx_data.p_data,"int3esc", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 4800;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int4sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 6000;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int5sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 8000;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int6sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 9600;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int7sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 11200;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int8sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 12800;
		    }
        if((memcmp(p_evt->params.rx_data.p_data,"int9sec", p_evt->params.rx_data.length) == 0))
        {
        cflag = 1;
        APP_ADV_INTERVAL = 14400;
		    }
    }
}

This is not a solution for the current problem. I remember sending codes like "int1000" and they were accepted before Jan 2021 but now something has happened and those same codes are not being accepted. Typing int100 activates the statement for "int10000" instead.

RE: UART behaviour changed automatically, numbers sent over RX Characteristic are not received the same on other side.

Kenneth — Wed, 10 Mar 2021 14:42:41 GMT

I assume this is pure luck/unluck/undefined with your current implementation.

To compare length I would assume you need to check that 'p_evt->params.rx_data.length' equal 'sizeof("int100")'.

Kenneth

RE: UART behaviour changed automatically, numbers sent over RX Characteristic are not received the same on other side.

IMFROST — Wed, 10 Mar 2021 14:31:32 GMT

Do you think something changed in the nRF connect app so now those values are different? Because when the factory was shipping the pcb around Mid December 2020 the codes were working but now they don't work. Just having problem when sending zeroes.

How do I check the length of the strings.

RE: UART behaviour changed automatically, numbers sent over RX Characteristic are not received the same on other side.

Kenneth — Wed, 10 Mar 2021 14:07:00 GMT

You should also check that the length of the two strings you want to compare are the same.

Kenneth