i.MX RT Crossover MCUsのトピックRe: USB CDC VCOM - Consecutive RXs Fail

USB CDC VCOM - Consecutive RXs Fail

variable_andrew — Fri, 20 Mar 2020 22:16:34 GMT

Using the MCUXpresso SDK 2.7.0 dev_cdc_vcomm_freertos demo as a starting point, if i remove the "echo" functionality - so that i'm just receiving data, receive gets stuck/ stops receiving data after 1 packet.

If I leave the echo in there, everything works fine.

Is there something in the USB CDC Vcomm stack that is resetting the receive functionality via the USB_DeviceCdcAcmSend function?

I just realized I asked the same question for RT1020 but never received an answer - so asking again for the RT1052:

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Fri, 20 Mar 2020 23:28:12 GMT

Hi Variable_andrew:

I am trying to reproduce your issue on my side, I will let you know my result later.

Regards

Daniel

Re: USB CDC VCOM - Consecutive RXs Fail

variable_andrew — Mon, 23 Mar 2020 18:18:45 GMT

Hi danielchen@fsl‌,

Thanks for the quick response - I'll continue in this thread since I'm now 100% on the RT1052 (no longer using the RT1021).

So in order to reproduce:

MCUXpresso 11.1.1, SDK2.7.0 -> SDK Example "dev_cdc_vcomm_freertos" -> Link using Internal SRAM only and DebugPrint goes to SemiHost

Then on my board - we're using USB OTG2, so I have to go to virtual_com.h and update the controller to:

#define CONTROLLER_ID kUSB_ControllerEhci1‍

If you're on the EVK, can skip that step.

After this, I run, and the demo works, echo's just fine.

Now to duplicate my issue, we simply have to try and echo only every third or 4th RX.

Just add a static int RX counter, still reset the TX buff every RX, but only try to send on every 3rd or 4th packet, and it'll fail as soon as you get multiple RX's without doing a TX.

Here's the modified AppTask for you to test:

void APPTask(void *handle)
{
    usb_status_t error = kStatus_USB_Error;
    static int numRxs = 0;

    USB_DeviceApplicationInit();

    while (1)
    {
        if ((1 == s_cdcVcom.attach) && (1 == s_cdcVcom.startTransactions))
        {
            /* User Code */
            if ((0 != s_recvSize) && (0xFFFFFFFF != s_recvSize))
            {
                int32_t i;
                numRxs++;

                /* Copy Buffer to Send Buff */
                for (i = 0; i < s_recvSize; i++)
                {
                    s_currSendBuf[s_sendSize++] = s_currRecvBuf[i];
                }
                s_recvSize = 0;
            }

            if (s_sendSize)
            {
              uint32_t size = s_sendSize;
              s_sendSize = 0;
              if ((numRxs % 4) == 1){
                error = USB_DeviceCdcAcmSend(s_cdcVcom.cdcAcmHandle, USB_CDC_VCOM_BULK_IN_ENDPOINT, s_currSendBuf, size);
                if (error != kStatus_USB_Success)
                {
                    /* Failure to send Data Handling code here */
                }
              }
            }
        }
    }
}‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Tue, 24 Mar 2020 14:50:24 GMT

Yes, with your code, I can reproduce your issue.

I think you can check the s_currSendBuf and the length.

I will update you when I have result.

Regards

Daniel

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Wed, 25 Mar 2020 10:20:59 GMT

Hi variable_andrew:

