USB CCID class with nordic nRF52840

[Mohammad]

I am developing USB CCID driver class for nordic nRF52840. I used USB CDC-ACM class as reference and tried to implement for USB CCID. I have developed the device descriptor for it.

USB CCID class has 1 interface and 3 endpoints(1 for interrupt, 1 for bulk IN and 1 for bulk OUT). Now the windows is detecting as Microsoft Usbccid smart card reader(WUDF)

In modify the app_usbd_ccid_internal.h, i need to know modification to be done in 

1. class instance data (app_usbd_ccid_inst_t)

2. CCID class specific request handled via control endpoint (app_usbd_ccid_req_t)

3.  ccid context (app_usbd_ccid_ctx_t)

I have attached my app_usbd_ccid_internal.h file with tis for refrence.

app_usbd_ccid_internal.h

[Håkon Alseth]

Hi,

 

This is a windows feature, that you cannot fix in firmware.

See a similar issue here, just for the USBD_CDC_ACM: https://devzone.nordicsemi.com/f/nordic-q-a/52438/usb-cdc-uart-is-recognized-as-teensy-com-port

If you have another driver, you can manually update the driver, as described in the above link.

 

Kind regards,

Håkon

[Mohammad]

Hi Håkon,

This is not the answer for the issue i described above.

Pls check above issue i have described. 

[Mohammad]

Hi,

Yes, it is  p_event->app_evt.type.

NRF_LOG_INFO("ccid_event_handler = %d", p_event->app_evt.type);

 Is it stuck inside there, or just not processing any events? It sounds like there's some CCID specific events that are not implemented?

Yes, it processes the events.

[Håkon Alseth]

Mohammad said:

 Is it stuck inside there, or just not processing any events? It sounds like there's some CCID specific events that are not implemented?

Yes, it processes the events.

Have you checked if the CCID class documentation has specific setup requests? Because it looks like there are unhandled/unimplemented cases in your implementation of the class.

 

Kind regards,

Håkon

[Mohammad]

Hi,

I have a problem with static ret_code_t setup_req_std_in(app_usbd_class_inst_t const * p_inst,
app_usbd_setup_evt_t const * p_setup_ev)

static ret_code_t setup_req_std_in(app_usbd_class_inst_t const * p_inst,
app_usbd_setup_evt_t const * p_setup_ev)
{
  NRF_LOG_INFO("setup_req_std_in ");

  /* Only Get Descriptor standard IN request is supported by CCID class */
  if ((app_usbd_setup_req_rec(p_setup_ev->setup.bmRequestType) == APP_USBD_SETUP_REQREC_DEVICE) &&
(p_setup_ev->setup.bRequest == APP_USBD_SETUP_STDREQ_GET_DESCRIPTOR)) 
  {
     NRF_LOG_INFO("Inside setup_req_std_in ");

     size_t dsc_len = 0;
     size_t max_size;

     uint8_t * p_trans_buff = app_usbd_core_setup_transfer_buff_get(&max_size);

     /* Try to find descriptor in class internals*/
     ret_code_t ret = app_usbd_class_descriptor_find(p_inst, p_setup_ev->setup.wValue.hb,
p_setup_ev->setup.wValue.lb, p_trans_buff, &dsc_len);

      if (ret != NRF_ERROR_NOT_FOUND) 
     {
          ASSERT(dsc_len < NRF_DRV_USBD_EPSIZE);
          return app_usbd_core_setup_rsp(&(p_setup_ev->setup), p_trans_buff, dsc_len);
      }
   }

   NRF_LOG_INFO("NRF_ERROR_NOT_SUPPORTED");
   return NRF_ERROR_NOT_SUPPORTED;
}

ret_code_t ret = app_usbd_class_descriptor_find(p_inst,p_setup_ev->setup.wValue.hb,p_setup_ev->setup.wValue.lb,p_trans_buff,&dsc_len);

It returns ret = NRF_ERROR_NOT_FOUND;

In the above p_setup_ev->setup.wValue.hb = 6; (desc_type)

p_setup_ev->setup.wValue.lb = 0;

p_trans_buff = 26 03 48 00 53 00 49 00 44 00 36 (Here I dont now the actual length of p_trans_buff)

dsc_len = 0;

I find the above value by debugging the code.

Inside the app_usbd_class_descriptor_find() function , the value of type as follows,

4, 33, 5, 5, 5 (4 for interface, 33 for CCID, 5 for endpoint)

So if(type == desc_type) condition is not satisfied. 

t returns ret = NRF_ERROR_NOT_FOUND; from app_usbd_class_descriptor_find

Help me to solve this issue

[Håkon Alseth]

