Add Ble Scan to usbd_ble_uart_freertos example

[shai.c]

Hi I'm trying to change peripheral usbd_ble_uart_freertos example to central with nus_c
I added the scan and nuc_c module.
When I'm testing the application, before making a connection with the peripheral all USB events are working as expected.
After the connection is established new USB events stop working.

This is the code that is responsible for USB events

Fullscreen

1

2

3

4

5

6

7

8

9

10

11

12

13

14

15

16

17

18

19

20

21

void usb_new_event_isr_handler(app_usbd_internal_evt_t const * const p_event, bool queued)

{

BaseType_t xHigherPriorityTaskWoken = pdFALSE;

UNUSED_PARAMETER(p_event);

UNUSED_PARAMETER(queued);

ASSERT(m_usbd_thread != NULL);

/* Release the semaphore */

vTaskNotifyGiveFromISR(m_usbd_thread, &xHigherPriorityTaskWoken);

portYIELD_FROM_ISR(xHigherPriorityTaskWoken);

}

static void usbd_thread(void * arg)

{

ret_code_t ret;

static const app_usbd_config_t usbd_config = {

.ev_isr_handler = usb_new_event_isr_handler,

.ev_state_proc = usbd_user_ev_handler

};

UNUSED_PARAMETER(arg);

ret = app_usbd_init(&usbd_config);

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

void usb_new_event_isr_handler(app_usbd_internal_evt_t const * const p_event, bool queued)
{
    BaseType_t xHigherPriorityTaskWoken = pdFALSE;
    UNUSED_PARAMETER(p_event);
    UNUSED_PARAMETER(queued);
    ASSERT(m_usbd_thread != NULL);
    /* Release the semaphore */
    vTaskNotifyGiveFromISR(m_usbd_thread, &xHigherPriorityTaskWoken);
    portYIELD_FROM_ISR(xHigherPriorityTaskWoken);
}

static void usbd_thread(void * arg)
{
    ret_code_t ret;
    static const app_usbd_config_t usbd_config = {
        .ev_isr_handler = usb_new_event_isr_handler,
        .ev_state_proc  = usbd_user_ev_handler
    };
    UNUSED_PARAMETER(arg);

    ret = app_usbd_init(&usbd_config);
    APP_ERROR_CHECK(ret);
    app_usbd_class_inst_t const * class_cdc_acm = app_usbd_cdc_acm_class_inst_get(&m_app_cdc_acm);
    ret = app_usbd_class_append(class_cdc_acm);
    APP_ERROR_CHECK(ret);
    ret = app_usbd_power_events_enable();
    APP_ERROR_CHECK(ret);

    // Set the first event to make sure that USB queue is processed after it is started
    UNUSED_RETURN_VALUE(xTaskNotifyGive(xTaskGetCurrentTaskHandle()));
    // Enter main loop.
    for (;;)
    {
        /* Waiting for event */
        UNUSED_RETURN_VALUE(ulTaskNotifyTake(pdTRUE, USB_THREAD_MAX_BLOCK_TIME));
        while (app_usbd_event_queue_process())
        {
            /* Nothing to do */
        }
    }
}

It looks that events are received in usb_new_event_isr_handler function, but usbd_thread keep waiting for the event - stuck in ulTaskNotifyTake

 

[Susheel Nuguru]

Which SDK version are you using? Seems like one of the vTaskNotifyGiveFromISR is being lost making your USBD state machine stuck at wait and unable to create any new usbd events

[Susheel Nuguru]

maybe you should let the usbd event queue process to run first before waiting as I see one race condition with first waiting and then processing

Fullscreen

1

2

3

4

5

6

7

8

9

10

for (;;)

{

while (app_usbd_event_queue_process())

{

/* Nothing to do */

}

/* Waiting for event */

UNUSED_RETURN_VALUE(ulTaskNotifyTake(pdTRUE, USB_THREAD_MAX_BLOCK_TIME));

}

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

    for (;;)
    {
        while (app_usbd_event_queue_process())
        {
            /* Nothing to do */
        }

        /* Waiting for event */
        UNUSED_RETURN_VALUE(ulTaskNotifyTake(pdTRUE, USB_THREAD_MAX_BLOCK_TIME));
    }    

[shai.c]

Hi,

i tried this but it didnt help

[Susheel Nuguru]

Hmm, then the problem does not seem to be in the code usbd_isr or the usb thread. Need to understand what your application is doing in the "Connected" event. Can you give me your minimalistic project to be able to reproduce this behaviour?