Please see my modified code, it can work on my side.


    while (1)
    {
        if ((1 == s_cdcVcom.attach) && (1 == s_cdcVcom.startTransactions))
        {
            /* User Code */
            if ((0 != s_recvSize) && (0xFFFFFFFF != s_recvSize))
            {
                int32_t i;

                memset(s_currSendBuf, 0, sizeof(s_currSendBuf));

                /* Copy Buffer to Send Buff */
                for (i = 0; i < s_recvSize; i++)
                {
                    s_currSendBuf[s_sendSize++] = s_currRecvBuf[i];
                    numRxs++;
                }

                s_recvSize = 0;
            }

            if (s_sendSize)
            {
                uint32_t size = s_sendSize ;
                s_sendSize = 0;

                if ((numRxs %4) == 1 )

                {
                 size = numRxs--;
                }

                error =
                    USB_DeviceCdcAcmSend(s_cdcVcom.cdcAcmHandle, USB_CDC_VCOM_BULK_IN_ENDPOINT, s_currSendBuf, size);
                if (error != kStatus_USB_Success)
                {
                    /* Failure to send Data Handling code here */
                }


            }‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍

Regards

Daniel

Re: USB CDC VCOM - Consecutive RXs Fail

variable_andrew — Wed, 25 Mar 2020 15:08:37 GMT

Hi danielchen@fsl‌,

You have changed the code from counting the # of RX messages to the # of RX bytes - so this doesn't answer my question, which is:

Why does the USB SDK freeze if I don't send a message (w/ USB_DeviceCDCACMSend) every time I have an incoming message (not byte)?
This shouldn't be required in USB - ie, a standard use case might be to send a text file over USB to the RT1052, this might be split into 100 packets. If using RTS/CTS, I shouldn't need to provide any responses at the app layer for packets 0-99 to receive all the data, and USB shouldn't freeze either.

In the code you've provided specifically avoids what I'm trying to fix instead of making RX work without USB_DeviceCDCACMSend.

Here is the most basic test case:

void APPTask(void *handle)
{
    usb_status_t error = kStatus_USB_Error;
    static int numRxs = 0;

    USB_DeviceApplicationInit();
    while (1)
    {
        if ((1 == s_cdcVcom.attach) && (1 == s_cdcVcom.startTransactions))
        {
            /* User Code */
            if ((0 != s_recvSize) && (0xFFFFFFFF != s_recvSize))
            {
                numRxs++;

                //don't do anytihng w/ the RX data, just reset:

                memset(s_currSendBuf, 0, sizeof(s_currSendBuf));
                s_recvSize = 0;
            }
        }
    }
}‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍

If you run this, and then connect over a terminal application to the RT1052, you can send lines of test to the RT1052 (like "Test1", "Test2", etc).

You'll notice that after sending 5 or 10 messages over USB, if you pause the application, numRxs will be stuck at 1 instead of 5 or 10 (because the RT1052 USB only received 1 message before freezing up).

Could you please take another look and see if you can understand why we can't just receive data (without echoing back data on a per-packet basis)?

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Wed, 25 Mar 2020 15:36:16 GMT

Yes, I observed the application will freeze if I don't send a message. Let me try it again for this case.

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Thu, 26 Mar 2020 15:21:18 GMT

After trying many times. I know that the demo's default behavior is:

Send back the data that is received from host, then start to receive the next data from host. If host doesn't receive the data from the device, the device will not receive next data.

In your case, I would suggest you receive your message in USB_DeviceCdcVcomCallback. (virtual_com.c)

case kUSB_DeviceCdcEventRecvResponse

For example:

        case kUSB_DeviceCdcEventRecvResponse:
        {
            if ((1 == s_cdcVcom.attach) && (1 == s_cdcVcom.startTransactions))
            {
                s_recvSize = epCbParam->length;

.....

and do nothing in APPTask.

Regards

Daniel

Re: USB CDC VCOM - Consecutive RXs Fail

EdSutter — Thu, 03 Sep 2020 20:03:02 GMT

Hi,

I'm having a similar problem on the '1062.

Was this resolved?

Re: USB CDC VCOM - Consecutive RXs Fail

bill_muzika — Sun, 06 Dec 2020 18:42:42 GMT

I had the same problem with SDK 2.8 on the LPC55S69. I found that adding a call to USB_DeviceCdcAcmRecv after receiving a packet allows for the next to be received. This is done in the example when a send is complete. I'm not sure if it was the most correct way to handle it, but it explains why the receives would not work unless a send was completed in between.

else if ((1 == g_deviceComposite->cdcVcom.attach) && (1 == g_deviceComposite->cdcVcom.startTransactions))
{
    if ((epCbParam->buffer != NULL) || ((epCbParam->buffer == NULL) && (epCbParam->length == 0)))
    {
        /* User: add your own code for send complete event */
        /* Schedule buffer for next receive event */
        USB_DeviceCdcAcmRecv(handle, USB_CDC_VCOM_DIC_BULK_OUT_ENDPOINT, s_currRecvBuf,
                             g_cdcVcomDicEndpoints[0].maxPacketSize);
    }
}

Re: USB CDC VCOM - Consecutive RXs Fail

EdSutter — Mon, 07 Dec 2020 11:25:13 GMT

Seems there are a few different unofficial ways to fix this issue (see here).

All of the solutions I've seen (or wrote myself) seem like hacks that should not have to be there.

@nxp Support: IMHO, this should be fixed in the stack.

Re: USB CDC VCOM - Consecutive RXs Fail

ChrisN — Thu, 11 Nov 2021 09:52:39 GMT

Hello Daniel,

It is over 18 months later, and the examples have not changed and still exhibit the error. Are you still on it?

Cheers,

Chris

Re: USB CDC VCOM - Consecutive RXs Fail

akspurr — Wed, 15 Jan 2025 09:20:49 GMT

Hi @danielchen,

Was a solution to this issue found?

I'm also experiencing this problem on the rt1052.

Thanks,

Alex

Re: USB CDC VCOM - Consecutive RXs Fail

danielchen — Wed, 15 Jan 2025 09:54:17 GMT

Hi @akspurr :

I would suggest you open a new thread for this.

Regards

Daniel

Re: USB CDC VCOM - Consecutive RXs Fail

akspurr — Wed, 15 Jan 2025 10:44:13 GMT

Thanks for the fast response, I'll open one shortly.