[U-Boot] [PATCH v3 00/13] usb: xhci: Add interrupt transfer support and full speed device support

[Marek Vasut]

On 07/21/2017 02:34 PM, Bin Meng wrote:
> Hi Marek,
> 
> On Fri, Jul 21, 2017 at 6:06 PM, Marek Vasut <marex at denx.de> wrote:
>> On 07/21/2017 12:04 PM, Bin Meng wrote:
>>> Hi Marek,
>>>
>>> On Fri, Jul 21, 2017 at 4:56 PM, Marek Vasut <marex at denx.de> wrote:
>>>> On 07/21/2017 10:54 AM, Bin Meng wrote:
>>>>> Hi Marek,
>>>>>
>>>>> On Thu, Jul 20, 2017 at 8:20 PM, Marek Vasut <marex at denx.de> wrote:
>>>>>> On 07/20/2017 02:16 PM, Bin Meng wrote:
>>>>>>> Hi Marek,
>>>>>>>
>>>>>>> On Thu, Jul 20, 2017 at 7:12 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>>>>>> This series is the final series of the xHCI driver update.
>>>>>>>>
>>>>>>>> This adds the missing interrupt transfer support to xHCI driver, so
>>>>>>>> that devices like USB keyboard that uses interrupt transfer when
>>>>>>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.
>>>>>>>>
>>>>>>>> This also adds full speed device support. Unlike low/high/super speed
>>>>>>>> devices, full speed device may report its endpoint 0 max packet size
>>>>>>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the
>>>>>>>> initial 8 bytes device descriptor, and later adjusts the context for
>>>>>>>> endpoint 0.
>>>>>>>>
>>>>>>>> This series also made several updates to xHCI driver to conform with
>>>>>>>> the spec.
>>>>>>>>
>>>>>>>> Previous two series:
>>>>>>>> [1]: usb: xhci: Fix USB xHCI support on Intel platform
>>>>>>>> https://lists.denx.de/pipermail/u-boot/2017-June/296166.html
>>>>>>>> [2]: usb: hub: Support USB 3.0 hubs
>>>>>>>> https://lists.denx.de/pipermail/u-boot/2017-June/296284.html
>>>>>>>>
>>>>>>>> This series is available at u-boot-x86/xhci-working3 for testing.
>>>>>>>>
>>>>>>>> Changes in v3:
>>>>>>>> - rebase on u-boot-usb/master
>>>>>>>> - add changes to poplar.h
>>>>>>>>
>>>>>>>> Bin Meng (13):
>>>>>>>>   usb: xhci: Add interrupt transfer support
>>>>>>>>   usb: configs: Clean up CONFIG_SYS_USB_EVENT_POLL_(xxx) in board
>>>>>>>>     configs
>>>>>>>>   usb: kbd: kconfig: Set a default polling mechanism for USB keyboard
>>>>>>>>   usb: Only get 64 bytes device descriptor for full speed devices
>>>>>>>>   usb: Read device descriptor after device is addressed for xHCI
>>>>>>>>   usb: xhci: Fix max packet size for full speed device endpoint 0
>>>>>>>>   usb: hub: Clear port reset before usb_hub_port_connect_change()
>>>>>>>>   usb: hub: Clear BH reset status change for a 3.0 hub
>>>>>>>>   usb: Handle audio extension endpoint descriptor in usb_parse_config()
>>>>>>>>   usb: xhci: Honor endpoint's interval
>>>>>>>>   usb: xhci: Program max burst size for endpoint
>>>>>>>>   usb: xhci: Set 'Error Count' to 0 for isoch endpoints
>>>>>>>>   usb: xhci: Set 'Average TRB Length' to 8 for control endpoints
>>>>>>>>
>>>>>>>
>>>>>>> This is the last xHCI series I have. Can you please take a look and apply?
>>>>>>
>>>>>> I'd like to see some RB/TB from Stefan Roese first if possible.
>>>>>>
>>>>>
>>>>> OK, if that's the case, can you send a pull request to Tom to get usb
>>>>> in? So that I can apply Stefan's x86 patch series on top of this. (I
>>>>> cannot send PR for x86 before the usb changes)
>>>>
>>>> I actually cannot because the USB is currently broken, sorry, I'm
>>>> waiting for a fix before I do that.
>>>>
>>>> Stefan, can you look at this series please?
>>>
>>> What "broken" issues are you seeing?
>>
>> I think Layerscape stopped building. Look at email with this subject
>> "Compile failure encountered on latest (2017-07-20)
>> git://git.denx.de/u-boot-usb.git branch master for target
>> ls1021atwr_nor_defconfig"
> 
> I see Patrice is working on a fix for the build failure.

Correct, it's almost done. Sorry for the delay.

-- 
Best regards,
Marek Vasut