Skip to content

Commit

Permalink
Possible fix for USB packets going missing. Thank gsh
Browse files Browse the repository at this point in the history
  • Loading branch information
popcornmix committed Jul 27, 2012
1 parent f958199 commit 7e76958
Showing 1 changed file with 15 additions and 6 deletions.
21 changes: 15 additions & 6 deletions drivers/usb/host/dwc_otg/dwc_otg_hcd_intr.c
Expand Up @@ -282,12 +282,21 @@ int32_t dwc_otg_hcd_handle_sof_intr(dwc_otg_hcd_t * hcd)
qh = DWC_LIST_ENTRY(qh_entry, dwc_otg_qh_t, qh_list_entry);
qh_entry = qh_entry->next;
if (dwc_frame_num_le(qh->sched_frame, hcd->frame_number)) {
/*
* Move QH to the ready list to be executed next
* (micro)frame.
*/
DWC_LIST_MOVE_HEAD(&hcd->periodic_sched_ready,
&qh->qh_list_entry);

if(qh->do_split && qh->sched_frame != hcd->frame_number)
{
// If we're late we may have missed the beginning of the frame
qh->sched_frame += qh->interval;
}
else
{
/*
* Move QH to the ready list to be executed next
* (micro)frame.
*/
DWC_LIST_MOVE_HEAD(&hcd->periodic_sched_ready,
&qh->qh_list_entry);
}
}
}
tr_type = dwc_otg_hcd_select_transactions(hcd);
Expand Down

3 comments on commit 7e76958

@NickBT
Copy link

@NickBT NickBT commented on 7e76958 Jul 27, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a missing .h from the commit by any chance?. I just did a git pull and dwc_otg_intr.c fails to compile with dwc_otg_hcd_t has no member 'non_periodic_channels' (line 926)

@popcornmix
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can see the change above - it doesn't cause the error you are seeing. Are you using standard bcmrpi_defconfig?
Can you check out a clean tree and try again?

@NickBT
Copy link

@NickBT NickBT commented on 7e76958 Jul 27, 2012 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.