Re: [PATCH v3 0/3] arm: allwinner: Wire up USB ports

[Philippe Mathieu-Daudé]

Cc'ing Niek.

On 2/17/20 9:48 PM, Guenter Roeck wrote:
> Instantiate EHCI and OHCI controllers on Allwinner A10.
>
> The first patch in the series moves the declaration of EHCISysBusState
> from hcd-ohci.c to hcd-ohci.h. This lets us add the structure to
> AwA10State. Similar, TYPE_SYSBUS_OHCI is moved to be able to use it
> outside its driver.
>
> The second patch introduces the ehci-sysbus property "companion-enable".
> This lets us use object_property_set_bool() to enable companion mode.
>
> The third patch instantiates EHCI and OHCI ports for Allwinner-A10
> and marks the OHCI ports as companions of the respective EHCI ports.
>
> Tested by attaching various high speed and full speed devices, and by
> booting from USB drive.
>
> v3: Rebased to master
> v2: Add summary
>      Rewrite to instantiate OHCI in companion mode; add patch 2/3
>      Merge EHCI and OHCI instantiation into a single patch
>
> ----------------------------------------------------------------
> Guenter Roeck (3):
>        hw: usb: hcd-ohci: Move OHCISysBusState and TYPE_SYSBUS_OHCI to include 
> file
>        hcd-ehci: Introduce "companion-enable" sysbus property
>        arm: allwinner: Wire up USB ports
>
>   hw/arm/allwinner-a10.c         | 43 ++++++++++++++++++++++++++++++++++++++++++
>   hw/usb/hcd-ehci-sysbus.c       |  2 ++
>   hw/usb/hcd-ohci.c              | 15 ---------------
>   hw/usb/hcd-ohci.h              | 16 ++++++++++++++++
>   include/hw/arm/allwinner-a10.h |  6 ++++++
>   5 files changed, 67 insertions(+), 15 deletions(-)