That can potentially happen (ie: the request wasn't for this specific descriptor), which is the reason why all class implementations forward that to the app_usbd_core.c::app_usbd_core_setup_rsp().

It is hard to tell what happens here, as I do not have all the details. What is the expected vs. actual behavior that you're seeing?

 

Kind regards,

Håkon

[Mohammad]

Hi,

Still, the problem is not cleared, that the program sticks in an infinite loop at this part

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

But I can able to process all the USBD CCID device class functions. I have tested all the CCID functions its working fine. The output of it is given below for your reference. 

<info> app: USBD CCID example started.
<info> usbd_ccid: ccid_event_handler app_evt.type = 11
<info> app: USB power detected
<info> usbd_ccid: ccid_event_handler app_evt.type = 8
<info> app: USB ready
<info> usbd_ccid: ccid_event_handler app_evt.type = 10
<info> usbd_ccid: ccid_event_handler app_evt.type = 15
<info> usbd_ccid: ccid_event_handler app_evt.type = 13
<info> usbd_ccid: ccid_event_handler app_evt.type = 2
<info> usbd_ccid: ccid_event_handler app_evt.type = 3
<info> usbd_ccid: ccid_event_handler app_evt.type = 15
<info> usbd_ccid: ccid_event_handler app_evt.type = 1
<info> usbd_ccid: ccid_event_handler app_evt.type = 1
<info> usbd_ccid: ccid_event_handler app_evt.type = 15
<info> usbd_ccid: ccid_event_handler app_evt.type = 5

<info> usbd_ccid: setup_event_handler
<info> usbd_ccid: setup_req_std_in
<info> usbd_ccid: NRF_ERROR_NOT_SUPPORTED

<info> usbd_ccid: ccid_event_handler app_evt.type = 15
<info> usbd_ccid: ccid_event_handler app_evt.type = 5
<info> usbd_ccid: setup_event_handler
<info> usbd_ccid: setup_req_class_in
<info> usbd_ccid: Data = A1 02 00 00 00 04
<info> usbd_ccid: APP_USBD_CCID_GET_CLOCK_FREQUENCIES

<info> usbd_ccid: ccid_event_handler app_evt.type = 5
<info> usbd_ccid: setup_event_handler
<info> usbd_ccid: setup_req_class_in
<info> usbd_ccid: Data = A1 03 00 00 00 04
<info> usbd_ccid: APP_USBD_CCID_GET_DATA_RATES

<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_ICCPOWEROFF
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_ICCPOWERON
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_GETPARAMETERS
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_XFRBLOCK
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev
<info> usbd_ccid: PC_TO_RDR_ICCPOWEROFF
<info> usbd_ccid: ccid_event_handler app_evt.type = 6
<info> usbd_ccid: ccid_endpoint_ev

While digging app_usbd_event_queue_process(), this function doesnt return false, because m_sof_events_cnt is 0x0005 every time.

I don't know why m_sof_events_cnt is 5 every time. 

[Håkon Alseth]

Hi,

 

Mohammad said:

But I can able to process all the USBD CCID device class functions. I have tested all the CCID functions its working fine. The output of it is given below for your reference. 

 It sounds like the USB host is polling the USB device for data in a given interval, but the device does not have something to send. This can be verified if you have a usb sniffer trace (if you have a dedicated usb sniffer, like a ellisys usb explorer (https://www.ellisys.com/products/usbex200/) or any other dedicated usb sniffer (totalphase is also popular: https://www.totalphase.com/solutions/apps/usb-analyzer-guide/)

 

Mohammad said:

While digging app_usbd_event_queue_process(), this function doesnt return false, because m_sof_events_cnt is 0x0005 every time.

I don't know why m_sof_events_cnt is 5 every time. 

 This timing interval can be driven by the host. It is a bit unclear to me based on the logs, does everything work as expected?

 

Kind regards,

Håkon

[Mohammad]

Hi

 This timing interval can be driven by the host. It is a bit unclear to me based on the logs, does everything work as expected?

Yes, USB CCID class messages are working fine as expected. Only the problem, it's not coming out of this loop

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

[Håkon Alseth]

Hi,

 

If you have logging enabled, and print many characters over the UART, and the host is polling the usb device for data every 5 frames, I suspect that the usbd_event_queue will always look like it returns true, as it get polled very frequently, and the remainder of the time is spent with handling the NRF_LOG calls.

Could you try blinking a LED or similar in your main loop to see if it toggles during this state?

 

Kind regards,

Håkon

[Mohammad]

Hi,

  Thanks for your response. I tried as per your suggestion (I disabled the  NRF_LOG)  but the result is the same. Not coming out from the loop.

[Håkon Alseth]

It is hard to say what can be wrong here, as the logs do not show in-depth information on what is happening, and there's no way to see what the host is doing.

It sounds like a item is queued while you're trying to handle the queue, but I cannot state this for certain without looking at the USB traces.

Does any of the handlers return a non-zero return code? 

 

Kind regards,

Håkon