Author: hselasky Date: Sat Nov 22 08:47:04 2014 New Revision: 274842 URL: https://svnweb.freebsd.org/changeset/base/274842
Log: Use correct length mask for split transactions. The hardware would sometimes put non-zero values in the upper length bits, which are available for high-speed-only USB transactions, breaking the reception of data. Modified: head/sys/dev/usb/controller/saf1761_otg.c head/sys/dev/usb/controller/saf1761_otg_reg.h Modified: head/sys/dev/usb/controller/saf1761_otg.c ============================================================================== --- head/sys/dev/usb/controller/saf1761_otg.c Sat Nov 22 08:09:26 2014 (r274841) +++ head/sys/dev/usb/controller/saf1761_otg.c Sat Nov 22 08:47:04 2014 (r274842) @@ -510,7 +510,10 @@ saf1761_host_bulk_data_rx(struct saf1761 td->error_any = 1; goto complete; } - count = (status & SOTG_PTD_DW3_XFER_COUNT); + if (td->dw1_value & SOTG_PTD_DW1_ENABLE_SPLIT) + count = (status & SOTG_PTD_DW3_XFER_COUNT_SPLIT); + else + count = (status & SOTG_PTD_DW3_XFER_COUNT_HS); got_short = 0; /* verify the packet byte count */ @@ -700,7 +703,10 @@ saf1761_host_intr_data_rx(struct saf1761 td->error_any = 1; goto complete; } - count = (status & SOTG_PTD_DW3_XFER_COUNT); + if (td->dw1_value & SOTG_PTD_DW1_ENABLE_SPLIT) + count = (status & SOTG_PTD_DW3_XFER_COUNT_SPLIT); + else + count = (status & SOTG_PTD_DW3_XFER_COUNT_HS); got_short = 0; /* verify the packet byte count */ @@ -895,7 +901,10 @@ saf1761_host_isoc_data_rx(struct saf1761 } else if (status & SOTG_PTD_DW3_HALTED) { goto complete; } - count = (status & SOTG_PTD_DW3_XFER_COUNT); + if (td->dw1_value & SOTG_PTD_DW1_ENABLE_SPLIT) + count = (status & SOTG_PTD_DW3_XFER_COUNT_SPLIT); + else + count = (status & SOTG_PTD_DW3_XFER_COUNT_HS); /* verify the packet byte count */ if (count != td->max_packet_size) { Modified: head/sys/dev/usb/controller/saf1761_otg_reg.h ============================================================================== --- head/sys/dev/usb/controller/saf1761_otg_reg.h Sat Nov 22 08:09:26 2014 (r274841) +++ head/sys/dev/usb/controller/saf1761_otg_reg.h Sat Nov 22 08:47:04 2014 (r274842) @@ -211,7 +211,8 @@ #define SOTG_PTD_DW3_CERR_3 (3U << 23) #define SOTG_PTD_DW3_CERR_2 (2U << 23) /* infinite NAKs */ #define SOTG_PTD_DW3_CERR_1 (1U << 23) -#define SOTG_PTD_DW3_XFER_COUNT 0x7FFF +#define SOTG_PTD_DW3_XFER_COUNT_HS 0x7FFF +#define SOTG_PTD_DW3_XFER_COUNT_SPLIT 0x03FF #define SOTG_PTD_DW4 16 #define SOTG_PTD_DW5 20 #define SOTG_PTD_DW6 24 _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"