Update some comments

This commit is contained in:
Gregory Nutt 2017-03-09 13:57:37 -06:00
parent a3b4475474
commit 04297d1b0f
4 changed files with 28 additions and 4 deletions

View File

@ -1881,7 +1881,13 @@ static ssize_t stm32_in_transfer(FAR struct stm32_usbhost_s *priv, int chidx,
if (xfrd > 0) if (xfrd > 0)
{ {
/* Yes, return the amount of data received */ /* Yes, return the amount of data received.
*
* REVISIT: This behavior is clearly correct for CDC/ACM
* bulk transfers and HID interrupt transfers. But I am
* not so certain for MSC bulk transfers which, I think,
* could have NAKed packets in the middle of a transfer.
*/
return xfrd; return xfrd;
} }

View File

@ -1886,7 +1886,13 @@ static ssize_t stm32_in_transfer(FAR struct stm32_usbhost_s *priv, int chidx,
if (xfrd > 0) if (xfrd > 0)
{ {
/* Yes, return the amount of data received */ /* Yes, return the amount of data received.
*
* REVISIT: This behavior is clearly correct for CDC/ACM
* bulk transfers and HID interrupt transfers. But I am
* not so certain for MSC bulk transfers which, I think,
* could have NAKed packets in the middle of a transfer.
*/
return xfrd; return xfrd;
} }

View File

@ -1880,7 +1880,13 @@ static ssize_t stm32_in_transfer(FAR struct stm32_usbhost_s *priv, int chidx,
if (xfrd > 0) if (xfrd > 0)
{ {
/* Yes, return the amount of data received */ /* Yes, return the amount of data received.
*
* REVISIT: This behavior is clearly correct for CDC/ACM
* bulk transfers and HID interrupt transfers. But I am
* not so certain for MSC bulk transfers which, I think,
* could have NAKed packets in the middle of a transfer.
*/
return xfrd; return xfrd;
} }

View File

@ -1885,7 +1885,13 @@ static ssize_t stm32l4_in_transfer(FAR struct stm32l4_usbhost_s *priv,
if (xfrd > 0) if (xfrd > 0)
{ {
/* Yes, return the amount of data received */ /* Yes, return the amount of data received.
*
* REVISIT: This behavior is clearly correct for CDC/ACM
* bulk transfers and HID interrupt transfers. But I am
* not so certain for MSC bulk transfers which, I think,
* could have NAKed packets in the middle of a transfer.
*/
return xfrd; return xfrd;
} }