sergiotarxz/nuttx
sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

SAMA5 OHCI: Fix errors introduced with blind changes to match changes to LPC17 OHCI. Fix some handling for the case where we get a falso alarm transfer completion after disconnection/cancellation

Browse Source
This commit is contained in:
Gregory Nutt 2015-05-15 09:02:17 -06:00
parent 87b065253d
commit 5a666cf85b
1 changed files with 56 additions and 44 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
arch/arm/src/sama5/sam_ohci.c
View File

@ -1050,7 +1050,7 @@ static inline int sam_rembulked(struct sam_ed_s *ed)
* after removing the ED node.
*/
sam_getreg(0, SAM_USBHOST_BULKED);
sam_putreg(0, SAM_USBHOST_BULKED);
if (sam_getreg(SAM_USBHOST_BULKHEADED) != 0)
{
/* If the bulk list is now empty, then disable it */
@ -2137,7 +2137,17 @@ static void sam_wdh_bottomhalf(void)
eplist = ed->eplist;
DEBUGASSERT(eplist != NULL);
/* Also invalidate the control ED so that it two will be re-read from
/* Do nothing if there is no transfer in progress. This situation may
* occur after cancellation of a transfer.
*/
#ifdef CONFIG_USBHOST_ASYNCH
if (eplist->wdhwait || eplist->callback)
#else
if (eplist->wdhwait)
#endif
{
/* Invalidate the control ED so that it two will be re-read from
* memory.
*/
@ -2159,9 +2169,9 @@ static void sam_wdh_bottomhalf(void)
}
#endif
/* Determine the number of bytes actually transfer by* subtracting the
* buffer start address from the CBP. A value of zero means that all
* bytes were transferred.
/* Determine the number of bytes actually transfer by* subtracting
* the buffer start address from the CBP. A value of zero means
* that all bytes were transferred.
*/
tmp = (uintptr_t)td->hw.cbp;
@ -2176,9 +2186,9 @@ static void sam_wdh_bottomhalf(void)
paddr = sam_physramaddr((uintptr_t)eplist->buffer);
DEBUGASSERT(tmp >= paddr);
/* Determine the size of the transfer by subtracting the current
* buffer pointer (CBP) from the initial buffer pointer (on packet
* receipt only).
/* Determine the size of the transfer by subtracting the
* current buffer pointer (CBP) from the initial buffer
* pointer (on packet receipt only).
*/
tmp -= paddr;
@ -2186,6 +2196,7 @@ static void sam_wdh_bottomhalf(void)
}
eplist->xfrd = (uint16_t)tmp;
}
/* Return the TD to the free list */
@ -3628,8 +3639,9 @@ static int sam_cancel(struct usbhost_driver_s *drvr, usbhost_ep_t ep)
struct sam_ed_s *ed;
struct sam_gtd_s *td;
struct sam_gtd_s *next;
uintptr_t paddr;
irqstate_t flags;
uintptr_t paddr;
uint32_t ctrl;
DEBUGASSERT(eplist && eplist->ed && eplist->tail);
ed = eplist->ed;