Posted on April 17, 2014 at 13:49
Hello Gabriel,
I changed to use the ''new'' HAL and no longer the Host-Device Lib. That merely because the Host-Device Lib is no longer supported.
The generated HAL CDC code has some ''issues'':
1) After CDC_Receive is called, USBD_LL_PrepareReceive is never called again. So, the USB NAK's data on that endpoint.
So you need to add a possibility that USBD_LL_PrepareReceive is called, either directly or indirectly by your concrete CDC implementation. So, within USBD_CDC.c and/or the appropriate ''_if'' file.
It could be handled in USBD_CDC.c, USBD_ACD_DataOut by adding
USBD_LL_PrepareReceive(pdev, epnum, hcdc->RxBuffer, hcdc->RxLength);
directly after
((USBD_CDC_ItfTypeDef *)pdev->pUserData)->Receive(pdev, hcdc->RxBuffer, hcdc->RxLength);
2) It might happen, on transmit, that the internal handling within USBD_CDC.c stucks, so that you could never again transmit data. I changed that, so, that I could recover from that situation.
In my case it happened, that hcdc->TxState keeps at ''1'' at it is not reset to ''0'' by the interrupt. So before returning an error by TransmitPacket, I check if there is still data to be sent.
In ''ll_usb'' I added:
uint32_t USB_EPTxCount(USB_OTG_GlobalTypeDef *USBx, USB_OTG_EPTypeDef *ep) {
USB_OTG_INEndpointTypeDef *rep = USBx_INEP(ep->num);
uint32_t retVal;
retVal = ((rep->DIEPTSIZ & (USB_OTG_DIEPTSIZ_PKTCNT)) >> 19) * ep->maxpacket;
retVal += (rep->DIEPTSIZ & USB_OTG_DIEPTSIZ_XFRSIZ);
return retVal;
}
in ''hal_pcd'':
uint32_t HAL_PCD_EP_GetTxCount(PCD_HandleTypeDef *hpcd, uint8_t ep_addr) {
uint32_t retVal;
//TODO: Check if we need a lock here. Its just a register read anyway
retVal = USB_EPTxCount(hpcd->Instance, &hpcd->IN_ep[ep_addr & 0x7F]);
return retVal;
}
and in ''usbd_conf'' I added:
uint32_t USBD_LL_GetTxCount (USBD_HandleTypeDef *pdev, uint8_t ep_addr) {
return HAL_PCD_EP_GetTxCount(pdev->pData, ep_addr);
}
If USBD_LL_GetTxCount returns 0 I reset hcdc->TxState.
