Apparently two channels are colliding?

RE: Apparently two channels are colliding?

angel — Mon, 29 May 2017 15:59:18 GMT

Hi Nordic,

Petter and I found the solution to the problem. Something quite weird was happening whenever the previous function was executed, and a lot of CHANNEL_COLLISION events were happening. The solution was to change the strategy from OPEN CH0, OPEN CH1, SEND both, CLOSE CH0, CLOSE CH1 to OPEN CH0, SEND CH0, CLOSE CH0, OPEN CH1, SEND CH1, CLOSE CH1. This way we never get channel collision and everything works fine. The only drawback though is the delay in sending the message to CH1 with respect to the event in the slave, which can go to maximum around 200 - 300 ms (max timeout in CH0).

Thanks!

RE: Apparently two channels are colliding?

angel — Tue, 09 May 2017 12:24:55 GMT

Hi Petter,

I just did it, here is the identificator: Case ID 36279

Thanks!

RE: Apparently two channels are colliding?

Petter Myhre — Tue, 09 May 2017 11:38:59 GMT

If you open a case through your MyPage at nordicsemi.com you can upload it to me privately there.

RE: Apparently two channels are colliding?

angel — Tue, 09 May 2017 11:36:03 GMT

Hi Petter,

Can I send it to you privately?

Thanks!

RE: Apparently two channels are colliding?

Petter Myhre — Tue, 09 May 2017 11:17:04 GMT

Could you share your complete projects (zip them) so I can compile and test them here?

RE: Apparently two channels are colliding?

angel — Tue, 09 May 2017 10:30:50 GMT

Hi Petter, thanks for the fast reply. Here you have the piece of code I use to send the message:

void send_message_and_get_ack(message_t* p_message, channel_t* p_channel_0, channel_t* p_channel_1)
{
    if(p_message->current_status == PROTOCOL_STATUS_DISABLED)
    {
        p_message->payload = PROTOCOL_STATUS_ENABLED;
    }
    else if(p_message->current_status == PROTOCOL_STATUS_ENABLED)
    {
        p_message->payload = PROTOCOL_STATUS_DISABLED;
    }

    static uint64_t time_start = 0; //variable for timeout

    if(!is_front_paired() || is_only_rear_message(p_message))
    {
        open_channel(p_channel_0);
        static uint8_t ack_retries = 0; // counter for ack retries
        // send and get ack from rear
        send_message(p_channel_0, p_message);
        bool message_failed = false;
        while(p_message->class != p_channel_0->received_ack)
        {
            if((get_timer_count() - time_start) > messages_defined.ack_timeout)
            {
                time_start = get_timer_count();
                ack_retries++;
                send_message(p_channel_0, p_message);
                if(ack_retries > MAX_ACK_RETRIES)
                {
                    message_failed = true;
                    break;
                }
            }
        }
        time_start = get_timer_count();
        ack_retries = 0;
        p_channel_0->received_ack = 0x00;
        if(!message_failed)
        {
            p_channel_0->master_present = 0x01;
            toggle_message_status(p_message, &messages_defined);
        }
        else                    //failed, what do? go to status OFF?
        {
            p_message->current_status = PROTOCOL_STATUS_DISABLED;
        }
        close_channel(p_channel_0);
    }
    else
    {
        static uint8_t ack_retries_0 = 0; // counter for ack retries
        static uint8_t ack_retries_1 = 0; // counter for ack retries
        bool message_failed = false;

        open_channel(p_channel_0);
        open_channel(p_channel_1);
        send_message(p_channel_0, p_message);
        send_message(p_channel_1, p_message);

        bool message_failed_0 = false;
        bool message_failed_1 = false;

        bool message_succeed_0 = false;
        bool message_succeed_1 = false;

        while(true)
        {
            if(p_message->class == p_channel_0->received_ack)
            {
                p_channel_0->master_present = 0x01;
                message_succeed_0 = true;
            }
            if(p_message->class == p_channel_1->received_ack)
            {
                p_channel_1->master_present = 0x01;
                message_succeed_1 = true;
            }
            if((get_timer_count() - time_start) > messages_defined.ack_timeout)
            {
                time_start = get_timer_count();

                if(p_message->class != p_channel_0->received_ack && !message_succeed_0)
                {
                    ack_retries_0++;
                    send_message(p_channel_0, p_message);
                }

                if(p_message->class != p_channel_1->received_ack && !message_succeed_1)
                {
                    ack_retries_1++;
                    send_message(p_channel_1, p_message);
                }

                if(ack_retries_0 > MAX_ACK_RETRIES)
                {
                    message_failed_0 = true;
                }

                if(ack_retries_1 > MAX_ACK_RETRIES)
                {
                    message_failed_1 = true;
                }

                if(message_failed_0 && message_failed_1)
                {
                    message_failed = true;
                    break;
                }
            }
            if((message_succeed_0 && message_succeed_1) ||
               (message_succeed_0 && message_failed_1) ||
               (message_succeed_1 && message_failed_0) ||
               (message_failed_0 && message_failed_1)
                )
            {
                break;
            }
        }
        time_start = get_timer_count();
        ack_retries_0 = 0;
        ack_retries_1 = 0;
        p_channel_0->received_ack = 0x00;
        p_channel_1->received_ack = 0x00;

        if(message_succeed_0 || message_succeed_1) //only if one of them, just to allow everything
        {
            toggle_message_status(p_message, &messages_defined);
        }
        else
        {
            p_message->current_status = PROTOCOL_STATUS_DISABLED;
        }
        close_channel(p_channel_0);
        close_channel(p_channel_1);
    }
}

RE: Apparently two channels are colliding?

Petter Myhre — Tue, 09 May 2017 09:01:19 GMT

Strange. I'm not sure what could cause this. Would it be possible to share your complete projects so I can test them here? Or simple projects that reproduces the behavior?