Re: [PATCH v3 00/13] usb: dwc2: Fix up gadget isochronous support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

John Youn <John.Youn@xxxxxxxxxxxx> writes:
> On 5/25/2016 6:06 PM, John Youn wrote:
>> The following patch series fixes up isochronous support for the dwc2
>> gadget. The existing isochronous support lacked a few features. Most
>> notably it did not properly sync up with the first packet and it
>> didn't handle the Incomplete ISO IN/OUT interrupts.
>> 
>> These patches have been sitting in our internal tree for a while and
>> tested mostly on a HAPS-based FPGA IP validation platform.
>> 
>> v3:
>> * Simplified setting endpoint interval in patch 7/13
>> 
>> v2:
>> * Changed if/else to switch statement
>> * Changed frame_overrun to bool
>> * Downgraded a info message to dbg
>> * Added John Keeping's tested-by
>> 
>> Regards,
>> John
>> 
>> 
>> Vardan Mikayelyan (13):
>>   usb: dwc2: Add missing register field definitions
>>   usb: dwc2: gadget: Remove unnecessary line
>>   usb: dwc2: gadget: Remove unnecessary code
>>   usb: dwc2: gadget: Corrected field names
>>   usb: dwc2: gadget: Fix transfer stop programming for out endpoint
>>   usb: dwc2: gadget: Add dwc2_gadget_incr_frame_num()
>>   usb: dwc2: gadget: Corrected interval calculation
>>   usb: dwc2: gadget: Add dwc2_gadget_read_ep_interrupts function
>>   usb: dwc2: gadget: Add dwc2_gadget_start_next_request function
>>   usb: dwc2: gadget: Add OUTTKNEPDIS and NAKINTRPT handlers
>>   usb: dwc2: gadget: Add Incomplete ISO IN/OUT Interrupt handlers
>>   usb: dwc2: gadget: Add EP disabled interrupt handler
>>   usb: dwc2: gadget: Final fixes for BDMA ISOC
>> 
>>  drivers/usb/dwc2/core.h   |   8 +-
>>  drivers/usb/dwc2/gadget.c | 574 ++++++++++++++++++++++++++++++++++++----------
>>  drivers/usb/dwc2/hw.h     |  14 ++
>>  3 files changed, 470 insertions(+), 126 deletions(-)
>> 
>
>
> Hi Felipe,
>
> Could you take this series in your testing/next branch?
>
> I'd like it to have as much testing as possible.

sorry for the long delay, It's there now.

-- 
balbi

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux