Issue with Sending generic level messages from main()

[DeveloperZ]

Hi,

I am working with a motion sensor which sends a message when it detects a motion. I am communicating between the nRF52 DK and the sensor via UART. I am able to receive messages correctly. I am using the experimental_dimming_client example and I tried to send a generic level message whenever the sensor detects the motion. I added the UART library and added the functionality to send generic level messages upon detecting motion in the main() as follows:

for (;;)
    {
        uint8_t cr;
        while(app_uart_get(&cr) != NRF_SUCCESS);
        while(app_uart_put(cr) != NRF_SUCCESS);

        if(cr == 'M')
        {
            button_event_handler(1);
        }

        (void)sd_app_evt_wait();
    }

M indicates motion and button event handler sends the generic level messages.

Here comes the issue:

I haven't provisioned the board yet. When a motion is detected, the board is able to execute the the button_event_handler function for case 1 which is sending a set message.

Now I provisioned the device(didn't set the publication address). For these two situations, output looks like this:

When I add the publication address, it asserts an error: 

app_error_weak.c,  108, Mesh assert at 0x0002AEC6 (:0)

After this, I went back to the stock SDK and tried to send generic level messages from the main function (without UART) which resulted in a Mesh error 15 (Forbidden Command)

I have been trying to debug and find why I am unable to publish messages to an address from the main() but I am not successful yet.

What might be the reason? Any ideas on how to resolve it?

Thank you.

[pirast]

Actually, I am running into the same issue. I want to shoot off  button_event_handler(1) not from rtt_input_handler callback but from another function (e.g. main). When I do that, I get the same mesh assert. Interestingly, the first time it works (after provisioning). But after power cycling it (settings are loaded), this does not work and it gives the mesh error.

I have tracked it down to net_state_seqnum_alloc where m_net_state.seqnum is = m_net_state.seqnum_max_available. Possibly this appears based on from which context the send function is run?

[pirast]

FYI I think I have found a solution; I am using Mesh SDK 2.2. Tracking down the mesh assert address, I found NRF_MESH_ASSERT_DEBUG(bearer_event_in_correct_irq_priority()); in timer_scheduler.c to be the critical section. After some research, I found https://devzone.nordicsemi.com/f/nordic-q-a/41645/error-nrf_error_invalid_state-when-send-data-via-mesh-3-0/163725 respectively https://devzone.nordicsemi.com/f/nordic-q-a/38479/mesh-sdk-2-2-0-bug-in-mesh_opt_core_adv_set/148935#148935 . Nesting the message publish code in sd_nvic_critical_region_enter (answer at the bottom) solves this for me.

[DeveloperZ]

Hi,

Thanks for the reply. I tried making the changes to the mesh_stack.c file mentioned in the links above. I even tried calling mesh_opt_core_adv_set() function inside static void start(void) function only before mesh_stack_start() call and also in the mesh_stack.c between calls sd_nvic_critical_region_enter()/_exit().

Adding these lines results in a Forbidden Command error 15 as shown below:

I also added following lines in mesh_stack.c

uint8_t is_nested;
ERROR_CHECK(sd_nvic_critical_region_enter(&is_nested));

mesh_opt_core_adv_set(CORE_TX_ROLE_ORIGINATOR, &net);

ERROR_CHECK(sd_nvic_critical_region_exit(is_nested));

Because I am using Mesh SDK 3.0.0, this solution isn't applicable. May be the support team know a better way to fix this issue.

Thanks for your response pirast.

[DeveloperZ]

If I made any error in adding the lines to the code, kindly correct me.

Thank you.

[Mttrinh]

Might be that the messages are being sent too fast, can you try waiting for the NRF_MESH_EVT_TX_COMPLETE before sending a new message?

Also, instead of running it in the for-loop you could create an interrupt that triggers whenever the sensor detects motion.

[DeveloperZ]

Mttrinh said:

instead of running it in the for-loop you could create an interrupt that triggers whenever the sensor detects motion.

That was the first thing that I did if you can see the question that I posted, I tried to send a generic level message upon receiving an indication from the sensor using UART that motion is detected.

Mttrinh said:

Might be that the messages are being sent too fast, can you try waiting for the NRF_MESH_EVT_TX_COMPLETE before sending a new message?

I might have taken it that way already if the program crashes while sending the second message after successfully transmitting the first. But, I am unable to transfer the first message itself. 

I also tried another thing, used the EnOcean switch example, instead of sending generic OnOff messages, I tried sending generic Level messages. They are transmitted without any error inside the app_switch_debounce().

Why can these messages be transmitted from app_switch_debounce() in EnOcean example, but not from the main() in any example?