* [PATCH v1 0/2] add DwUsb3Dxe driver
@ 2018-08-20 10:31 Haojian Zhuang
2018-08-20 10:31 ` [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol Haojian Zhuang
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Haojian Zhuang @ 2018-08-20 10:31 UTC (permalink / raw)
To: edk2-devel
Changelog:
v1:
* Add Designware USB 3.0 device driver.
Haojian Zhuang (2):
EmbeddedPkg: add DwUsb protocol
EmbeddedPkg/Drivers: add DwUsb3Dxe driver
EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c | 2434 +++++++++++++++++++++++++++
EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec | 44 +
EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h | 632 +++++++
EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf | 52 +
EmbeddedPkg/Include/Protocol/DwUsb.h | 81 +
5 files changed, 3243 insertions(+)
create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c
create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec
create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h
create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf
create mode 100644 EmbeddedPkg/Include/Protocol/DwUsb.h
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol 2018-08-20 10:31 [PATCH v1 0/2] add DwUsb3Dxe driver Haojian Zhuang @ 2018-08-20 10:31 ` Haojian Zhuang 2018-10-04 14:49 ` Leif Lindholm 2018-08-20 10:31 ` [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver Haojian Zhuang 2018-10-04 14:15 ` [PATCH v1 0/2] " Leif Lindholm 2 siblings, 1 reply; 9+ messages in thread From: Haojian Zhuang @ 2018-08-20 10:31 UTC (permalink / raw) To: edk2-devel The protocol defines the callbacks that could be implemented in platform driver. DwUsb device driver needs these callbacks to implement USB device functionality. Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> --- EmbeddedPkg/Include/Protocol/DwUsb.h | 81 ++++++++++++++++++++ 1 file changed, 81 insertions(+) diff --git a/EmbeddedPkg/Include/Protocol/DwUsb.h b/EmbeddedPkg/Include/Protocol/DwUsb.h new file mode 100644 index 000000000000..b9fb776f9258 --- /dev/null +++ b/EmbeddedPkg/Include/Protocol/DwUsb.h @@ -0,0 +1,81 @@ +/** @file + + Copyright (c) 2018, Linaro. All rights reserved. + + This program and the accompanying materials are licensed and made available + under the terms and conditions of the BSD License which accompanies this + distribution. The full text of the license may be found at + http://opensource.org/licenses/bsd-license.php + + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. + +**/ + +#ifndef __DW_USB_H__ +#define __DW_USB_H__ + +// +// Protocol GUID +// +#define DW_USB_PROTOCOL_GUID { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} + +// +// Protocol interface structure +// +typedef struct _DW_USB_PROTOCOL DW_USB_PROTOCOL; + +#define USB_HOST_MODE 0 +#define USB_DEVICE_MODE 1 +#define USB_CABLE_NOT_ATTACHED 2 + +#define LANG_LENGTH 8 +#define MANU_FACTURER_STRING_LENGTH 32 +#define PRODUCT_STRING_LENGTH 32 +#define SERIAL_STRING_LENGTH 17 + +typedef +EFI_STATUS +(EFIAPI *DW_USB_GET_LANG) ( + OUT CHAR16 *Lang, + OUT UINT8 *Length + ); + +typedef +EFI_STATUS +(EFIAPI *DW_USB_GET_MANU_FACTURER) ( + OUT CHAR16 *ManuFacturer, + OUT UINT8 *Length + ); + +typedef +EFI_STATUS +(EFIAPI *DW_USB_GET_PRODUCT) ( + OUT CHAR16 *Product, + OUT UINT8 *Length + ); + +typedef +EFI_STATUS +(EFIAPI *DW_USB_GET_SERIAL_NO) ( + OUT CHAR16 *SerialNo, + OUT UINT8 *Length + ); + +typedef +EFI_STATUS +(EFIAPI *DW_USB_PHY_INIT) ( + IN UINT8 Mode + ); + +struct _DW_USB_PROTOCOL { + DW_USB_GET_LANG GetLang; + DW_USB_GET_MANU_FACTURER GetManuFacturer; + DW_USB_GET_PRODUCT GetProduct; + DW_USB_GET_SERIAL_NO GetSerialNo; + DW_USB_PHY_INIT PhyInit; +}; + +extern EFI_GUID gDwUsbProtocolGuid; + +#endif /* __DW_USB_H__ */ -- 2.7.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol 2018-08-20 10:31 ` [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol Haojian Zhuang @ 2018-10-04 14:49 ` Leif Lindholm 2018-10-22 2:39 ` Haojian Zhuang 0 siblings, 1 reply; 9+ messages in thread From: Leif Lindholm @ 2018-10-04 14:49 UTC (permalink / raw) To: Haojian Zhuang; +Cc: edk2-devel, Ard Biesheuvel On Mon, Aug 20, 2018 at 06:31:24PM +0800, Haojian Zhuang wrote: > The protocol defines the callbacks that could be implemented in > platform driver. DwUsb device driver needs these callbacks to > implement USB device functionality. > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> > --- > EmbeddedPkg/Include/Protocol/DwUsb.h | 81 ++++++++++++++++++++ > 1 file changed, 81 insertions(+) > > diff --git a/EmbeddedPkg/Include/Protocol/DwUsb.h b/EmbeddedPkg/Include/Protocol/DwUsb.h > new file mode 100644 > index 000000000000..b9fb776f9258 > --- /dev/null > +++ b/EmbeddedPkg/Include/Protocol/DwUsb.h > @@ -0,0 +1,81 @@ > +/** @file > + > + Copyright (c) 2018, Linaro. All rights reserved. > + > + This program and the accompanying materials are licensed and made available > + under the terms and conditions of the BSD License which accompanies this > + distribution. The full text of the license may be found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > + > +**/ > + > +#ifndef __DW_USB_H__ > +#define __DW_USB_H__ Can you expand all instances of public interfaces to DESIGNWARE? > + > +// > +// Protocol GUID > +// > +#define DW_USB_PROTOCOL_GUID { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} > + > +// > +// Protocol interface structure > +// > +typedef struct _DW_USB_PROTOCOL DW_USB_PROTOCOL; > + > +#define USB_HOST_MODE 0 > +#define USB_DEVICE_MODE 1 > +#define USB_CABLE_NOT_ATTACHED 2 And add a DESIGNWARE_ prefix to these? > + > +#define LANG_LENGTH 8 > +#define MANU_FACTURER_STRING_LENGTH 32 MANUFACTURER. Why the hardcoded string lengths? Are they mandated somewhere? > +#define PRODUCT_STRING_LENGTH 32 > +#define SERIAL_STRING_LENGTH 17 > + > +typedef > +EFI_STATUS > +(EFIAPI *DW_USB_GET_LANG) ( > + OUT CHAR16 *Lang, > + OUT UINT8 *Length > + ); > + > +typedef > +EFI_STATUS > +(EFIAPI *DW_USB_GET_MANU_FACTURER) ( > + OUT CHAR16 *ManuFacturer, ManuFacturer -> Manufacturer. > + OUT UINT8 *Length > + ); > + > +typedef > +EFI_STATUS > +(EFIAPI *DW_USB_GET_PRODUCT) ( > + OUT CHAR16 *Product, > + OUT UINT8 *Length > + ); > + > +typedef > +EFI_STATUS > +(EFIAPI *DW_USB_GET_SERIAL_NO) ( SERIAL_NUMBER > + OUT CHAR16 *SerialNo, SerialNumber. > + OUT UINT8 *Length > + ); > + > +typedef > +EFI_STATUS > +(EFIAPI *DW_USB_PHY_INIT) ( > + IN UINT8 Mode > + ); > + > +struct _DW_USB_PROTOCOL { > + DW_USB_GET_LANG GetLang; > + DW_USB_GET_MANU_FACTURER GetManuFacturer; GetManuFacturer -> GetManufacturer. > + DW_USB_GET_PRODUCT GetProduct; > + DW_USB_GET_SERIAL_NO GetSerialNo; > + DW_USB_PHY_INIT PhyInit; > +}; > + > +extern EFI_GUID gDwUsbProtocolGuid; gDesignWareUsbProtocolGuid. / Leif > + > +#endif /* __DW_USB_H__ */ > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol 2018-10-04 14:49 ` Leif Lindholm @ 2018-10-22 2:39 ` Haojian Zhuang 0 siblings, 0 replies; 9+ messages in thread From: Haojian Zhuang @ 2018-10-22 2:39 UTC (permalink / raw) To: Leif Lindholm; +Cc: edk2-devel@lists.01.org, Ard Biesheuvel On Thu, 4 Oct 2018 at 22:49, Leif Lindholm <leif.lindholm@linaro.org> wrote: > > On Mon, Aug 20, 2018 at 06:31:24PM +0800, Haojian Zhuang wrote: > > The protocol defines the callbacks that could be implemented in > > platform driver. DwUsb device driver needs these callbacks to > > implement USB device functionality. > > > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> > > --- > > EmbeddedPkg/Include/Protocol/DwUsb.h | 81 ++++++++++++++++++++ > > 1 file changed, 81 insertions(+) > > > > diff --git a/EmbeddedPkg/Include/Protocol/DwUsb.h b/EmbeddedPkg/Include/Protocol/DwUsb.h > > new file mode 100644 > > index 000000000000..b9fb776f9258 > > --- /dev/null > > +++ b/EmbeddedPkg/Include/Protocol/DwUsb.h > > @@ -0,0 +1,81 @@ > > +/** @file > > + > > + Copyright (c) 2018, Linaro. All rights reserved. > > + > > + This program and the accompanying materials are licensed and made available > > + under the terms and conditions of the BSD License which accompanies this > > + distribution. The full text of the license may be found at > > + http://opensource.org/licenses/bsd-license.php > > + > > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > > + > > +**/ > > + > > +#ifndef __DW_USB_H__ > > +#define __DW_USB_H__ > > Can you expand all instances of public interfaces to DESIGNWARE? > OK > > + > > +// > > +// Protocol GUID > > +// > > +#define DW_USB_PROTOCOL_GUID { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} > > + > > +// > > +// Protocol interface structure > > +// > > +typedef struct _DW_USB_PROTOCOL DW_USB_PROTOCOL; > > + > > +#define USB_HOST_MODE 0 > > +#define USB_DEVICE_MODE 1 > > +#define USB_CABLE_NOT_ATTACHED 2 > > And add a DESIGNWARE_ prefix to these? > OK > > + > > +#define LANG_LENGTH 8 > > +#define MANU_FACTURER_STRING_LENGTH 32 > > MANUFACTURER. > OK > Why the hardcoded string lengths? > Are they mandated somewhere? > > > +#define PRODUCT_STRING_LENGTH 32 > > +#define SERIAL_STRING_LENGTH 17 > > + > > +typedef > > +EFI_STATUS > > +(EFIAPI *DW_USB_GET_LANG) ( > > + OUT CHAR16 *Lang, > > + OUT UINT8 *Length > > + ); > > + > > +typedef > > +EFI_STATUS > > +(EFIAPI *DW_USB_GET_MANU_FACTURER) ( > > + OUT CHAR16 *ManuFacturer, > > ManuFacturer -> > Manufacturer. > OK > > + OUT UINT8 *Length > > + ); > > + > > +typedef > > +EFI_STATUS > > +(EFIAPI *DW_USB_GET_PRODUCT) ( > > + OUT CHAR16 *Product, > > + OUT UINT8 *Length > > + ); > > + > > +typedef > > +EFI_STATUS > > +(EFIAPI *DW_USB_GET_SERIAL_NO) ( > > SERIAL_NUMBER > > > + OUT CHAR16 *SerialNo, > > SerialNumber. > OK > > + OUT UINT8 *Length > > + ); > > + > > +typedef > > +EFI_STATUS > > +(EFIAPI *DW_USB_PHY_INIT) ( > > + IN UINT8 Mode > > + ); > > + > > +struct _DW_USB_PROTOCOL { > > + DW_USB_GET_LANG GetLang; > > + DW_USB_GET_MANU_FACTURER GetManuFacturer; > > GetManuFacturer -> > GetManufacturer. > OK > > + DW_USB_GET_PRODUCT GetProduct; > > + DW_USB_GET_SERIAL_NO GetSerialNo; > > + DW_USB_PHY_INIT PhyInit; > > +}; > > + > > +extern EFI_GUID gDwUsbProtocolGuid; > > gDesignWareUsbProtocolGuid. > OK Best Regards Haojian ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver 2018-08-20 10:31 [PATCH v1 0/2] add DwUsb3Dxe driver Haojian Zhuang 2018-08-20 10:31 ` [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol Haojian Zhuang @ 2018-08-20 10:31 ` Haojian Zhuang 2018-10-04 16:32 ` Leif Lindholm 2018-10-04 14:15 ` [PATCH v1 0/2] " Leif Lindholm 2 siblings, 1 reply; 9+ messages in thread From: Haojian Zhuang @ 2018-08-20 10:31 UTC (permalink / raw) To: edk2-devel Add Designware USB 3.0 device driver. It's focus on USB device functionality, not USB Host functionality. The USB driver is mainly used for Android Fastboot application. Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> --- EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec | 44 + EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf | 52 + EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h | 632 +++++ EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c | 2434 ++++++++++++++++++++ 4 files changed, 3162 insertions(+) diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec new file mode 100644 index 000000000000..038e4881a948 --- /dev/null +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec @@ -0,0 +1,44 @@ +#/** @file +# Framework Module Development Environment Industry Standards +# +# This Package provides headers and libraries that conform to EFI/PI Industry standards. +# Copyright (c) 2007, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2012-2014, ARM Ltd. All rights reserved.<BR> +# Copyright (c) 2018, Linaro. All rights reserved.<BR> +# +# This program and the accompanying materials are licensed and made available under +# the terms and conditions of the BSD License which accompanies this distribution. +# The full text of the license may be found at +# http://opensource.org/licenses/bsd-license.php +# +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +# +#**/ + +[Defines] + DEC_SPECIFICATION = 0x00010019 + PACKAGE_NAME = DwUsb3DxePkg + PACKAGE_GUID = 9b7aa6fe-405b-4955-af1f-5faf183aedec + PACKAGE_VERSION = 0.1 + + +################################################################################ +# +# Include Section - list of Include Paths that are provided by this package. +# Comments are used for Keywords and Module Types. +# +# Supported Module Types: +# BASE SEC PEI_CORE PEIM DXE_CORE DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER DXE_SAL_DRIVER UEFI_DRIVER UEFI_APPLICATION +# +################################################################################ + +[Guids.common] + gDwUsb3DxeTokenSpaceGuid = { 0x098a50d3, 0x92e2, 0x461a, { 0xb6, 0xba, 0xf8, 0x5d, 0x9d, 0x89, 0x49, 0xf5 }} + +[Protocols.common] + gDwUsbProtocolGuid = { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} + +[PcdsFixedAtBuild.common] + # DwUsb Driver PCDs + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress|0x0|UINT32|0x00000001 diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf new file mode 100644 index 000000000000..510b51a34de7 --- /dev/null +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf @@ -0,0 +1,52 @@ +#/** @file +# +# Copyright (c) 2018, Linaro Limited. All rights reserved. +# +# This program and the accompanying materials +# are licensed and made available under the terms and conditions of the BSD License +# which accompanies this distribution. The full text of the license may be found at +# http://opensource.org/licenses/bsd-license.php +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. +# +# +#**/ + +[Defines] + INF_VERSION = 0x00010019 + BASE_NAME = DwUsb3Dxe + FILE_GUID = 0879cd34-c399-4315-9891-56024072e711 + MODULE_TYPE = UEFI_DRIVER + VERSION_STRING = 1.0 + ENTRY_POINT = DwUsb3EntryPoint + +[Sources.common] + DwUsb3Dxe.c + +[LibraryClasses] + CacheMaintenanceLib + DebugLib + DmaLib + IoLib + MemoryAllocationLib + TimerLib + UefiBootServicesTableLib + UefiDriverEntryPoint + UsbSerialNumberLib + +[Protocols] + gEfiDriverBindingProtocolGuid + gUsbDeviceProtocolGuid + +[Packages] + ArmPkg/ArmPkg.dec + EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec + EmbeddedPkg/EmbeddedPkg.dec + MdePkg/MdePkg.dec + MdeModulePkg/MdeModulePkg.dec + +[Pcd] + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress + +[Depex] + TRUE diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h new file mode 100644 index 000000000000..92b610553169 --- /dev/null +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h @@ -0,0 +1,632 @@ +/** @file + + Copyright (c) 2017, Linaro Limited. All rights reserved. + + This program and the accompanying materials + are licensed and made available under the terms and conditions of the BSD License + which accompanies this distribution. The full text of the license may be found at + http://opensource.org/licenses/bsd-license.php + + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. + +**/ + +#ifndef __DW_USB3_DXE_H__ +#define __DW_USB3_DXE_H__ + +#define DW_USB3_BASE FixedPcdGet32 (PcdDwUsb3DxeBaseAddress) + +#define GSBUCFG0 (DW_USB3_BASE + 0xC100) +#define GCTL (DW_USB3_BASE + 0xC110) + +#define GCTL_PWRDNSCALE_MASK (0x1FFF << 19) +#define GCTL_PWRDNSCALE(x) (((x) & 0x1FFF) << 19) +#define GCTL_U2RSTECN BIT16 +#define GCTL_PRTCAPDIR_MASK (BIT13 | BIT12) +#define GCTL_PRTCAPDIR_HOST BIT12 +#define GCTL_PRTCAPDIR_DEVICE BIT13 +#define GCTL_PRTCAPDIR_OTG (BIT13 | BIT12) +#define GCTL_U2EXIT_LFPS BIT2 + +#define GUSB2PHYCFG(x) (DW_USB3_BASE + 0xC200 + (((x) & 0xF) << 2)) + +#define GUSB2PHYCFG_USBTRDTIM_MASK (0xF << 10) +#define GUSB2PHYCFG_USBTRDTIM(x) (((x) & 0xF) << 10) +#define GUSB2PHYCFG_SUSPHY BIT6 + +#define GUSB3PIPECTL(x) (DW_USB3_BASE + 0xC2C0 + (((x) & 0x3) << 2)) + +#define PIPECTL_DELAYP1TRANS BIT18 +#define PIPECTL_SUSPEND_EN BIT17 +#define PIPECTL_LFPS_FILTER BIT9 +#define PIPECTL_TX_DEMPH_MASK (0x3 << 1) +#define PIPECTL_TX_DEMPH(x) (((x) & 0x3) << 1) + +#define GTXFIFOSIZ(x) (DW_USB3_BASE + 0xC300 + (((x) & 0x1F) << 2)) +#define GRXFIFOSIZ(x) (DW_USB3_BASE + 0xC380 + (((x) & 0x1F) << 2)) + +#define FIFOSIZ_ADDR(x) (((x) & 0xFFFF) << 16) +#define FIFOSIZ_DEP(x) ((x) & 0xFFFF) + +#define GEVNTADRL(x) (DW_USB3_BASE + 0xC400 + (((x) & 0x1F) << 2)) +#define GEVNTADRH(x) (DW_USB3_BASE + 0xC404 + (((x) & 0x1F) << 2)) +#define GEVNTSIZ(x) (DW_USB3_BASE + 0xC408 + (((x) & 0x1F) << 2)) + +#define GEVNTSIZ_EVNTINTMASK BIT31 +#define GEVNTSIZ_EVNTSIZ_MASK (0xFFFF) +#define GEVNTSIZ_EVNTSIZ(x) ((x) & 0xFFFF) + +#define GEVNTCOUNT(x) (DW_USB3_BASE + 0xC40C + (((x) & 0x1F) << 2)) +#define GEVNTCOUNT_EVNTCOUNT_MASK (0xFFFF) +#define GEVNTCOUNT_EVNTCOUNT(x) ((x) & 0xFFFF) + +// Non-Endpoint specific event flag +#define GEVNT_INTTYPE_MASK (0x7F << 1) +#define GEVNT_INTTYPE(x) (((x) & 0x7F) << 1) +#define EVENT_I2C_INT 4 +#define EVENT_CARKIT_INT 3 +#define EVENT_OTG_INT 1 +#define EVENT_DEV_INT 0 + +#define GEVNT_NON_EP BIT0 +// Endpoint specific event flag +#define GEVNT_DEPEVT_INTTYPE_MASK (0xF << 6) +#define GEVNT_DEPEVT_INTTYPE(x) (((x) & 0xF) << 6) +#define GEVNT_DEPEVT_INTTYPE_EPCMD_CMPL (7 << 6) +#define GEVNT_DEPEVT_INTTYPE_STRM_EVT (6 << 6) +#define GEVNT_DEPEVT_INTTYPE_FIFOXRUN (4 << 6) +#define GEVNT_DEPEVT_INTTYPE_XFER_NRDY (3 << 6) +#define GEVNT_DEPEVT_INTTYPE_XFER_IN_PROG (2 << 6) +#define GEVNT_DEPEVT_INTTYPE_XFER_CMPL (1 << 6) +#define GEVNT_DEPEVT_EPNUM_MASK (0x1F << 1) +#define GEVNT_DEPEVT_EPNUM_SHIFT 1 +#define GEVNT_DEPEVT_EPNUM(x) (((x) & 0x1F) << 1) +// Devices specific event flag +#define GEVNT_DEVT_MASK (0xF << 8) +#define GEVNT_DEVT_SHIFT 8 +#define GEVNT_DEVT(x) (((x) & 0xF) << 8) +#define GEVNT_DEVT_INACT_TIMEOUT_RCVD (0x15 << 8) +#define GEVNT_DEVT_VNDR_DEV_TST_RCVD (0x14 << 8) +#define GEVNT_DEVT_OVERFLOW (0x13 << 8) +#define GEVNT_DEVT_CMD_CMPL (0x12 << 8) +#define GEVNT_DEVT_ERRATICERR (0x11 << 8) +#define GEVNT_DEVT_SOF (0x7 << 8) +#define GEVNT_DEVT_EOPF (0x6 << 8) +#define GEVNT_DEVT_HIBER_REQ (0x5 << 8) +#define GEVNT_DEVT_WKUP (0x4 << 8) +#define GEVNT_DEVT_ULST_CHNG (0x3 << 8) +#define GEVNT_DEVT_CONNDONE (0x2 << 8) +#define GEVNT_DEVT_USBRESET (0x1 << 8) +#define GEVNT_DEVT_DISCONN (0x0 << 8) + +#define DCFG (DW_USB3_BASE + 0xC700) + +#define DCFG_NUMP_MASK (0x1F << 17) +#define DCFG_NUMP(x) (((x) & 0x1F) << 17) +#define DCFG_DEVADDR_MASK (0x7F << 3) +#define DCFG_DEVADDR(x) (((x) & 0x7F) << 3) +#define DCFG_DEVSPD_MASK (0x7) +#define DCFG_DEVSPD(x) ((x) & 0x7) +#define DEVSPD_HS_PHY_30MHZ_OR_60MHZ 0 +#define DEVSPD_FS_PHY_30MHZ_OR_60MHZ 1 +#define DEVSPD_LS_PHY_6MHZ 2 +#define DEVSPD_FS_PHY_48MHZ 3 +#define DEVSPD_SS_PHY_125MHZ_OR_250MHZ 4 + +#define DCTL (DW_USB3_BASE + 0xC704) + +#define DCTL_RUN_STOP BIT31 +#define DCTL_CSFTRST BIT30 +#define DCTL_INIT_U2_EN BIT12 +#define DCTL_ACCEPT_U2_EN BIT11 +#define DCTL_INIT_U1_EN BIT10 +#define DCTL_ACCEPT_U1_EN BIT9 + +#define DEVTEN (DW_USB3_BASE + 0xC708) +#define DEVTEN_CONNECTDONEEN BIT2 +#define DEVTEN_USBRSTEN BIT1 +#define DEVTEN_DISCONNEN BIT0 + +#define DSTS (DW_USB3_BASE + 0xC70C) +#define DSTS_GET_DEVSPD(x) ((x) & 0x7) + +#define DALEPENA (DW_USB3_BASE + 0xC720) + +#define DEPCMDPAR2(x) (DW_USB3_BASE + 0xC800 + ((x) & 0x1F) * 0x10) +#define DEPCMDPAR1(x) (DW_USB3_BASE + 0xC804 + ((x) & 0x1F) * 0x10) +#define DEPCMDPAR0(x) (DW_USB3_BASE + 0xC808 + ((x) & 0x1F) * 0x10) +#define DEPCMD(x) (DW_USB3_BASE + 0xc80C + ((x) & 0x1F) * 0x10) + +#define DEPCMD_COMMANDPARAM_MASK (0xFFFF << 16) +#define DEPCMD_COMMANDPARAM(x) (((x) & 0xFFFF) << 16) +/* Stream Number or uFrame (input) */ +#define DEPCMD_STR_NUM_OR_UF_MASK (0xFFFF << 16) +#define DEPCMD_STR_NUM_OR_UF(x) (((x) & 0xFFFF) << 16) +/* Transfer Resource Index (output) */ +#define DEPCMD_XFER_RSRC_IDX_SHIFT 16 +#define DEPCMD_XFER_RSRC_IDX_MASK (0x7F << 16) +#define DEPCMD_XFER_RSRC_IDX(x) (((x) & 0x7F) << 16) +#define GET_DEPCMD_XFER_RSRC_IDX(x) (((x) >> 16) & 0x7F) +#define DEPCMD_CMDACT BIT10 +#define DEPCMD_CMDTYPE_MASK 0xFF +#define DEPCMD_CMDTYPE(x) ((x) & 0xFF) + +/* EP registers range as: OUT0, IN0, OUT1, IN1, ... */ +#define EP_OUT_IDX(x) ((x) * 2) +#define EP_IN_IDX(x) (((x) * 2) + 1) + +#define CMDTYPE_SET_EP_CFG 1 +#define CMDTYPE_SET_XFER_CFG 2 +#define CMDTYPE_GET_EP_STATE 3 +#define CMDTYPE_SET_STALL 4 +#define CMDTYPE_CLR_STALL 5 +#define CMDTYPE_START_XFER 6 +#define CMDTYPE_UPDATE_XFER 7 +#define CMDTYPE_END_XFER 8 +#define CMDTYPE_START_NEW_CFG 9 + +#define EPTYPE_CONTROL 0 +#define EPTYPE_ISOC 1 +#define EPTYPE_BULK 2 +#define EPTYPE_INTR 3 + +#define CFG_ACTION_INIT 0 +#define CFG_ACTION_RESTORE 1 +#define CFG_ACTION_MODIFY 2 + +#define EPCFG0_CFG_ACTION_MASK (0x3 << 30) +#define EPCFG0_CFG_ACTION(x) (((x) & 0x3) << 30) +#define EPCFG0_BRSTSIZ_MASK (0xF << 22) +#define EPCFG0_BRSTSIZ(x) (((x) & 0xF) << 22) +#define EPCFG0_TXFNUM_MASK (0x1F << 17) +#define EPCFG0_TXFNUM(x) (((x) & 0x1F) << 17) +#define EPCFG0_MPS_MASK (0x7FF << 3) +#define EPCFG0_MPS(x) (((x) & 0x7FF) << 3) +#define EPCFG0_EPTYPE_MASK (0x3 << 1) +#define EPCFG0_EPTYPE_SHIFT 1 +#define EPCFG0_EPTYPE(x) (((x) & 0x3) << 1) + +/* Endpoint Number */ +#define EPCFG1_EP_NUM_MASK (0xF << 26) +#define EPCFG1_EP_NUM(x) (((x) & 0xF) << 26) +/* Endpoint Direction */ +#define EPCFG1_EP_DIR_IN BIT25 +/* Stream Not Ready */ +#define EPCFG1_XFER_NRDY BIT10 +/* XferInProgress Enable */ +#define EPCFG1_XFER_IN_PROG BIT9 +/* Stream Completed */ +#define EPCFG1_XFER_CMPL BIT8 + +#define USB_SPEED_UNKNOWN 0 +#define USB_SPEED_LOW 1 +#define USB_SPEED_FULL 2 +#define USB_SPEED_HIGH 3 +#define USB_SPEED_VARIABLE 4 +#define USB_SPEED_SUPER 5 + +// DMA registers +#define DSCSTS_TRBRSP_MASK (0xF << 28) +#define DSCSTS_TRBRSP(x) (((x) & 0xF) << 28) +#define GET_DSCSTS_TRBRSP(x) (((x) >> 28) & 0xF) +#define TRBRSP_MISSED_ISOC_IN 1 +#define TRBRSP_SETUP_PEND 2 +#define TRBRSP_XFER_IN_PROG 4 +#define DSCSTS_PCM1_MASK (0x3 << 24) +#define DSCSTS_PCM1(x) (((x) & 0x3) << 24) +#define DSCSTS_XFERCNT_MASK 0xFFFFFF +#define DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) +#define GET_DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) + +#define DSCCTL_STRMID_SOFN(x) (((x) & 0xFFFF) << 14) +#define DSCCTL_IOC BIT11 +#define DSCCTL_ISP BIT10 +#define DSCCTL_TRBCTL_MASK (0x3F << 4) +#define DSCCTL_TRBCTL(x) (((x) & 0x3F) << 4) +#define DSCCTL_LST BIT1 +#define DSCCTL_HWO BIT0 +#define TRBCTL_NORMAL 1 +#define TRBCTL_SETUP 2 +#define TRBCTL_STATUS_2 3 +#define TRBCTL_STATUS_3 4 +#define TRBCTL_CTLDATA_1ST 5 +#define TRBCTL_ISOC_1ST 6 +#define TRBCTL_ISOC 7 +#define TRBCTL_LINK 8 +#define TRBCTL_NORMAL_ZLP 9 + + +#define UE_DIR_IN 0x80 +#define UE_DIR_OUT 0 +#define UE_SET_DIR(a, d) ((a) | (((d) & 1) << 7)) +#define UE_GET_DIR(a) ((a) & 0x80) +#define UE_ADDR 0x0F +#define UE_GET_ADDR(a) ((a) & UE_ADDR) + +#define UT_GET_DIR(a) ((a) & 0x80) +#define UT_WRITE 0x00 +#define UT_READ 0x80 + +#define UT_GET_TYPE(a) ((a) & 0x60) +#define UT_STANDARD 0x00 +#define UT_CLASS 0x20 +#define UT_VENDOR 0x40 + +#define UT_GET_RECIPIENT(a) ((a) & 0x1f) +#define UT_DEVICE 0x00 +#define UT_INTERFACE 0x01 +#define UT_ENDPOINT 0x02 +#define UT_OTHER 0x03 + +#define UR_GET_STATUS 0x00 +#define UR_CLEAR_FEATURE 0x01 +#define UR_SET_FEATURE 0x03 +#define UR_SET_ADDRESS 0x05 +#define UR_GET_DESCRIPTOR 0x06 +#define UR_SET_DESCRIPTOR 0x07 +#define UR_GET_CONFIG 0x08 +#define UR_SET_CONFIG 0x09 +#define UR_GET_INTERFACE 0x0A +#define UR_SET_INTERFACE 0x0B +#define UR_SYNCH_FRAME 0x0C +#define UR_SET_SEL 0x30 +#define UR_SET_ISOC_DELAY 0x31 + +/* Feature numbers */ +#define UF_ENDPOINT_HALT 0 +#define UF_DEVICE_REMOTE_WAKEUP 1 +#define UF_TEST_MODE 2 +#define UF_DEVICE_B_HNP_ENABLE 3 +#define UF_DEVICE_A_HNP_SUPPORT 4 +#define UF_DEVICE_A_ALT_HNP_SUPPORT 5 +#define UF_FUNCTION_SUSPEND 0 +#define UF_U1_ENABLE 48 +#define UF_U2_ENABLE 49 +#define UF_LTM_ENABLE 50 + +#define UDESC_DEVICE 0x01 +#define UDESC_CONFIG 0x02 +#define UDESC_STRING 0x03 +#define UDESC_INTERFACE 0x04 +#define UDESC_ENDPOINT 0x05 +#define UDESC_SS_USB_COMPANION 0x30 +#define UDESC_DEVICE_QUALIFIER 0x06 +#define UDESC_BOS 0x0f +#define UDESC_DEVICE_CAPABILITY 0x10 + +#define STRING_LANGUAGE 0 +#define STRING_MANUFACTURER 1 +#define STRING_PRODUCT 2 +#define STRING_SERIAL 3 + +#define CONFIG_VALUE 1 + +#define USB3_BULK_IN_EP 1 +#define USB3_BULK_OUT_EP 1 + +#define USB_ENUM_ADB_PORT_VID 0x18D1 +#define USB_ENUM_ADB_PORT_PID 0xD00D +#define USB_ENUM_INTERFACE_ADB_SUBCLASS 0x42 +#define USB_ENUM_INTERFACE_ADB_PROTOCOL 0x03 + +struct usb3_pcd; + +typedef enum pcd_state { + USB3_STATE_UNCONNECTED, /* no host */ + USB3_STATE_DEFAULT, + USB3_STATE_ADDRESSED, + USB3_STATE_CONFIGURED, +} pcdstate_e; + +typedef enum ep0_state { + EP0_IDLE, + EP0_IN_DATA_PHASE, + EP0_OUT_DATA_PHASE, + EP0_IN_WAIT_NRDY, + EP0_OUT_WAIT_NRDY, + EP0_IN_STATUS_PHASE, + EP0_OUT_STATUS_PHASE, + EP0_STALL, +} ep0state_e; + +typedef struct usb3_dma_desc { + /** Buffer Pointer - Low address quadlet */ + UINT32 bptl; + + /** Buffer Pointer - High address quadlet */ + UINT32 bpth; + + /** Status quadlet. Fields defined in enum @ref desc_sts_data. */ + UINT32 status; + + /** Control quadlet. Fields defined in enum @ref desc_ctl_data. */ + UINT32 control; +} usb3_dma_desc_t; + +typedef struct usb3_pcd_req { + usb3_dma_desc_t *trb; + UINT64 trbdma; + + UINT32 length; + UINT32 actual; + + UINT64 *bufdma; + int (*complete)(unsigned actual, int status); +} usb3_pcd_req_t; + +typedef struct usb_device_request { + UINT8 bmRequestType; + UINT8 bRequest; + UINT16 wValue; + UINT16 wIndex; + UINT16 wLength; +} usb_device_request_t; + +#pragma pack(1) +/** USB_DT_DEVICE: Device descriptor */ +typedef struct usb_device_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT16 bcdUSB; +#define USB_CLASS_COMM 0x02 +#define USB_CLASS_VENDOR_SPEC 0xFF +#define USB_SC_VENDOR_SPEC 0xFF +#define USB_PR_VENDOR_SPEC 0xFF + UINT8 bDeviceClass; + UINT8 bDeviceSubClass; + UINT8 bDeviceProtocol; + UINT8 bMaxPacketSize0; + UINT16 idVendor; + UINT16 idProduct; + UINT16 bcdDevice; + UINT8 iManufacturer; + UINT8 iProduct; + UINT8 iSerialNumber; + UINT8 bNumConfigurations; +} usb_device_descriptor_t; + +/* USB_DT_CONFIG: Config descriptor */ +typedef struct usb_config_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT16 wTotalLength; + UINT8 bNumInterfaces; +#define CONFIG_VALUE 1 + UINT8 bConfigurationValue; + UINT8 iConfiguration; +#define USB_CONFIG_ATT_ONE (1 << 7) + UINT8 bmAttributes; +#define USB_CONFIG_VBUS_DRAW (0xFA) + UINT8 bMaxPower; +} usb_config_descriptor_t; + +/* USB_DT_DEVICE_QUALIFIER: Device Qualifier descriptor */ +typedef struct usb_qualifier_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT16 bcdUSB; + UINT8 bDeviceClass; + UINT8 bDeviceSubClass; + UINT8 bDeviceProtocol; + UINT8 bMaxPacketSize0; + UINT8 bNumConfigurations; + UINT8 bRESERVED; +} usb_qualifier_descriptor_t; + +/* USB_DT_INTERFACE: Interface descriptor */ +typedef struct usb_interface_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT8 bInterfaceNumber; + UINT8 bAlternateSetting; + UINT8 bNumEndpoints; + UINT8 bInterfaceClass; + UINT8 bInterfaceSubClass; + UINT8 bInterfaceProtocol; + UINT8 iInterface; +} usb_interface_descriptor_t; + +/* USB_DT_ENDPOINT: Endpoint descriptor */ +typedef struct usb_endpoint_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT8 bEndpointAddress; + UINT8 bmAttributes; +#define USB_ENDPOINT_XFER_CONTROL 0x00 +#define USB_ENDPOINT_XFER_ISOC 0x01 +#define USB_ENDPOINT_XFER_BULK 0x02 +#define USB_ENDPOINT_XFER_INT 0x03 + UINT16 wMaxPacketSize; + UINT8 bInterval; +} usb_endpoint_descriptor_t; + +/* USB_DT_SS_ENDPOINT_COMP: SuperSpeed Endpoint Companion descriptor */ +typedef struct usb_ss_ep_comp_descriptor { + UINT8 bLength; + UINT8 bDescriptorType; + + UINT8 bMaxBurst; + UINT8 bmAttributes; + UINT16 wBytesPerInterval; +} usb_ss_ep_comp_descriptor_t; + +/* WUSB BOS Descriptor (Binary device Object Store) */ +typedef struct wusb_bos_desc { + UINT8 bLength; + UINT8 bDescriptorType; + UINT16 wTotalLength; + UINT8 bNumDeviceCaps; +} wusb_bos_desc_t; + +#define USB_DEVICE_CAPABILITY_20_EXTENSION 0x02 +typedef struct usb_dev_cap_20_ext_desc { + UINT8 bLength; + UINT8 bDescriptorType; + UINT8 bDevCapabilityType; +#define USB_20_EXT_LPM 0x02 + UINT32 bmAttributes; +} usb_dev_cap_20_ext_desc_t; + +#define USB_DEVICE_CAPABILITY_SS_USB 0x03 +typedef struct usb_dev_cap_ss_usb { + UINT8 bLength; + UINT8 bDescriptorType; + UINT8 bDevCapabilityType; +#define USB_DC_SS_USB_LTM_CAPABLE 0x02 + UINT8 bmAttributes; +#define USB_DC_SS_USB_SPEED_SUPPORT_LOW 0x01 +#define USB_DC_SS_USB_SPEED_SUPPORT_FULL 0x02 +#define USB_DC_SS_USB_SPEED_SUPPORT_HIGH 0x04 +#define USB_DC_SS_USB_SPEED_SUPPORT_SS 0x08 + UINT32 wSpeedsSupported; + UINT8 bFunctionalitySupport; + UINT8 bU1DevExitLat; + UINT32 wU2DevExitLat; +} usb_dev_cap_ss_usb_t; + +#define USB_DEVICE_CAPABILITY_CONTAINER_ID 0x04 +typedef struct usb_dev_cap_container_id { + UINT8 bLength; + UINT8 bDescriptorType; + UINT8 bDevCapabilityType; + UINT8 bReserved; + UINT8 containerID[16]; +} usb_dev_cap_container_id_t; +#pragma pack() + +typedef union usb_setup_pkt { + usb_device_request_t req; + UINT32 d32[2]; + UINT8 d8[8]; +} usb_setup_pkt_t; + +typedef struct usb3_pcd_ep { + struct usb3_pcd *pcd; + + UINT8 EpInIdx; + UINT8 EpOutIdx; + UINT8 phys; + + //UINT8 phys; + UINT8 num; + UINT8 type; + UINT8 maxburst; + UINT16 maxpacket; + /* Tx FIFO # for IN EPs */ + UINT8 tx_fifo_num; + + /* The Transfer Resource Index from the Start Transfer command */ + UINT8 tri_out; + UINT8 tri_in; + + UINT8 stopped; + /* Send ZLP */ + UINT8 send_zlp; + /* True if 3-stage control transfer */ + UINT8 three_stage; + /* True if transfer has been started on EP */ + UINT8 xfer_started; + /* EP direction 0 = OUT */ + UINT8 is_in; + /* True if endpoint is active */ + UINT8 active; + /* Initial data pid of bulk endpoint */ + UINT8 data_pid_start; + + /* ep_desc (excluding ep0) */ + usb3_dma_desc_t *ep_desc; + +#if 0 + /* TRB descriptor must be aligned to 16 bytes */ + UINT8 epx_desc[32]; +#endif + + /* request (excluding ep0) */ + usb3_pcd_req_t req; +} usb3_pcd_ep_t; + +typedef struct usb3_pcd { + //struct usb3_device *usb3_dev; + + INT32 link_state; + pcdstate_e state; + UINT8 new_config; + ep0state_e ep0state; + + UINT32 eps_enabled; + UINT32 ltm_enable; + + usb3_pcd_ep_t ep0; + usb3_pcd_ep_t out_ep; + usb3_pcd_ep_t in_ep; + + /* + usb3_dev_global_regs_t *dev_global_regs; + usb3_dev_ep_regs_t *out_ep_regs; + usb3_dev_ep_regs_t *in_ep_regs; + */ + + usb3_pcd_req_t ep0_req; + + UINT8 speed; + + usb3_dma_desc_t *ep0_setup_desc; + usb3_dma_desc_t *ep0_in_desc; + usb3_dma_desc_t *ep0_out_desc; + + /* TRB descriptor must be aligned to 16 bytes */ +#if 0 + UINT8 ep0_setup[32]; + UINT8 ep0_in[32]; + UINT8 ep0_out[32]; + + usb_setup_pkt_t ep0_setup_pkt[5]; + +#define USB3_STATUS_BUF_SIZE 512 + UINT8 ep0_status_buf[USB3_STATUS_BUF_SIZE]; + +#define USB3_BULK_BUF_SIZE 2048 + UINT8 ss_bulk_buf[USB3_BULK_BUF_SIZE]; +#endif + + UINT32 file_type; + UINT32 file_address; + UINT32 file_capacity; + UINT32 file_total_frame; + UINT32 file_curr_frame; + UINT32 file_next_frame; + UINT32 file_received; + UINT32 file_complete; + + UINT16 test_mode_nr; + UINT16 test_mode; +} usb3_pcd_t; + +struct usb_enum_port_param { + UINT16 idVendor; + UINT16 idProduct; + UINT8 bInterfaceSubClass; + UINT8 bInterfaceProtocol; +}; + +#if 0 +typedef struct usb3_pcd_req { + usb3_dma_desc_t *trb; + UINT64 trbdma; + + UINT32 length; + UINT32 actual; + + UINT64 *bufdma; + int (*complete)(unsigned actual, int status); +} usb3_pcd_req_t; + +#endif + +#endif /* __DW_USB3_DXE_H__ */ diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c new file mode 100644 index 000000000000..83d5e4736de0 --- /dev/null +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c @@ -0,0 +1,2434 @@ +/** @file + + Copyright (c) 2018, Linaro Limited. All rights reserved. + + This program and the accompanying materials are licensed and made available + under the terms and conditions of the BSD License which accompanies this + distribution. The full text of the license may be found at + http://opensource.org/licenses/bsd-license.php + + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. + +**/ + +#include <IndustryStandard/Usb.h> +#include <Library/ArmLib.h> +#include <Library/BaseLib.h> +#include <Library/BaseMemoryLib.h> +#include <Library/CacheMaintenanceLib.h> +#include <Library/DebugLib.h> +#include <Library/DmaLib.h> +#include <Library/IoLib.h> +#include <Library/MemoryAllocationLib.h> +#include <Library/TimerLib.h> +#include <Library/UefiBootServicesTableLib.h> +#include <Library/UefiDriverEntryPoint.h> +#include <Library/UefiRuntimeServicesTableLib.h> +#include <Protocol/DwUsb.h> +#include <Protocol/DriverBinding.h> +#include <Protocol/UsbDevice.h> + +#include "DwUsb3Dxe.h" + +#define FIFO_DIR_TX 0 +#define FIFO_DIR_RX 1 + +#define TX_FIFO_ADDR 0 +#define RX_FIFO_ADDR 0 + +#define RAM_WIDTH 8 +#define RAM_TX0_DEPTH 2048 +#define RAM_TX1_DEPTH 4096 +#define RAM_RX_DEPTH 8192 + +#define USB_TYPE_LENGTH 16 +#define USB_BLOCK_HIGH_SPEED_SIZE 512 +#define DATA_SIZE 131072 +#define CMD_SIZE 512 +#define MATCH_CMD_LITERAL(Cmd, Buf) !AsciiStrnCmp (Cmd, Buf, sizeof (Cmd) - 1) + +#define ALIGN(x, a) (((x) + ((a) - 1)) & ~((a) - 1)) + +// +// The time between interrupt polls, in units of 100 nanoseconds +// 10 Microseconds +// +#define DW_INTERRUPT_POLL_PERIOD 100 + +#define DWUSB3_EVENT_BUF_SIZE 256 + +// +// Maxpacket size for EP0, defined by USB3 spec +// +#define USB3_MAX_EP0_SIZE 512 + +// +// Maxpacket size for any EP, defined by USB3 spec +// +#define USB3_MAX_PACKET_SIZE 1024 +#define USB2_HS_MAX_PACKET_SIZE 512 +#define USB2_FS_MAX_PACKET_SIZE 64 + +#define USB3_STATE_UNCONNECTED 0 +#define USB3_STATE_DEFAULT 1 +#define USB3_STATE_ADDRESSED 2 +#define USB3_STATE_CONFIGURED 3 + +#define USB3_STATUS_BUF_SIZE 512 + +#define GET_EVENTBUF_COUNT() \ + (GEVNTCOUNT_EVNTCOUNT (MmioRead32 (GEVNTCOUNT (0)))) +#define UPDATE_EVENTBUF_COUNT(x) \ + (MmioWrite32 (GEVNTCOUNT (0), GEVNTCOUNT_EVNTCOUNT (x))) + +#define SET_DEVADDR(x) \ + (MmioAndThenOr32 (DCFG, ~DCFG_DEVADDR_MASK, DCFG_DEVADDR (x))) + +EFI_GUID gDwUsbProtocolGuid = DW_USB_PROTOCOL_GUID; + +STATIC DW_USB_PROTOCOL *DwUsb; + +STATIC usb3_pcd_t gPcd; +STATIC UINT32 *gEventBuf, *gEventPtr; +STATIC struct usb_device_descriptor gDwUsb3DevDesc; +STATIC VOID *gRxBuf; + +STATIC usb_setup_pkt_t *gEndPoint0SetupPacket; +STATIC UINT8 *gEndPoint0StatusBuf; +STATIC USB_DEVICE_RX_CALLBACK mDataReceivedCallback; +STATIC UINTN mDataBufferSize; + +struct usb_interface_descriptor intf = { + sizeof (struct usb_interface_descriptor), + UDESC_INTERFACE, + 0, + 0, + 2, + USB_CLASS_VENDOR_SPEC, + 0x42, + 0x03, + 0 +}; + +const struct usb_ss_ep_comp_descriptor ep_comp = { + sizeof (struct usb_ss_ep_comp_descriptor), + UDESC_SS_USB_COMPANION, + 0, + 0, + 0 +}; + +const struct usb_endpoint_descriptor hs_bulk_in = { + sizeof (struct usb_endpoint_descriptor), + UDESC_ENDPOINT, + UE_DIR_IN | USB3_BULK_IN_EP, + USB_ENDPOINT_XFER_BULK, + 0x200, + 0 +}; + +const struct usb_endpoint_descriptor +hs_bulk_out = { + sizeof(struct usb_endpoint_descriptor), /* bLength */ + UDESC_ENDPOINT, /* bDescriptorType */ + + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ + 0x200, /* wMaxPacketSize: 512 of high-speed */ + 1, /* bInterval */ +}; + +const struct usb_endpoint_descriptor ss_bulk_in = { + sizeof(struct usb_endpoint_descriptor), /* bLength */ + UDESC_ENDPOINT, /* bDescriptorType */ + + UE_DIR_IN | USB3_BULK_IN_EP, /* bEndpointAddress */ + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ + 0x400, /* wMaxPacketSize: 1024 of super-speed */ + 0, /* bInterval */ +}; + +const struct usb_endpoint_descriptor ss_bulk_out = { + sizeof(struct usb_endpoint_descriptor), /* bLength */ + UDESC_ENDPOINT, /* bDescriptorType */ + + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ + 0x400, /* wMaxPacketSize: 1024 of super-speed */ + 0, /* bInterval */ +}; + +/** The BOS Descriptor */ + +const struct usb_dev_cap_20_ext_desc cap1 = { + sizeof(struct usb_dev_cap_20_ext_desc), /* bLength */ + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ + USB_DEVICE_CAPABILITY_20_EXTENSION, /* bDevCapabilityType */ + 0x2, /* bmAttributes */ +}; + +const struct usb_dev_cap_ss_usb +cap2 = { + sizeof(struct usb_dev_cap_ss_usb), /* bLength */ + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ + USB_DEVICE_CAPABILITY_SS_USB, /* bDevCapabilityType */ + 0x0, /* bmAttributes */ + (USB_DC_SS_USB_SPEED_SUPPORT_SS | + USB_DC_SS_USB_SPEED_SUPPORT_HIGH), /* wSpeedsSupported */ + 0x2, /* bFunctionalitySupport */ + 0xa, /* bU1DevExitLat */ + 0x100, /* wU2DevExitLat */ +}; + +const struct usb_dev_cap_container_id +cap3 = { + sizeof(struct usb_dev_cap_container_id),/* bLength */ + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ + USB_DEVICE_CAPABILITY_CONTAINER_ID, /* bDevCapabilityType */ + 0, /* bReserved */ + {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, /* containerID */ +}; + +const struct wusb_bos_desc +bos = { + sizeof(struct wusb_bos_desc), /* bLength */ + UDESC_BOS, /* bDescriptorType */ + (sizeof(struct wusb_bos_desc) + \ + sizeof(cap1) + sizeof(cap2) + \ + sizeof(cap3)), /* wTotalLength */ + 3, /* bNumDeviceCaps */ +}; + +STATIC struct usb_enum_port_param usb_port_activity_config = { + .idVendor = USB_ENUM_ADB_PORT_VID, + .idProduct = USB_ENUM_ADB_PORT_PID, + .bInterfaceSubClass = USB_ENUM_INTERFACE_ADB_SUBCLASS, + .bInterfaceProtocol = USB_ENUM_INTERFACE_ADB_PROTOCOL +}; + +STATIC +UINT32 +DwUsb3GetEventBufEvent ( + IN UINTN Size + ) +{ + UINT32 Event; + + Event = *gEventPtr++; + if ((UINT32)(UINTN)gEventPtr >= (UINT32)(UINTN)gEventBuf + Size) { + gEventPtr = gEventBuf; + } + return Event; +} + +STATIC +VOID +DwUsb3SetFifoSize ( + IN UINT32 Addr, + IN UINT32 Depth, + IN UINT32 Dir, + IN UINT32 FifoNum + ) +{ + UINT32 Reg = 0; + + if (Dir == FIFO_DIR_TX) { + Reg = GTXFIFOSIZ (FifoNum); + } else if (Dir == FIFO_DIR_RX) { + Reg = GRXFIFOSIZ (FifoNum); + } else { + ASSERT (0); + } + MmioWrite32 (Reg, FIFOSIZ_DEP (Depth) | FIFOSIZ_ADDR (Addr)); +} + +STATIC +UINT32 +Handshake ( + IN UINT32 Reg, + IN UINT32 Mask, + IN UINT32 Done + ) +{ + UINT32 Timeout = 100000; + + do { + if ((MmioRead32 (Reg) & Mask) == Done) { + return 1; + } + MicroSecondDelay (1); + } while (Timeout-- > 0); + return 0; +} + +STATIC +VOID +DwUsb3FillDesc ( + IN usb3_dma_desc_t *desc, + IN UINT64 dma_addr, + IN UINT32 dma_len, + IN UINT32 stream, + IN UINT32 type, + IN UINT32 ctrlbits, + IN UINT32 own + ) +{ + desc->bptl = (UINT32)(dma_addr & 0xFFFFFFFF); + desc->bpth = (UINT32)(dma_addr >> 32); + desc->status = DSCSTS_XFERCNT (dma_len); + if (type) { + desc->control = DSCCTL_TRBCTL (type); + } + desc->control |= DSCCTL_STRMID_SOFN (stream) | ctrlbits; + ArmDataSynchronizationBarrier (); + /* must execute this operation at last */ + if (own) { + desc->control |= DSCCTL_HWO; + } + ArmDataSynchronizationBarrier (); +} + +STATIC +VOID +DwUsb3DepStartNewCfg ( + IN UINT32 EpIdx, + IN UINT32 RsrcIdx + ) +{ + /* start the command */ + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_XFER_RSRC_IDX (RsrcIdx) | DEPCMD_CMDTYPE (CMDTYPE_START_NEW_CFG) | \ + DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +STATIC +VOID +DwUsb3DepCfg ( + IN UINT32 EpIdx, + IN UINT32 DepCfg0, + IN UINT32 DepCfg1, + IN UINT32 DepCfg2 + ) +{ + MmioWrite32 (DEPCMDPAR2 (EpIdx), DepCfg2); + MmioWrite32 (DEPCMDPAR1 (EpIdx), DepCfg1); + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepCfg0); + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_CMDTYPE (CMDTYPE_SET_EP_CFG) | DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +STATIC +VOID +DwUsb3DepXferCfg ( + IN UINT32 EpIdx, + IN UINT32 DepStrmCfg + ) +{ + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepStrmCfg); + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_CMDTYPE (CMDTYPE_SET_XFER_CFG) | DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +STATIC +UINT8 +DwUsb3DepStartXfer ( + IN UINT32 EpIdx, + IN UINT64 DmaAddr, + IN UINT32 StreamOrUf + ) +{ + UINT32 Data; + + MmioWrite32 (DEPCMDPAR1 (EpIdx), (UINT32)DmaAddr); + MmioWrite32 (DEPCMDPAR0 (EpIdx), (UINT32)(DmaAddr >> 32)); + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_STR_NUM_OR_UF (StreamOrUf) | DEPCMD_CMDTYPE (CMDTYPE_START_XFER) | \ + DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); + Data = MmioRead32 (DEPCMD (EpIdx)); + return GET_DEPCMD_XFER_RSRC_IDX(Data); +} + +STATIC +VOID +DwUsb3DepStopXfer ( + IN UINT32 EpIdx, + IN UINT32 Tri + ) +{ + MmioWrite32 (DEPCMDPAR2 (EpIdx), 0); + MmioWrite32 (DEPCMDPAR1 (EpIdx), 0); + MmioWrite32 (DEPCMDPAR0 (EpIdx), 0); + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_END_XFER) | \ + DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +VOID +DwUsb3DepUpdateXfer ( + IN UINT32 EpIdx, + IN UINT32 Tri + ) +{ + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_UPDATE_XFER) | \ + DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +STATIC +VOID +DwUsb3DepClearStall ( + IN UINTN EpIdx + ) +{ + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_CMDTYPE (CMDTYPE_CLR_STALL) | DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + + +STATIC +VOID +DwUsb3DepSetStall ( + IN UINTN EpIdx + ) +{ + MmioWrite32 ( + DEPCMD (EpIdx), + DEPCMD_CMDTYPE (CMDTYPE_SET_STALL) | DEPCMD_CMDACT + ); + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); +} + +STATIC +VOID +DwUsb3EnableEp ( + IN UINT32 EpIdx, + IN usb3_pcd_ep_t *ep + ) +{ + UINT32 Dalepena; + + Dalepena = MmioRead32 (DALEPENA); + /* If the EP is already enabled, skip to set it again. */ + if (Dalepena & (1 << EpIdx)) { + return; + } + Dalepena |= 1 << EpIdx; + MmioWrite32 (DALEPENA, Dalepena); +} + +STATIC +VOID +DwUsb3Ep0Activate ( + IN OUT usb3_pcd_t *pcd + ) +{ + /* issue DEPCFG command to EP0 OUT */ + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 0); + DwUsb3DepCfg ( + EP_OUT_IDX (0), + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512), + EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY, + 0 + ); + /* issue DEPSTRMCFG command to EP0 OUT */ + DwUsb3DepXferCfg (EP_OUT_IDX (0), 1); // one stream + /* issue DEPCFG command to EP0 IN */ + DwUsb3DepCfg ( + EP_IN_IDX (0), + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512) | \ + EPCFG0_TXFNUM (pcd->ep0.tx_fifo_num), + EPCFG1_XFER_NRDY | EPCFG1_XFER_CMPL | EPCFG1_EP_DIR_IN, + 0 + ); + /* issue DEPSTRMCFG command to EP0 IN */ + DwUsb3DepXferCfg (EP_IN_IDX (0), 1); // one stream + pcd->ep0.active = 1; +} + +STATIC +VOID +DwUsb3EpActivate ( + IN OUT usb3_pcd_t *pcd, + IN OUT usb3_pcd_ep_t *ep + ) +{ + UINT32 EpIdx, DepCfg0, DepCfg1; + if (ep->is_in) { + EpIdx = EP_IN_IDX (ep->num); + } else { + EpIdx = EP_OUT_IDX (ep->num); + } + + /* Start a new configurate when enable the first EP. */ + if (!pcd->eps_enabled) { + pcd->eps_enabled = 1; + /* Issue DEPCFG command to physical EP1 (logical EP0 IN) first. + * It resets the core's Tx FIFO mapping table. + */ + DepCfg0 = EPCFG0_EPTYPE (EPTYPE_CONTROL); + DepCfg0 |= EPCFG0_CFG_ACTION (CFG_ACTION_MODIFY); + DepCfg1 = EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY | EPCFG1_EP_DIR_IN; + + switch (pcd->speed) { + case USB_SPEED_SUPER: + DepCfg0 |= EPCFG0_MPS (512); + break; + case USB_SPEED_HIGH: + case USB_SPEED_FULL: + DepCfg0 |= EPCFG0_MPS (64); + break; + case USB_SPEED_LOW: + DepCfg0 |= EPCFG0_MPS (8); + break; + default: + ASSERT (0); + break; + } + DwUsb3DepCfg (EP_IN_IDX (0), DepCfg0, DepCfg1, 0); + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 2); + } + /* issue DEPCFG command to EP */ + DepCfg0 = EPCFG0_EPTYPE (ep->type); + DepCfg0 |= EPCFG0_MPS (ep->maxpacket); + if (ep->is_in) { + DepCfg0 |= EPCFG0_TXFNUM (ep->tx_fifo_num); + } + DepCfg0 |= EPCFG0_BRSTSIZ (ep->maxburst); + DepCfg1 = EPCFG1_EP_NUM (ep->num); + if (ep->is_in) { + DepCfg1 |= EPCFG1_EP_DIR_IN; + } else { + DepCfg1 |= EPCFG1_XFER_CMPL; + } + DwUsb3DepCfg (EpIdx, DepCfg0, DepCfg1, 0); + /* issue DEPSTRMCFG command to EP */ + DwUsb3DepXferCfg (EpIdx, 1); + DwUsb3EnableEp (EpIdx, ep); + ep->active = 1; +} + +STATIC +VOID +DwUsb3Ep0OutStart ( + IN usb3_pcd_t *pcd + ) +{ + usb3_dma_desc_t *desc; + + /* Get the SETUP packet DMA Descriptor (TRB) */ + desc = pcd->ep0_setup_desc; + + /* DMA Descriptor setup */ + DwUsb3FillDesc ( + desc, + (UINT64)gEndPoint0SetupPacket, + pcd->ep0.maxpacket, + 0, + TRBCTL_SETUP, + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, + 1 + ); + + /* issue DEPSTRTXFER command to EP0 OUT */ + pcd->ep0.tri_out = DwUsb3DepStartXfer (EP_OUT_IDX (0), (UINT64)desc, 0); +} + +STATIC +VOID +DwUsb3Init ( + VOID + ) +{ + UINT32 Data, Addr; + usb3_pcd_t *pcd = &gPcd; + + /* soft reset the usb core */ + do { + MmioAndThenOr32 (DCTL, ~DCTL_RUN_STOP, DCTL_CSFTRST); + + do { + MicroSecondDelay (1000); + Data = MmioRead32 (DCTL); + } while (Data & DCTL_CSFTRST); + // + // wait for at least 3 PHY clocks + // + MicroSecondDelay (1000); + } while (0); + + pcd->link_state = 0; + + /* TI PHY: Set Turnaround Time = 9 (8-bit UTMI+ / ULPI) */ + MmioAndThenOr32 ( + GUSB2PHYCFG (0), + ~GUSB2PHYCFG_USBTRDTIM_MASK, + GUSB2PHYCFG_USBTRDTIM (9) + ); + + /* set TX FIFO size */ + Addr = TX_FIFO_ADDR; + DwUsb3SetFifoSize (Addr, RAM_TX0_DEPTH / RAM_WIDTH, FIFO_DIR_TX, 0); + Addr += RAM_TX0_DEPTH / RAM_WIDTH; + DwUsb3SetFifoSize (Addr, RAM_TX1_DEPTH / RAM_WIDTH, FIFO_DIR_TX, 1); + /* set RX FIFO size */ + DwUsb3SetFifoSize (RX_FIFO_ADDR, RAM_RX_DEPTH / RAM_WIDTH, FIFO_DIR_RX, 0); + + /* set LFPS filter delay1trans */ + MmioAndThenOr32 ( + GUSB3PIPECTL (0), + ~PIPECTL_DELAYP1TRANS, + PIPECTL_LFPS_FILTER | PIPECTL_TX_DEMPH (1) + ); + + /* set GCTL */ + Data = GCTL_U2EXIT_LFPS | GCTL_PRTCAPDIR_DEVICE | GCTL_U2RSTECN | + GCTL_PWRDNSCALE(2); + MmioWrite32 (GCTL, Data); + + /* init event buf */ + MmioWrite32 (GEVNTADRL(0), (UINT32)(UINTN)gEventBuf); + MmioWrite32 (GEVNTADRH(0), (UINTN)gEventBuf >> 32); + MmioWrite32 (GEVNTSIZ(0), DWUSB3_EVENT_BUF_SIZE << 2); + MmioWrite32 (GEVNTCOUNT(0), 0); + + /* set max speed to super speed */ + MmioAndThenOr32 ( + DCFG, + ~DCFG_DEVSPD_MASK, + DCFG_DEVSPD (DEVSPD_SS_PHY_125MHZ_OR_250MHZ) + ); + + /* set nump */ + MmioAndThenOr32 (DCFG, ~DCFG_NUMP_MASK, DCFG_NUMP (16)); + + /* init address */ + SET_DEVADDR (0); + + /* disable phy suspend */ + MmioAnd32 (GUSB3PIPECTL (0), ~PIPECTL_SUSPEND_EN); + MmioAnd32 (GUSB2PHYCFG (0), ~GUSB2PHYCFG_SUSPHY); + + /* clear any pending interrupts */ + Data = MmioRead32 (GEVNTCOUNT (0)); + MmioWrite32 (GEVNTCOUNT (0), Data); + /* enable device interrupts */ + MmioWrite32 (DEVTEN, DEVTEN_CONNECTDONEEN | DEVTEN_USBRSTEN); + /* activate EP0 */ + DwUsb3Ep0Activate (pcd); + /* start EP0 to receive SETUP packets */ + DwUsb3Ep0OutStart (pcd); + + /* enable EP0 OUT/IN in DALEPENA */ + MmioWrite32 (DALEPENA, (1 << EP_OUT_IDX (0)) | (1 << EP_IN_IDX (0))); + + /* set RUN/STOP bit */ + MmioOr32 (DCTL, DCTL_RUN_STOP); +} + +STATIC +VOID +DriverInit ( + VOID + ) +{ + usb3_pcd_t *pcd = &gPcd; + usb3_pcd_ep_t *ep; + EFI_STATUS Status; + + pcd->speed = USB_SPEED_UNKNOWN; + + // + // init EP0 + // + ep = &pcd->ep0; + ep->pcd = pcd; + ep->stopped = 1; + ep->is_in = 0; + ep->active = 0; + ep->phys = 0; + ep->num = 0; + ep->tx_fifo_num = 0; + ep->type = EPTYPE_CONTROL; + ep->maxburst = 0; + ep->maxpacket = USB3_MAX_EP0_SIZE; + ep->send_zlp = 0; + ep->req.length = 0; + ep->req.actual = 0; + pcd->ep0_req.length = 0; + pcd->ep0_req.actual = 0; + + // + // init EP1 OUT + // + ep = &pcd->out_ep; + ep->pcd = pcd; + ep->stopped = 1; + ep->is_in = 0; + ep->active = 0; + ep->phys = USB3_BULK_OUT_EP << 1; + ep->num = 1; + ep->tx_fifo_num = 0; + // + // bulk ep is activated + // + ep->type = EPTYPE_BULK; + ep->maxburst = 0; + ep->maxpacket = USB3_MAX_PACKET_SIZE; + ep->send_zlp = 0; + ep->req.length = 0; + ep->req.actual = 0; + + // + // init EP1 IN + // + ep = &pcd->in_ep; + ep->stopped = 1; + ep->is_in = 1; + ep->active = 0; + ep->phys = (USB3_BULK_IN_EP << 1) | 1; + ep->num = 1; + ep->tx_fifo_num = USB3_BULK_IN_EP; + // + // bulk ep is activated + // + ep->type = EPTYPE_BULK; + ep->maxburst = 0; + ep->maxpacket = USB3_MAX_PACKET_SIZE; + ep->send_zlp = 0; + ep->req.length = 0; + ep->req.actual = 0; + + pcd->ep0state = EP0_IDLE; + pcd->ep0.maxpacket = USB3_MAX_EP0_SIZE; + pcd->ep0.type = EPTYPE_CONTROL; + + Status = DmaAllocateBuffer ( + EfiBootServicesData, + EFI_SIZE_TO_PAGES (sizeof (usb3_dma_desc_t) * 5), + (VOID *)&pcd->ep0_setup_desc + ); + if (EFI_ERROR (Status)) { + return; + } + pcd->ep0_in_desc = pcd->ep0_setup_desc++; + pcd->ep0_out_desc = pcd->ep0_in_desc++; + pcd->in_ep.ep_desc = pcd->ep0_out_desc++; + pcd->out_ep.ep_desc = pcd->in_ep.ep_desc++; +} + +STATIC +VOID +DwUsb3HandleUsbResetInterrupt ( + IN usb3_pcd_t *pcd + ) +{ + usb3_pcd_ep_t *ep; + + // + // clear stall on each EP + // + ep = &pcd->in_ep; + if (ep->xfer_started) { + if (ep->is_in) { + DwUsb3DepStopXfer (EP_IN_IDX (ep->num), ep->tri_in); + } else { + DwUsb3DepStopXfer (EP_OUT_IDX (ep->num), ep->tri_out); + } + } + if (ep->stopped) { + if (ep->is_in) { + DwUsb3DepClearStall (EP_IN_IDX (ep->num)); + } else { + DwUsb3DepClearStall (EP_OUT_IDX (ep->num)); + } + } + + ep = &pcd->out_ep; + if (ep->xfer_started) { + if (ep->is_in) { + DwUsb3DepStopXfer (EP_IN_IDX (ep->num), ep->tri_in); + } else { + DwUsb3DepStopXfer (EP_OUT_IDX (ep->num), ep->tri_out); + } + } + if (ep->stopped) { + if (ep->is_in) { + DwUsb3DepClearStall (EP_IN_IDX (ep->num)); + } else { + DwUsb3DepClearStall (EP_OUT_IDX (ep->num)); + } + } + + // + // set device address to 0 + // + SET_DEVADDR (0); + + pcd->ltm_enable = 0; + DEBUG ((DEBUG_INFO, "usb reset\n")); +} + +STATIC +UINT32 +DwUsb3GetDeviceSpeed ( + IN usb3_pcd_t *pcd + ) +{ + UINT32 Data, Speed; + + Data = MmioRead32 (DSTS); + switch (DSTS_GET_DEVSPD (Data)) { + case DEVSPD_HS_PHY_30MHZ_OR_60MHZ: + Speed = USB_SPEED_HIGH; + break; + case DEVSPD_FS_PHY_30MHZ_OR_60MHZ: + case DEVSPD_FS_PHY_48MHZ: + Speed = USB_SPEED_FULL; + break; + case DEVSPD_LS_PHY_6MHZ: + Speed = USB_SPEED_LOW; + break; + case DEVSPD_SS_PHY_125MHZ_OR_250MHZ: + Speed = USB_SPEED_SUPER; + break; + default: + DEBUG ((DEBUG_ERROR, "DwUsb3GetDeviceSpeed: invalid DSTS:0x%x\n", Data)); + Speed = USB_SPEED_UNKNOWN; + break; + } + return Speed; +} + +STATIC +VOID +DwUsb3PcdSetSpeed ( + IN usb3_pcd_t *pcd, + IN UINTN speed + ) +{ + // + // set the MPS of EP0 based on the connection speed + // + switch (speed) { + case USB_SPEED_SUPER: + pcd->ep0.maxpacket = 512; + pcd->in_ep.maxpacket = USB3_MAX_PACKET_SIZE; + pcd->out_ep.maxpacket = USB3_MAX_PACKET_SIZE; + break; + case USB_SPEED_HIGH: + pcd->ep0.maxpacket = 64; + pcd->in_ep.maxpacket = USB2_HS_MAX_PACKET_SIZE; + pcd->out_ep.maxpacket = USB2_HS_MAX_PACKET_SIZE; + break; + case USB_SPEED_FULL: + pcd->ep0.maxpacket = 64; + pcd->in_ep.maxpacket = USB2_FS_MAX_PACKET_SIZE; + pcd->out_ep.maxpacket = USB2_FS_MAX_PACKET_SIZE; + break; + default: + DEBUG ((DEBUG_ERROR, "invalid speed: %d\n", speed)); + break; + } +} + +STATIC +VOID +DwUsb3HandleConnectDoneInterrupt ( + IN usb3_pcd_t *pcd + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + UINT32 DiepCfg0, DoepCfg0, DiepCfg1, DoepCfg1; + UINT32 Speed; + + ep0->stopped = 0; + Speed = (UINT32)DwUsb3GetDeviceSpeed (pcd); + pcd->speed = (UINT8)Speed; + + DwUsb3PcdSetSpeed (pcd, Speed); + // + // set the MPS of EP0 based on the connection speed + // + DiepCfg0 = EPCFG0_EPTYPE (EPTYPE_CONTROL) | \ + EPCFG0_CFG_ACTION (CFG_ACTION_MODIFY); + DiepCfg1 = EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY | \ + EPCFG1_EP_DIR_IN; + DoepCfg0 = EPCFG0_EPTYPE (EPTYPE_CONTROL) | \ + EPCFG0_CFG_ACTION (CFG_ACTION_MODIFY); + DoepCfg1 = EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY; + + switch (Speed) { + case USB_SPEED_SUPER: + DiepCfg0 |= EPCFG0_MPS (512); + DoepCfg0 |= EPCFG0_MPS (512); + break; + case USB_SPEED_HIGH: + case USB_SPEED_FULL: + DiepCfg0 |= EPCFG0_MPS (64); + DoepCfg0 |= EPCFG0_MPS (64); + break; + case USB_SPEED_LOW: + DiepCfg0 |= EPCFG0_MPS (8); + DoepCfg0 |= EPCFG0_MPS (8); + break; + default: + DEBUG (( + DEBUG_ERROR, + "DwUsb3HandleConnectDoneInterrupt: invalid speed %d\n", + Speed + )); + break; + } + DiepCfg0 |= EPCFG0_TXFNUM (ep0->tx_fifo_num); + // + // issue DEPCFG command to EP0 OUT + // + DwUsb3DepCfg (EP_OUT_IDX (0), DoepCfg0, DoepCfg1, 0); + // + // issue DEPCFG command to EP0 IN + // + DwUsb3DepCfg (EP_IN_IDX (0), DiepCfg0, DiepCfg1, 0); + pcd->state = USB3_STATE_DEFAULT; +} + +STATIC +VOID +DwUsb3HandleDeviceInterrupt ( + IN usb3_pcd_t *pcd, + IN UINT32 Event + ) +{ + switch (Event & GEVNT_DEVT_MASK) { + case GEVNT_DEVT_USBRESET: + DwUsb3HandleUsbResetInterrupt (pcd); + break; + case GEVNT_DEVT_CONNDONE: + DwUsb3HandleConnectDoneInterrupt (pcd); + break; + default: + DEBUG ((DEBUG_ERROR, "DwUsb3HandleDeviceInterrupt: invalid event\n")); + break; + } +} + +STATIC +usb3_pcd_ep_t * +DwUsb3GetOutEndPoint ( + IN usb3_pcd_t *pcd, + IN UINT32 EndPointNum + ) +{ + if (EndPointNum == 0) { + return &pcd->ep0; + } + return &pcd->out_ep; +} + +STATIC +usb3_pcd_ep_t * +DwUsb3GetInEndPoint ( + IN usb3_pcd_t *pcd, + IN UINT32 EndPointNum + ) +{ + if (EndPointNum == 0) { + return &pcd->ep0; + } + return &pcd->in_ep; +} + +STATIC +VOID +EndPoint0DoStall ( + IN usb3_pcd_t *pcd + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + + // + // stall EP0 IN & OUT simultanelusly + // + ep0->is_in = 1; + DwUsb3DepSetStall (EP_IN_IDX (0)); + ep0->is_in = 0; + DwUsb3DepSetStall (EP_OUT_IDX (0)); + // + // prepare for the next setup transfer + // + ep0->stopped = 1; + pcd->ep0state = EP0_IDLE; + DwUsb3Ep0OutStart (pcd); +} + +STATIC +VOID +EndPoint0ContinueTransfer ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_req_t *req + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + usb3_dma_desc_t *desc; + UINT64 desc_dma; + UINT8 tri; + + // + // send a 0-byte length packet after the end of transfer + // + if (ep0->is_in) { + desc = pcd->ep0_in_desc; + desc_dma = (UINT64)pcd->ep0_in_desc; + // + // DMA descriptor setup + // + DwUsb3FillDesc ( + desc, + (UINT64)req->bufdma, + 0, + 0, + TRBCTL_NORMAL, + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, + 1 + ); + tri = DwUsb3DepStartXfer (EP_IN_IDX (0), desc_dma, 0); + ep0->tri_in = tri; + } +} + +STATIC +VOID +EndPoint0CompleteRequest ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_req_t *req, + IN usb3_dma_desc_t *desc + ) +{ + usb3_pcd_ep_t *ep = &pcd->ep0; + + if (req == NULL) { + return; + } + + if ((pcd->ep0state == EP0_OUT_DATA_PHASE) || + (pcd->ep0state == EP0_IN_DATA_PHASE)) { + if (ep->is_in) { + if (GET_DSCSTS_XFERCNT (desc->status) == 0) { + pcd->ep0.is_in = 0; + pcd->ep0state = EP0_OUT_WAIT_NRDY; + } + } else { + pcd->ep0.is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; + } + } +} + +STATIC +VOID +DwUsb3OsGetTrb ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep, + IN usb3_pcd_req_t *req + ) +{ + // + // If EP0, fill request with EP0 IN/OUT data TRB + // + if (ep == &pcd->ep0) { + if (ep->is_in) { + req->trb = pcd->ep0_in_desc; + req->trbdma = (UINT64)pcd->ep0_in_desc; + } else { + req->trb = pcd->ep0_out_desc; + req->trbdma = (UINT64)pcd->ep0_out_desc; + } + } else { + // + // fill request with TRB from the non-EP0 allocation + // + req->trb = ep->ep_desc; + req->trbdma = (UINT64)ep->ep_desc; + } +} + +STATIC +VOID +DwUsb3EndPoint0StartTransfer ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_req_t *req + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + usb3_dma_desc_t *desc; + UINT64 desc_dma; + UINT32 desc_type, len; + + // + // get the DMA descriptor (TRB) for this request + // + DwUsb3OsGetTrb (pcd, ep0, req); + desc = req->trb; + desc_dma = req->trbdma; + + if (ep0->is_in) { + // + // start DMA on EP0 IN + // DMA Descriptor (TRB) setup + // + len = req->length; + if (pcd->ep0state == EP0_IN_STATUS_PHASE) { + if (ep0->three_stage) { + desc_type = TRBCTL_STATUS_3; + } else { + desc_type = TRBCTL_STATUS_2; + } + } else { + desc_type = TRBCTL_CTLDATA_1ST; + } + DwUsb3FillDesc ( + desc, + (UINT64)req->bufdma, + len, + 0, + desc_type, + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, + 1 + ); + // + // issue DEPSTRTXFER command to EP0 IN + // + ep0->tri_in = DwUsb3DepStartXfer (EP_IN_IDX (0), desc_dma, 0); + } else { + // + // start DMA on EP0 OUT + // DMA Descriptor (TRB) setup + // + len = ALIGN (req->length, ep0->maxpacket); + if (pcd->ep0state == EP0_OUT_STATUS_PHASE) { + if (ep0->three_stage) { + desc_type = TRBCTL_STATUS_3; + } else { + desc_type = TRBCTL_STATUS_2; + } + } else { + desc_type = TRBCTL_CTLDATA_1ST; + } + DwUsb3FillDesc ( + desc, + (UINT64)req->bufdma, + len, + 0, + desc_type, + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, + 1 + ); + // + // issue DEPSTRTXFER command to EP0 OUT + // + ep0->tri_out = DwUsb3DepStartXfer (EP_OUT_IDX (0), desc_dma, 0); + } +} + +STATIC +INTN +DwUsb3EndPointXStartTransfer ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep + ) +{ + usb3_pcd_req_t *req = &ep->req; + usb3_dma_desc_t *desc; + UINT64 desc_dma; + UINT32 len; + + // + // get the TRB for this request + // + DwUsb3OsGetTrb (pcd, ep, req); + desc = req->trb; + desc_dma = req->trbdma; + + if (ep->is_in) { + // + // For IN, TRB length is just xfer length + // + len = req->length; + if (ep->xfer_started && !(desc->control & DSCCTL_HWO)) { + DEBUG ((DEBUG_INFO, "[%a] last tx succ, but not in 10s!\n", __func__)); + ep->xfer_started = 0; + } + } else { + // + // For OUT, TRB length must be multiple of maxpacket + // must be power of 2, use cheap AND + // + len = (req->length + ep->maxpacket - 1) & ~(ep->maxpacket - 1); + req->length = len; + } + // + // DMA descriptor setup + // + DwUsb3FillDesc ( + desc, + (UINT64)req->bufdma, + len, + 0, + TRBCTL_NORMAL, + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, + 1 + ); + if (ep->is_in) { + // + // start DMA on EPn IN + // + if (ep->xfer_started) { + // + // issue DEPUPDTXFER command to EP + // + DwUsb3DepUpdateXfer (EP_IN_IDX (ep->num), ep->tri_in); + } else { + ep->tri_in = DwUsb3DepStartXfer (EP_IN_IDX (ep->num), desc_dma, 0); + ep->xfer_started = 1; + } + } else { + // + // start DMA on EPn OUT + // + if (ep->xfer_started) { + // + // issue DEPUPDTXFER command to EP + // + DwUsb3DepUpdateXfer (EP_OUT_IDX (ep->num), ep->tri_out); + } else { + ep->tri_out = DwUsb3DepStartXfer (EP_OUT_IDX (ep->num), desc_dma, 0); + ep->xfer_started = 1; + } + } + if (ep->is_in) { + UINT32 count = 0; + // + // wait until send complete + // + while ((desc->control & DSCCTL_HWO) && (count < 1000000)) { + MicroSecondDelay (10); + count++; + } + if (count >= 1000000) { + DEBUG ((DEBUG_INFO, "[%a]: ep%d transfer timeout!\n", __func__, ep->num)); + DEBUG (( + DEBUG_INFO, + "please disconnect then connect USB cable again to recovery!\n" + )); + return -1; + } + ep->xfer_started = 0; + } + return 0; +} + +STATIC +VOID +SetupInStatusPhase ( + IN usb3_pcd_t *pcd, + IN VOID *buf + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + + if (pcd->ep0state == EP0_STALL) + return; + + ep0->is_in = 1; + pcd->ep0state = EP0_IN_STATUS_PHASE; + pcd->ep0_req.bufdma = buf; + pcd->ep0_req.length = 0; + pcd->ep0_req.actual = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); +} + +STATIC +VOID +SetupOutStatusPhase ( + IN usb3_pcd_t *pcd, + IN VOID *buf + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + + if (pcd->ep0state == EP0_STALL) + return; + + ep0->is_in = 0; + pcd->ep0state = EP0_OUT_STATUS_PHASE; + pcd->ep0_req.bufdma = buf; + pcd->ep0_req.length = 0; + pcd->ep0_req.actual = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); +} + +STATIC +VOID +DwUsb3HandleEndPoint0 ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_req_t *req, + IN UINT32 event + ) +{ + usb3_pcd_ep_t *ep0 = &pcd->ep0; + usb3_dma_desc_t *desc = NULL; + UINT32 byte_count, len; + + switch (pcd->ep0state) { + case EP0_IN_DATA_PHASE: + if (req == NULL) { + req = &pcd->ep0_req; + } + desc = pcd->ep0_in_desc; + + if (desc->control & DSCCTL_HWO) { + goto out; + } + + if (GET_DSCSTS_TRBRSP (desc->status) == TRBRSP_SETUP_PEND) { + // + // start of a new control transfer + // + desc->status = 0; + } + byte_count = req->length - GET_DSCSTS_XFERCNT (desc->status); + req->actual += byte_count; + req->bufdma += byte_count; + + if (req->actual < req->length) { + // + // IN CONTINUE, stall EP0 + // + EndPoint0DoStall (pcd); + } else if (ep0->send_zlp) { + // + // CONTINUE TRANSFER IN ZLP + // + EndPoint0ContinueTransfer (pcd, req); + ep0->send_zlp = 0; + } else { + // + // COMPLETE IN TRANSFER + // + EndPoint0CompleteRequest (pcd, req, desc); + } + break; + case EP0_OUT_DATA_PHASE: + if (req == NULL) { + req = &pcd->ep0_req; + } + desc = pcd->ep0_out_desc; + + if (desc->control & DSCCTL_HWO) { + goto out; + } + + if (GET_DSCSTS_TRBRSP (desc->status) == TRBRSP_SETUP_PEND) { + // + // start of a new control transfer + // + desc->status = 0; + } + len = (req->length + ep0->maxpacket - 1) & ~(ep0->maxpacket - 1); + byte_count = len - GET_DSCSTS_XFERCNT (desc->status); + req->actual += byte_count; + req->bufdma += byte_count; + + if (req->actual < req->length) { + // + // IN CONTINUE, stall EP0 + // + EndPoint0DoStall (pcd); + } else if (ep0->send_zlp) { + // + // CONTINUE TRANSFER IN ZLP + // + EndPoint0ContinueTransfer (pcd, req); + ep0->send_zlp = 0; + } else { + // + // COMPLETE IN TRANSFER + // + EndPoint0CompleteRequest (pcd, req, desc); + } + break; + case EP0_IN_WAIT_NRDY: + if (ep0->is_in) { + SetupInStatusPhase (pcd, gEndPoint0SetupPacket); + } else { + ASSERT (0); + } + break; + case EP0_OUT_WAIT_NRDY: + if (!ep0->is_in) { + SetupOutStatusPhase (pcd, gEndPoint0SetupPacket); + } else { + ASSERT (0); + } + break; + case EP0_IN_STATUS_PHASE: + if (ep0->is_in) { + desc = pcd->ep0_in_desc; + } else { + ASSERT (0); + } + EndPoint0CompleteRequest (pcd, req, desc); + pcd->ep0state = EP0_IDLE; + ep0->stopped = 1; + ep0->is_in = 0; // OUT for next SETUP + // + // prepare for more SETUP packets + // + DwUsb3Ep0OutStart (pcd); + break; + case EP0_OUT_STATUS_PHASE: + if (!ep0->is_in) { + desc = pcd->ep0_out_desc; + } else { + ASSERT (0); + } + EndPoint0CompleteRequest (pcd, req, desc); + pcd->ep0state = EP0_IDLE; + ep0->stopped = 1; + ep0->is_in = 0; // OUT for next SETUP + // + // prepare for more SETUP packets + // + DwUsb3Ep0OutStart (pcd); + break; + case EP0_STALL: + break; + case EP0_IDLE: + break; + default: + DEBUG ((DEBUG_ERROR, "%a: invalid state %d\n", __func__, pcd->ep0state)); + break; + } +out: + return; +} + +STATIC +usb3_pcd_ep_t * +Addr2EndPoint ( + IN usb3_pcd_t *pcd, + IN UINT16 index + ) +{ + UINT32 ep_num; + + ep_num = UE_GET_ADDR (index); + if (ep_num == 0) { + return &pcd->ep0; + } else { + if (UE_GET_DIR (index) == UE_DIR_IN) { + return &pcd->in_ep; + } + return &pcd->out_ep; + } +} + +STATIC +VOID +DwUsb3DoGetStatus ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + UINT8 *status = gEndPoint0StatusBuf; + usb3_pcd_ep_t *ep; + + if (ctrl->wLength != 2) { + EndPoint0DoStall (pcd); + return; + } + + switch (UT_GET_RECIPIENT (ctrl->bmRequestType)) { + case UT_DEVICE: + *status = 0; // bus powered + if (pcd->speed == USB_SPEED_SUPER) { + if (pcd->state == USB3_STATE_CONFIGURED) { + if (MmioRead32 (DCTL) & DCTL_INIT_U1_EN) { + *status |= 1 << 2; + } + if (MmioRead32 (DCTL) & DCTL_INIT_U2_EN) { + *status |= 1 << 3; + } + *status |= (UINT8)(pcd->ltm_enable << 4); + } + } + *(status + 1) = 0; + break; + case UT_INTERFACE: + *status = 0; + *(status + 1) = 0; + break; + case UT_ENDPOINT: + ep = Addr2EndPoint (pcd, ctrl->wIndex); + *status = ep->stopped; + *(status + 1) = 0; + break; + default: + EndPoint0DoStall (pcd); + return; + } + pcd->ep0_req.bufdma = (UINT64 *)status; + pcd->ep0_req.length = 2; + pcd->ep0_req.actual = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); +} + +STATIC +VOID +DoClearHalt ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep + ) +{ + if (ep->is_in) { + DwUsb3DepClearStall (EP_IN_IDX (ep->num)); + } else { + DwUsb3DepClearStall (EP_OUT_IDX (ep->num)); + } + if (ep->stopped) { + ep->stopped = 0; + } +} + +STATIC +VOID +Usb3PcdEpEnable ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep + ) +{ + // + // activate the EP + // + ep->stopped = 0; + ep->xfer_started = 0; + ep->ep_desc->control = 0; + ep->ep_desc->status = 0; + // + // set initial data pid. + // + if (ep->type == EPTYPE_BULK) { + ep->data_pid_start = 0; + } + DwUsb3EpActivate (pcd, ep); +} + +STATIC +VOID +DwUsb3DoClearFeature ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + usb3_pcd_ep_t *ep; + + switch (UT_GET_RECIPIENT (ctrl->bmRequestType)) { + case UT_DEVICE: + switch (ctrl->wValue) { + case UF_U1_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED)) { + EndPoint0DoStall (pcd); + return; + } + MmioAnd32 (DCTL, ~DCTL_INIT_U1_EN); + break; + case UF_U2_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED)) { + EndPoint0DoStall (pcd); + return; + } + MmioAnd32 (DCTL, ~DCTL_INIT_U2_EN); + break; + case UF_LTM_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED) || + (ctrl->wIndex != 0)) { + EndPoint0DoStall (pcd); + return; + } + pcd->ltm_enable = 0; + break; + default: + EndPoint0DoStall (pcd); + return; + } + break; + case UT_INTERFACE: + if (ctrl->wValue) { + EndPoint0DoStall (pcd); + return; + } + break; + case UT_ENDPOINT: + ep = Addr2EndPoint (pcd, ctrl->wIndex); + if (ctrl->wValue != UF_ENDPOINT_HALT) { + EndPoint0DoStall (pcd); + return; + } + DoClearHalt (pcd, ep); + break; + default: + DEBUG ((DEBUG_ERROR, "invalid bmRequestType :%d\n", UT_GET_RECIPIENT (ctrl->bmRequestType))); + break; + } + pcd->ep0.is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; +} + +STATIC +VOID +DwUsb3DoSetFeature ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + usb3_pcd_ep_t *ep; + + switch (UT_GET_RECIPIENT (ctrl->bmRequestType)) { + case UT_DEVICE: + switch (ctrl->wValue) { + case UF_DEVICE_REMOTE_WAKEUP: + break; + case UF_TEST_MODE: + pcd->test_mode_nr = ctrl->wIndex >> 8; + pcd->test_mode = 1; + break; + case UF_U1_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED)) { + EndPoint0DoStall (pcd); + return; + } + MmioOr32 (DCTL, DCTL_INIT_U1_EN); + break; + case UF_U2_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED)) { + EndPoint0DoStall (pcd); + return; + } + MmioOr32 (DCTL, DCTL_INIT_U2_EN); + break; + case UF_LTM_ENABLE: + if ((pcd->speed != USB_SPEED_SUPER) || + (pcd->state != USB3_STATE_CONFIGURED) || + (ctrl->wIndex != 0)) { + EndPoint0DoStall (pcd); + return; + } + pcd->ltm_enable = 1; + break; + default: + EndPoint0DoStall (pcd); + return; + } + break; + case UT_INTERFACE: + if (ctrl->wValue) { + EndPoint0DoStall (pcd); + return; + } + break; + case UT_ENDPOINT: + ep = Addr2EndPoint (pcd, ctrl->wIndex); + if (ctrl->wValue != UF_ENDPOINT_HALT) { + EndPoint0DoStall (pcd); + return; + } + ep->stopped = 1; + if (ep->is_in) { + DwUsb3DepClearStall (EP_IN_IDX (ep->num)); + } else { + DwUsb3DepClearStall (EP_OUT_IDX (ep->num)); + } + break; + default: + DEBUG ((DEBUG_ERROR, "invalid bmRequestType %d\n", UT_GET_RECIPIENT (ctrl->bmRequestType))); + break; + } + pcd->ep0.is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; +} + +STATIC +VOID +DwUsb3DoSetAddress ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + + if (ctrl->bmRequestType == UT_DEVICE) { + SET_DEVADDR (ctrl->wValue); + pcd->ep0.is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; + if (ctrl->wValue) { + pcd->state = USB3_STATE_ADDRESSED; + } else { + pcd->state = USB3_STATE_DEFAULT; + } + } +} + +STATIC +VOID +DwUsb3DoGetConfig ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + UINT8 *status = gEndPoint0StatusBuf; + + if (ctrl->bmRequestType != (UT_READ | UT_STANDARD | UT_DEVICE)) { + EndPoint0DoStall (pcd); + return; + } + // + // Notify host the current config value + // + *status = pcd->new_config; + pcd->ep0_req.bufdma = (UINT64 *)status; + pcd->ep0_req.length = 1; + pcd->ep0_req.actual = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); +} + +STATIC +VOID +DwUsb3DoSetConfig ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + UINT16 wvalue = ctrl->wValue; + usb3_pcd_ep_t *ep; + + if (ctrl->bmRequestType != (UT_WRITE | UT_STANDARD | UT_DEVICE)) { + EndPoint0DoStall (pcd); + return; + } + + if (!wvalue || (wvalue == CONFIG_VALUE)) { + pcd->new_config = (UINT8)wvalue; + // + // set new configuration + // + if (wvalue) { + // + // activate bulk in endpoint + // + ep = &pcd->in_ep; + Usb3PcdEpEnable (pcd, ep); + // + // activate bulk out endpoint + // + ep = &pcd->out_ep; + Usb3PcdEpEnable (pcd, ep); + pcd->state = USB3_STATE_CONFIGURED; + { + // + // prepare for EP1 OUT + // + usb3_pcd_ep_t *ep = &pcd->out_ep; + usb3_pcd_req_t *req = &ep->req; + + // + // AndroidFast App will free the rx buffer. + // + gRxBuf = AllocatePool (DATA_SIZE); + ASSERT (gRxBuf != NULL); + WriteBackDataCacheRange (gRxBuf, DATA_SIZE); + req->bufdma = (UINT64 *)gRxBuf; + if (mDataBufferSize == 0) { + req->length = CMD_SIZE; + } else if (mDataBufferSize > DATA_SIZE) { + req->length = DATA_SIZE; + mDataBufferSize = mDataBufferSize - DATA_SIZE; + } else if (mDataBufferSize > CMD_SIZE) { + req->length = CMD_SIZE; + mDataBufferSize = mDataBufferSize - CMD_SIZE; + } else { + req->length = mDataBufferSize; + mDataBufferSize = 0; + } + DwUsb3EndPointXStartTransfer (pcd, ep); + } + } else { + pcd->state = USB3_STATE_ADDRESSED; + } + pcd->ep0.is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; + } else { + EndPoint0DoStall (pcd); + } +} + +STATIC +VOID +DwUsb3DoGetDescriptor ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + UINT8 dt = ctrl->wValue >> 8; + UINT8 index = (UINT8)ctrl->wValue; + UINT16 len = ctrl->wLength; + UINT8 *buf = gEndPoint0StatusBuf; + UINT16 value = 0; + EFI_USB_STRING_DESCRIPTOR *Descriptor = NULL; + + if (ctrl->bmRequestType != (UT_READ | UT_STANDARD | UT_DEVICE)) { + EndPoint0DoStall (pcd); + return; + } + + switch (dt) { + case UDESC_DEVICE: + { + struct usb_device_descriptor *dev = &gDwUsb3DevDesc; + dev->bLength = sizeof (struct usb_device_descriptor); + dev->bDescriptorType = UDESC_DEVICE; + dev->bDeviceClass = 0; + dev->bDeviceSubClass = 0; + dev->bDeviceProtocol = 0; + if (pcd->speed == USB_SPEED_SUPER) { + dev->bcdUSB = 0x300; + // + // 2^9 = 512 + // + dev->bMaxPacketSize0 = 9; + } else { + dev->bcdUSB = 0x0200; + dev->bMaxPacketSize0 = 0x40; + } + dev->idVendor = usb_port_activity_config.idVendor; + dev->idProduct = usb_port_activity_config.idProduct; + dev->bcdDevice = 0x0100; + dev->iManufacturer = STRING_MANUFACTURER; + dev->iProduct = STRING_PRODUCT; + dev->iSerialNumber = STRING_SERIAL; + dev->bNumConfigurations = 1; + value = sizeof (struct usb_device_descriptor); + CopyMem ((void *)buf, (void *)dev, value); + } + break; + case UDESC_DEVICE_QUALIFIER: + { + struct usb_qualifier_descriptor *qual = (struct usb_qualifier_descriptor *)buf; + struct usb_device_descriptor *dev = &gDwUsb3DevDesc; + + qual->bLength = sizeof (*qual); + qual->bDescriptorType = UDESC_DEVICE_QUALIFIER; + qual->bcdUSB = dev->bcdUSB; + qual->bDeviceClass = dev->bDeviceClass; + qual->bDeviceSubClass = dev->bDeviceSubClass; + qual->bDeviceProtocol = dev->bDeviceProtocol; + qual->bMaxPacketSize0 = dev->bMaxPacketSize0; + qual->bNumConfigurations = 1; + qual->bRESERVED = 0; + value = sizeof (struct usb_qualifier_descriptor); + } + break; + + case UDESC_CONFIG: + { + struct usb_config_descriptor *config = (struct usb_config_descriptor *)buf; + + config->bLength = sizeof (*config); + config->bDescriptorType = UDESC_CONFIG; + config->bNumInterfaces = 1; + config->bConfigurationValue = 1; + config->iConfiguration = 0; + config->bmAttributes = USB_CONFIG_ATT_ONE; + + if (pcd->speed == USB_SPEED_SUPER) { + config->bMaxPower = 0x50; + } else { + config->bMaxPower = 0x80; + } + buf += sizeof (*config); + + intf.bInterfaceSubClass = usb_port_activity_config.bInterfaceSubClass; + intf.bInterfaceProtocol = usb_port_activity_config.bInterfaceProtocol; + CopyMem ((void *)buf, (void *)&intf, sizeof (intf)); + buf += sizeof (intf); + + switch (pcd->speed) { + case USB_SPEED_SUPER: + CopyMem (buf, &ss_bulk_in, sizeof (ss_bulk_in)); + buf += sizeof (ss_bulk_in); + CopyMem (buf, &ep_comp, sizeof (ep_comp)); + buf += sizeof (ep_comp); + CopyMem (buf, &ss_bulk_out, sizeof (ss_bulk_out)); + buf += sizeof (ss_bulk_out); + CopyMem (buf, &ep_comp, sizeof (ep_comp)); + + config->wTotalLength = sizeof (*config) + sizeof (intf) + sizeof (ss_bulk_in) + + sizeof (ep_comp) + sizeof (ss_bulk_out) + sizeof (ep_comp); + break; + + default: // HS/FS + { + struct usb_endpoint_descriptor *endp = (struct usb_endpoint_descriptor *)buf; + + CopyMem (buf, &hs_bulk_in, sizeof (hs_bulk_in)); + (endp++)->wMaxPacketSize = pcd->in_ep.maxpacket; + buf += sizeof (hs_bulk_in); + CopyMem (buf, &hs_bulk_out, sizeof (hs_bulk_out)); + endp->wMaxPacketSize = pcd->out_ep.maxpacket; + config->wTotalLength = sizeof (*config) + sizeof (intf) + sizeof (hs_bulk_in) + + sizeof (hs_bulk_out); + break; + } + } + value = config->wTotalLength; + } + break; + + case UDESC_STRING: + { + switch (index) { + case STRING_LANGUAGE: + Descriptor = (EFI_USB_STRING_DESCRIPTOR *)(UINTN)gEndPoint0StatusBuf; + ASSERT (Descriptor != NULL); + Descriptor->Length = LANG_LENGTH * sizeof (CHAR16); + Descriptor->DescriptorType = USB_DESC_TYPE_STRING; + DwUsb->GetLang (Descriptor->String, &Descriptor->Length); + value = Descriptor->Length; + break; + case STRING_MANUFACTURER: + Descriptor = (EFI_USB_STRING_DESCRIPTOR *)(UINTN)gEndPoint0StatusBuf; + ASSERT (Descriptor != NULL); + Descriptor->Length = MANU_FACTURER_STRING_LENGTH * sizeof (CHAR16); + Descriptor->DescriptorType = USB_DESC_TYPE_STRING; + DwUsb->GetManuFacturer (Descriptor->String, &Descriptor->Length); + value = Descriptor->Length; + break; + case STRING_PRODUCT: + Descriptor = (EFI_USB_STRING_DESCRIPTOR *)(UINTN)gEndPoint0StatusBuf; + ASSERT (Descriptor != NULL); + Descriptor->Length = PRODUCT_STRING_LENGTH * sizeof (CHAR16); + Descriptor->DescriptorType = USB_DESC_TYPE_STRING; + DwUsb->GetProduct (Descriptor->String, &Descriptor->Length); + value = Descriptor->Length; + break; + case STRING_SERIAL: + Descriptor = (EFI_USB_STRING_DESCRIPTOR *)(UINTN)gEndPoint0StatusBuf; + ASSERT (Descriptor != NULL); + Descriptor->Length = SERIAL_STRING_LENGTH * sizeof (CHAR16); + Descriptor->DescriptorType = USB_DESC_TYPE_STRING; + DwUsb->GetSerialNo (Descriptor->String, &Descriptor->Length); + value = Descriptor->Length; + break; + default: + EndPoint0DoStall (pcd); + break; + } + } + break; + + case UDESC_BOS: + if (pcd->speed != USB_SPEED_SUPER) { + EndPoint0DoStall (pcd); + return; + } + value = bos.wTotalLength; + CopyMem (buf, &bos, sizeof (bos)); + buf += sizeof (bos); + CopyMem (buf, &cap1, sizeof (cap1)); + buf += sizeof (cap1); + CopyMem (buf, &cap2, sizeof (cap2)); + buf += sizeof (cap2); + CopyMem (buf, &cap3, sizeof (cap3)); + break; + default: + EndPoint0DoStall (pcd); + return; + } + pcd->ep0_req.bufdma = (UINT64 *)gEndPoint0StatusBuf; + pcd->ep0_req.length = value < len ? value : len; + pcd->ep0_req.actual = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); +} + +STATIC +VOID +DwUsb3DoSetup ( + IN usb3_pcd_t *pcd + ) +{ + usb_device_request_t *ctrl = &gEndPoint0SetupPacket->req; + usb3_pcd_ep_t *ep0 = &pcd->ep0; + UINT16 wLength; + + wLength = ctrl->wLength; + ep0->stopped = 0; + ep0->three_stage = 1; + if (ctrl->bmRequestType & UE_DIR_IN) { + ep0->is_in = 1; + pcd->ep0state = EP0_IN_DATA_PHASE; + } else { + ep0->is_in = 0; + pcd->ep0state = EP0_OUT_DATA_PHASE; + } + + if (wLength == 0) { + ep0->is_in = 1; + pcd->ep0state = EP0_IN_WAIT_NRDY; + ep0->three_stage = 0; + } + if (UT_GET_TYPE (ctrl->bmRequestType) != UT_STANDARD) { + EndPoint0DoStall (pcd); + return; + } + + switch (ctrl->bRequest) { + case UR_GET_STATUS: + DwUsb3DoGetStatus (pcd); + break; + case UR_CLEAR_FEATURE: + DwUsb3DoClearFeature (pcd); + break; + case UR_SET_FEATURE: + DwUsb3DoSetFeature (pcd); + break; + case UR_SET_ADDRESS: + DwUsb3DoSetAddress (pcd); + break; + case UR_SET_CONFIG: + DwUsb3DoSetConfig (pcd); + MmioOr32 (DCTL, DCTL_ACCEPT_U1_EN); + MmioOr32 (DCTL, DCTL_ACCEPT_U2_EN); + DEBUG ((DEBUG_INFO, "enum done")); + pcd->ltm_enable = 0; + break; + case UR_GET_CONFIG: + DwUsb3DoGetConfig (pcd); + break; + case UR_GET_DESCRIPTOR: + DwUsb3DoGetDescriptor (pcd); + break; + case UR_SET_SEL: + // + // for now this is a no-op + // + pcd->ep0_req.bufdma = (UINT64 *)gEndPoint0StatusBuf; + pcd->ep0_req.length = USB3_STATUS_BUF_SIZE; + pcd->ep0_req.actual = 0; + ep0->send_zlp = 0; + DwUsb3EndPoint0StartTransfer (pcd, &pcd->ep0_req); + break; + default: + EndPoint0DoStall (pcd); + break; + } +} + +STATIC +VOID +DwUsb3OsHandleEndPoint0 ( + IN usb3_pcd_t *pcd, + IN UINT32 event + ) +{ + if (pcd->ep0state == EP0_IDLE) { + DwUsb3DoSetup (pcd); + } else { + DwUsb3HandleEndPoint0 (pcd, NULL, event); + } +} + +STATIC +VOID +DwUsb3RequestDone ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep, + IN usb3_pcd_req_t *req, + IN UINTN status + ) +{ + if (ep != &pcd->ep0) { + req->trb = NULL; + } + if (req->complete) { + req->complete (req->actual, status); + } else { + if (!ep->is_in) { + ASSERT (req->actual <= req->length); + InvalidateDataCacheRange (gRxBuf, req->actual); + mDataReceivedCallback (req->actual, gRxBuf); + } + } + req->actual = 0; +} + +STATIC +VOID +DwUsb3EndPointcompleteRequest ( + IN usb3_pcd_t *pcd, + IN usb3_pcd_ep_t *ep, + IN UINT32 event + ) +{ + usb3_pcd_req_t *req = &ep->req; + usb3_dma_desc_t *desc = req->trb; + UINT32 byte_count; + + ep->send_zlp = 0; + if (!desc) { + return; + } + + if (desc->control & DSCCTL_HWO) { + return; + } + + if (ep->is_in) { + // + // IN ep + // + if (GET_DSCSTS_XFERCNT (desc->status) == 0) { + req->actual += req->length; + } + // + // reset IN tri + // + ep->tri_in = 0; + // + // complete the IN request + // + DwUsb3RequestDone (pcd, ep, req, 0); + } else { + // + // OUT ep + // + byte_count = req->length - GET_DSCSTS_XFERCNT (desc->status); + req->actual += byte_count; + // + //req->bufdma += byte_count; + // reset OUT tri + // + ep->tri_out = 0; + // + // OUT transfer complete or not + // complete the OUT request + // + DwUsb3RequestDone (pcd, ep, req, 0); + { + // + // prepare for EP1 OUT + // + usb3_pcd_ep_t *ep = &pcd->out_ep; + usb3_pcd_req_t *req = &ep->req; + + ZeroMem (req, sizeof (usb3_pcd_req_t)); + gRxBuf = AllocatePool (DATA_SIZE); + ASSERT (gRxBuf != NULL); + WriteBackDataCacheRange (gRxBuf, DATA_SIZE); + req->bufdma = (UINT64 *)gRxBuf; + if (mDataBufferSize == 0) { + req->length = CMD_SIZE; + } else if (mDataBufferSize > DATA_SIZE) { + req->length = DATA_SIZE; + mDataBufferSize = mDataBufferSize - DATA_SIZE; + } else if (mDataBufferSize > CMD_SIZE) { + req->length = CMD_SIZE; + mDataBufferSize = mDataBufferSize - CMD_SIZE; + } else { + req->length = mDataBufferSize; + mDataBufferSize = 0; + } + DwUsb3EndPointXStartTransfer (pcd, ep); + } + } +} + +STATIC +VOID +DwUsb3HandleEndPointInterrupt ( + IN usb3_pcd_t *pcd, + IN UINTN PhySep, + IN UINT32 event + ) +{ + usb3_pcd_ep_t *ep; + UINT32 epnum, is_in; + + // + // Physical Out EPs are even, physical In EPs are odd + // + is_in = (UINT32)PhySep & 1; + epnum = ((UINT32)PhySep >> 1) & 0xF; + + // + // Get the EP pointer + // + if (is_in) { + ep = DwUsb3GetInEndPoint (pcd, epnum); + } else { + ep = DwUsb3GetOutEndPoint (pcd, epnum); + } + + switch (event & GEVNT_DEPEVT_INTTYPE_MASK) { + case GEVNT_DEPEVT_INTTYPE_XFER_CMPL: + ep->xfer_started = 0; + // + // complete the transfer + // + if (epnum == 0) { + DwUsb3OsHandleEndPoint0 (pcd, event); + } else { + DwUsb3EndPointcompleteRequest (pcd, ep, event); + } + break; + case GEVNT_DEPEVT_INTTYPE_XFER_IN_PROG: + break; + case GEVNT_DEPEVT_INTTYPE_XFER_NRDY: + if (epnum == 0) { + switch (pcd->ep0state) { + case EP0_IN_WAIT_NRDY: + if (is_in) { + DwUsb3OsHandleEndPoint0 (pcd, event); + } else { + } + break; + case EP0_OUT_WAIT_NRDY: + if (!is_in) { + DwUsb3OsHandleEndPoint0 (pcd, event); + } else { + } + break; + default: + break; + } + } else { + } + break; + default: + DEBUG (( + DEBUG_ERROR, + "invalid event %d\n", + event & GEVNT_DEPEVT_INTTYPE_MASK + )); + break; + } +} + +STATIC +UINTN +DwUsb3HandleEvent ( + VOID + ) +{ + usb3_pcd_t *pcd = &gPcd; + UINT32 Count, Index, Event, Intr; + UINT32 PhySep; + + Count = GET_EVENTBUF_COUNT (); + // + // reset event buffer when it's full + // + if ((GEVNTCOUNT_EVNTCOUNT (Count) == GEVNTCOUNT_EVNTCOUNT_MASK) || + (Count >= DWUSB3_EVENT_BUF_SIZE * sizeof (UINT32))) { + UPDATE_EVENTBUF_COUNT (Count); + Count = 0; + } + + for (Index = 0; Index < Count; Index += sizeof (UINT32)) { + Event = DwUsb3GetEventBufEvent (DWUSB3_EVENT_BUF_SIZE << 2); + UPDATE_EVENTBUF_COUNT (sizeof (UINT32)); + if (Event == 0) { + // + // ignore null events + // + continue; + } + if (Event & GEVNT_NON_EP) { + Intr = Event & GEVNT_INTTYPE_MASK; + if (Intr == GEVNT_INTTYPE (EVENT_DEV_INT)) { + DwUsb3HandleDeviceInterrupt (pcd, Event); + } + } else { + PhySep = (Event & GEVNT_DEPEVT_EPNUM_MASK) >> GEVNT_DEPEVT_EPNUM_SHIFT; + DwUsb3HandleEndPointInterrupt (pcd, PhySep, Event); + } + } + return 0; +} + +STATIC +VOID +DwUsb3Poll ( + IN EFI_EVENT Event, + IN VOID *Context + ) +{ + if (DwUsb3HandleEvent ()) { + DEBUG ((DEBUG_ERROR, "error: exit from usb_poll\n")); + return; + } +} + +EFI_STATUS +EFIAPI +DwUsb3Start ( + IN USB_DEVICE_DESCRIPTOR *DeviceDescriptor, + IN VOID **Descriptors, + IN USB_DEVICE_RX_CALLBACK RxCallback, + IN USB_DEVICE_TX_CALLBACK TxCallback + ) +{ + EFI_STATUS Status; + EFI_EVENT TimerEvent; + + Status = DmaAllocateBuffer ( + EfiBootServicesData, + EFI_SIZE_TO_PAGES (DWUSB3_EVENT_BUF_SIZE << 2), + (VOID *)&gEventBuf + ); + if (EFI_ERROR (Status)) { + return Status; + } + if (gEventBuf == NULL) { + return EFI_OUT_OF_RESOURCES; + } + ZeroMem (gEventBuf, EFI_SIZE_TO_PAGES (DWUSB3_EVENT_BUF_SIZE << 2)); + gEventPtr = gEventBuf; + DriverInit (); + DwUsb3Init (); + Status = gBS->CreateEvent ( + EVT_TIMER | EVT_NOTIFY_SIGNAL, + TPL_CALLBACK, + DwUsb3Poll, + NULL, + &TimerEvent + ); + ASSERT_EFI_ERROR (Status); + if (EFI_ERROR (Status)) { + return Status; + } + + Status = gBS->SetTimer ( + TimerEvent, + TimerPeriodic, + DW_INTERRUPT_POLL_PERIOD + ); + ASSERT_EFI_ERROR (Status); + mDataReceivedCallback = RxCallback; + return Status; +} + +EFI_STATUS +DwUsb3Send ( + IN UINT8 EndpointIndex, + IN UINTN Size, + IN CONST VOID *Buffer + ) +{ + usb3_pcd_t *pcd = &gPcd; + usb3_pcd_ep_t *ep = &pcd->in_ep; + usb3_pcd_req_t *req = &ep->req; + + WriteBackDataCacheRange ((VOID *)Buffer, Size); + req->bufdma = (UINT64 *)Buffer; + req->length = Size; + DwUsb3EndPointXStartTransfer (pcd, ep); + return EFI_SUCCESS; +} + +USB_DEVICE_PROTOCOL mUsbDevice = { + DwUsb3Start, + DwUsb3Send +}; + +EFI_STATUS +EFIAPI +DwUsb3EntryPoint ( + IN EFI_HANDLE ImageHandle, + IN EFI_SYSTEM_TABLE *SystemTable + ) +{ + EFI_STATUS Status; + + Status = DmaAllocateBuffer ( + EfiBootServicesData, + EFI_SIZE_TO_PAGES (sizeof (usb_setup_pkt_t) * 5), + (VOID *)&gEndPoint0SetupPacket + ); + if (EFI_ERROR (Status)) { + return Status; + } + Status = DmaAllocateBuffer ( + EfiBootServicesData, + EFI_SIZE_TO_PAGES (USB3_STATUS_BUF_SIZE * sizeof (UINT8)), + (VOID *)&gEndPoint0StatusBuf + ); + if (EFI_ERROR (Status)) { + goto Out; + } + Status = gBS->LocateProtocol (&gDwUsbProtocolGuid, NULL, (VOID **) &DwUsb); + if (EFI_ERROR (Status)) { + goto OutProtocol; + } + + Status = DwUsb->PhyInit(USB_DEVICE_MODE); + if (EFI_ERROR (Status)) { + goto OutProtocol; + } + + Status = gBS->InstallProtocolInterface ( + &ImageHandle, + &gUsbDeviceProtocolGuid, + EFI_NATIVE_INTERFACE, + &mUsbDevice + ); + if (EFI_ERROR (Status)) { + goto OutProtocol; + } + return Status; + +OutProtocol: + DmaFreeBuffer ( + EFI_SIZE_TO_PAGES (USB3_STATUS_BUF_SIZE * sizeof (UINT8)), + (VOID *)gEndPoint0StatusBuf + ); +Out: + DmaFreeBuffer ( + EFI_SIZE_TO_PAGES (sizeof (usb_setup_pkt_t) * 5), + (VOID *)gEndPoint0SetupPacket + ); + return Status; +} -- 2.7.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver 2018-08-20 10:31 ` [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver Haojian Zhuang @ 2018-10-04 16:32 ` Leif Lindholm 2018-10-22 2:51 ` Haojian Zhuang 0 siblings, 1 reply; 9+ messages in thread From: Leif Lindholm @ 2018-10-04 16:32 UTC (permalink / raw) To: Haojian Zhuang; +Cc: edk2-devel, Ard Biesheuvel On Mon, Aug 20, 2018 at 06:31:25PM +0800, Haojian Zhuang wrote: > Add Designware USB 3.0 device driver. It's focus on USB device > functionality, not USB Host functionality. The USB driver is > mainly used for Android Fastboot application. > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> > --- > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec | 44 + > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf | 52 + > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h | 632 +++++ > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c | 2434 ++++++++++++++++++++ > 4 files changed, 3162 insertions(+) > > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > new file mode 100644 > index 000000000000..038e4881a948 > --- /dev/null > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > @@ -0,0 +1,44 @@ > +#/** @file > +# Framework Module Development Environment Industry Standards Please change this to reflect this package rather than EmbeddedPkg one (which appears to have stolen it from an old version of MdePkg.dec :) > +# > +# This Package provides headers and libraries that conform to EFI/PI Industry standards. > +# Copyright (c) 2007, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2012-2014, ARM Ltd. All rights reserved.<BR> I see no need to keep these copyright statements. > +# Copyright (c) 2018, Linaro. All rights reserved.<BR> > +# > +# This program and the accompanying materials are licensed and made available under > +# the terms and conditions of the BSD License which accompanies this distribution. > +# The full text of the license may be found at > +# http://opensource.org/licenses/bsd-license.php > +# > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > +# > +#**/ > + > +[Defines] > + DEC_SPECIFICATION = 0x00010019 > + PACKAGE_NAME = DwUsb3DxePkg > + PACKAGE_GUID = 9b7aa6fe-405b-4955-af1f-5faf183aedec > + PACKAGE_VERSION = 0.1 > + > + > +################################################################################ > +# > +# Include Section - list of Include Paths that are provided by this package. > +# Comments are used for Keywords and Module Types. > +# > +# Supported Module Types: > +# BASE SEC PEI_CORE PEIM DXE_CORE DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER DXE_SAL_DRIVER UEFI_DRIVER UEFI_APPLICATION > +# > +################################################################################ > + > +[Guids.common] > + gDwUsb3DxeTokenSpaceGuid = { 0x098a50d3, 0x92e2, 0x461a, { 0xb6, 0xba, 0xf8, 0x5d, 0x9d, 0x89, 0x49, 0xf5 }} > + > +[Protocols.common] > + gDwUsbProtocolGuid = { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} Please make this file a common .dec for DwUsb and DwUsb3 instead of duplicating guid information. (If we weren't moving it to edk2-platforms, it would still have been better to add it to EmbeddedPkg.dec.) > + > +[PcdsFixedAtBuild.common] > + # DwUsb Driver PCDs > + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress|0x0|UINT32|0x00000001 > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf > new file mode 100644 > index 000000000000..510b51a34de7 > --- /dev/null > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf > @@ -0,0 +1,52 @@ > +#/** @file > +# > +# Copyright (c) 2018, Linaro Limited. All rights reserved. > +# > +# This program and the accompanying materials > +# are licensed and made available under the terms and conditions of the BSD License > +# which accompanies this distribution. The full text of the license may be found at > +# http://opensource.org/licenses/bsd-license.php > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > +# > +# > +#**/ > + > +[Defines] > + INF_VERSION = 0x00010019 > + BASE_NAME = DwUsb3Dxe > + FILE_GUID = 0879cd34-c399-4315-9891-56024072e711 > + MODULE_TYPE = UEFI_DRIVER > + VERSION_STRING = 1.0 > + ENTRY_POINT = DwUsb3EntryPoint > + > +[Sources.common] > + DwUsb3Dxe.c > + > +[LibraryClasses] > + CacheMaintenanceLib > + DebugLib > + DmaLib > + IoLib > + MemoryAllocationLib > + TimerLib > + UefiBootServicesTableLib > + UefiDriverEntryPoint > + UsbSerialNumberLib I don't see this library anywhere in the tree? > + > +[Protocols] > + gEfiDriverBindingProtocolGuid > + gUsbDeviceProtocolGuid > + > +[Packages] > + ArmPkg/ArmPkg.dec > + EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > + EmbeddedPkg/EmbeddedPkg.dec > + MdePkg/MdePkg.dec > + MdeModulePkg/MdeModulePkg.dec > + > +[Pcd] > + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress > + > +[Depex] > + TRUE > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h > new file mode 100644 > index 000000000000..92b610553169 > --- /dev/null > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h > @@ -0,0 +1,632 @@ > +/** @file > + > + Copyright (c) 2017, Linaro Limited. All rights reserved. > + > + This program and the accompanying materials > + are licensed and made available under the terms and conditions of the BSD License > + which accompanies this distribution. The full text of the license may be found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > + > +**/ > + > +#ifndef __DW_USB3_DXE_H__ > +#define __DW_USB3_DXE_H__ DW -> DESIGNWARE (throughout). > + > +#define DW_USB3_BASE FixedPcdGet32 (PcdDwUsb3DxeBaseAddress) > + > +#define GSBUCFG0 (DW_USB3_BASE + 0xC100) > +#define GCTL (DW_USB3_BASE + 0xC110) > + > +#define GCTL_PWRDNSCALE_MASK (0x1FFF << 19) > +#define GCTL_PWRDNSCALE(x) (((x) & 0x1FFF) << 19) > +#define GCTL_U2RSTECN BIT16 > +#define GCTL_PRTCAPDIR_MASK (BIT13 | BIT12) > +#define GCTL_PRTCAPDIR_HOST BIT12 > +#define GCTL_PRTCAPDIR_DEVICE BIT13 > +#define GCTL_PRTCAPDIR_OTG (BIT13 | BIT12) > +#define GCTL_U2EXIT_LFPS BIT2 > + > +#define GUSB2PHYCFG(x) (DW_USB3_BASE + 0xC200 + (((x) & 0xF) << 2)) > + > +#define GUSB2PHYCFG_USBTRDTIM_MASK (0xF << 10) > +#define GUSB2PHYCFG_USBTRDTIM(x) (((x) & 0xF) << 10) > +#define GUSB2PHYCFG_SUSPHY BIT6 > + > +#define GUSB3PIPECTL(x) (DW_USB3_BASE + 0xC2C0 + (((x) & 0x3) << 2)) > + > +#define PIPECTL_DELAYP1TRANS BIT18 > +#define PIPECTL_SUSPEND_EN BIT17 > +#define PIPECTL_LFPS_FILTER BIT9 > +#define PIPECTL_TX_DEMPH_MASK (0x3 << 1) > +#define PIPECTL_TX_DEMPH(x) (((x) & 0x3) << 1) > + > +#define GTXFIFOSIZ(x) (DW_USB3_BASE + 0xC300 + (((x) & 0x1F) << 2)) > +#define GRXFIFOSIZ(x) (DW_USB3_BASE + 0xC380 + (((x) & 0x1F) << 2)) > + > +#define FIFOSIZ_ADDR(x) (((x) & 0xFFFF) << 16) > +#define FIFOSIZ_DEP(x) ((x) & 0xFFFF) > + > +#define GEVNTADRL(x) (DW_USB3_BASE + 0xC400 + (((x) & 0x1F) << 2)) > +#define GEVNTADRH(x) (DW_USB3_BASE + 0xC404 + (((x) & 0x1F) << 2)) > +#define GEVNTSIZ(x) (DW_USB3_BASE + 0xC408 + (((x) & 0x1F) << 2)) > + > +#define GEVNTSIZ_EVNTINTMASK BIT31 > +#define GEVNTSIZ_EVNTSIZ_MASK (0xFFFF) > +#define GEVNTSIZ_EVNTSIZ(x) ((x) & 0xFFFF) > + > +#define GEVNTCOUNT(x) (DW_USB3_BASE + 0xC40C + (((x) & 0x1F) << 2)) > +#define GEVNTCOUNT_EVNTCOUNT_MASK (0xFFFF) > +#define GEVNTCOUNT_EVNTCOUNT(x) ((x) & 0xFFFF) > + > +// Non-Endpoint specific event flag > +#define GEVNT_INTTYPE_MASK (0x7F << 1) > +#define GEVNT_INTTYPE(x) (((x) & 0x7F) << 1) > +#define EVENT_I2C_INT 4 > +#define EVENT_CARKIT_INT 3 > +#define EVENT_OTG_INT 1 > +#define EVENT_DEV_INT 0 > + > +#define GEVNT_NON_EP BIT0 > +// Endpoint specific event flag > +#define GEVNT_DEPEVT_INTTYPE_MASK (0xF << 6) > +#define GEVNT_DEPEVT_INTTYPE(x) (((x) & 0xF) << 6) > +#define GEVNT_DEPEVT_INTTYPE_EPCMD_CMPL (7 << 6) > +#define GEVNT_DEPEVT_INTTYPE_STRM_EVT (6 << 6) > +#define GEVNT_DEPEVT_INTTYPE_FIFOXRUN (4 << 6) > +#define GEVNT_DEPEVT_INTTYPE_XFER_NRDY (3 << 6) > +#define GEVNT_DEPEVT_INTTYPE_XFER_IN_PROG (2 << 6) > +#define GEVNT_DEPEVT_INTTYPE_XFER_CMPL (1 << 6) > +#define GEVNT_DEPEVT_EPNUM_MASK (0x1F << 1) > +#define GEVNT_DEPEVT_EPNUM_SHIFT 1 > +#define GEVNT_DEPEVT_EPNUM(x) (((x) & 0x1F) << 1) > +// Devices specific event flag > +#define GEVNT_DEVT_MASK (0xF << 8) > +#define GEVNT_DEVT_SHIFT 8 > +#define GEVNT_DEVT(x) (((x) & 0xF) << 8) > +#define GEVNT_DEVT_INACT_TIMEOUT_RCVD (0x15 << 8) > +#define GEVNT_DEVT_VNDR_DEV_TST_RCVD (0x14 << 8) > +#define GEVNT_DEVT_OVERFLOW (0x13 << 8) > +#define GEVNT_DEVT_CMD_CMPL (0x12 << 8) > +#define GEVNT_DEVT_ERRATICERR (0x11 << 8) > +#define GEVNT_DEVT_SOF (0x7 << 8) > +#define GEVNT_DEVT_EOPF (0x6 << 8) > +#define GEVNT_DEVT_HIBER_REQ (0x5 << 8) > +#define GEVNT_DEVT_WKUP (0x4 << 8) > +#define GEVNT_DEVT_ULST_CHNG (0x3 << 8) > +#define GEVNT_DEVT_CONNDONE (0x2 << 8) > +#define GEVNT_DEVT_USBRESET (0x1 << 8) > +#define GEVNT_DEVT_DISCONN (0x0 << 8) > + > +#define DCFG (DW_USB3_BASE + 0xC700) > + > +#define DCFG_NUMP_MASK (0x1F << 17) > +#define DCFG_NUMP(x) (((x) & 0x1F) << 17) > +#define DCFG_DEVADDR_MASK (0x7F << 3) > +#define DCFG_DEVADDR(x) (((x) & 0x7F) << 3) > +#define DCFG_DEVSPD_MASK (0x7) > +#define DCFG_DEVSPD(x) ((x) & 0x7) > +#define DEVSPD_HS_PHY_30MHZ_OR_60MHZ 0 > +#define DEVSPD_FS_PHY_30MHZ_OR_60MHZ 1 > +#define DEVSPD_LS_PHY_6MHZ 2 > +#define DEVSPD_FS_PHY_48MHZ 3 > +#define DEVSPD_SS_PHY_125MHZ_OR_250MHZ 4 > + > +#define DCTL (DW_USB3_BASE + 0xC704) > + > +#define DCTL_RUN_STOP BIT31 > +#define DCTL_CSFTRST BIT30 > +#define DCTL_INIT_U2_EN BIT12 > +#define DCTL_ACCEPT_U2_EN BIT11 > +#define DCTL_INIT_U1_EN BIT10 > +#define DCTL_ACCEPT_U1_EN BIT9 > + > +#define DEVTEN (DW_USB3_BASE + 0xC708) > +#define DEVTEN_CONNECTDONEEN BIT2 > +#define DEVTEN_USBRSTEN BIT1 > +#define DEVTEN_DISCONNEN BIT0 > + > +#define DSTS (DW_USB3_BASE + 0xC70C) > +#define DSTS_GET_DEVSPD(x) ((x) & 0x7) > + > +#define DALEPENA (DW_USB3_BASE + 0xC720) > + > +#define DEPCMDPAR2(x) (DW_USB3_BASE + 0xC800 + ((x) & 0x1F) * 0x10) > +#define DEPCMDPAR1(x) (DW_USB3_BASE + 0xC804 + ((x) & 0x1F) * 0x10) > +#define DEPCMDPAR0(x) (DW_USB3_BASE + 0xC808 + ((x) & 0x1F) * 0x10) > +#define DEPCMD(x) (DW_USB3_BASE + 0xc80C + ((x) & 0x1F) * 0x10) > + > +#define DEPCMD_COMMANDPARAM_MASK (0xFFFF << 16) > +#define DEPCMD_COMMANDPARAM(x) (((x) & 0xFFFF) << 16) > +/* Stream Number or uFrame (input) */ > +#define DEPCMD_STR_NUM_OR_UF_MASK (0xFFFF << 16) > +#define DEPCMD_STR_NUM_OR_UF(x) (((x) & 0xFFFF) << 16) > +/* Transfer Resource Index (output) */ > +#define DEPCMD_XFER_RSRC_IDX_SHIFT 16 > +#define DEPCMD_XFER_RSRC_IDX_MASK (0x7F << 16) > +#define DEPCMD_XFER_RSRC_IDX(x) (((x) & 0x7F) << 16) > +#define GET_DEPCMD_XFER_RSRC_IDX(x) (((x) >> 16) & 0x7F) > +#define DEPCMD_CMDACT BIT10 > +#define DEPCMD_CMDTYPE_MASK 0xFF > +#define DEPCMD_CMDTYPE(x) ((x) & 0xFF) > + > +/* EP registers range as: OUT0, IN0, OUT1, IN1, ... */ > +#define EP_OUT_IDX(x) ((x) * 2) > +#define EP_IN_IDX(x) (((x) * 2) + 1) > + > +#define CMDTYPE_SET_EP_CFG 1 > +#define CMDTYPE_SET_XFER_CFG 2 > +#define CMDTYPE_GET_EP_STATE 3 > +#define CMDTYPE_SET_STALL 4 > +#define CMDTYPE_CLR_STALL 5 > +#define CMDTYPE_START_XFER 6 > +#define CMDTYPE_UPDATE_XFER 7 > +#define CMDTYPE_END_XFER 8 > +#define CMDTYPE_START_NEW_CFG 9 > + > +#define EPTYPE_CONTROL 0 > +#define EPTYPE_ISOC 1 > +#define EPTYPE_BULK 2 > +#define EPTYPE_INTR 3 > + > +#define CFG_ACTION_INIT 0 > +#define CFG_ACTION_RESTORE 1 > +#define CFG_ACTION_MODIFY 2 > + > +#define EPCFG0_CFG_ACTION_MASK (0x3 << 30) > +#define EPCFG0_CFG_ACTION(x) (((x) & 0x3) << 30) > +#define EPCFG0_BRSTSIZ_MASK (0xF << 22) > +#define EPCFG0_BRSTSIZ(x) (((x) & 0xF) << 22) > +#define EPCFG0_TXFNUM_MASK (0x1F << 17) > +#define EPCFG0_TXFNUM(x) (((x) & 0x1F) << 17) > +#define EPCFG0_MPS_MASK (0x7FF << 3) > +#define EPCFG0_MPS(x) (((x) & 0x7FF) << 3) > +#define EPCFG0_EPTYPE_MASK (0x3 << 1) > +#define EPCFG0_EPTYPE_SHIFT 1 > +#define EPCFG0_EPTYPE(x) (((x) & 0x3) << 1) > + > +/* Endpoint Number */ > +#define EPCFG1_EP_NUM_MASK (0xF << 26) > +#define EPCFG1_EP_NUM(x) (((x) & 0xF) << 26) > +/* Endpoint Direction */ > +#define EPCFG1_EP_DIR_IN BIT25 > +/* Stream Not Ready */ > +#define EPCFG1_XFER_NRDY BIT10 > +/* XferInProgress Enable */ > +#define EPCFG1_XFER_IN_PROG BIT9 > +/* Stream Completed */ > +#define EPCFG1_XFER_CMPL BIT8 > + > +#define USB_SPEED_UNKNOWN 0 > +#define USB_SPEED_LOW 1 > +#define USB_SPEED_FULL 2 > +#define USB_SPEED_HIGH 3 > +#define USB_SPEED_VARIABLE 4 > +#define USB_SPEED_SUPER 5 > + > +// DMA registers > +#define DSCSTS_TRBRSP_MASK (0xF << 28) > +#define DSCSTS_TRBRSP(x) (((x) & 0xF) << 28) > +#define GET_DSCSTS_TRBRSP(x) (((x) >> 28) & 0xF) > +#define TRBRSP_MISSED_ISOC_IN 1 > +#define TRBRSP_SETUP_PEND 2 > +#define TRBRSP_XFER_IN_PROG 4 > +#define DSCSTS_PCM1_MASK (0x3 << 24) > +#define DSCSTS_PCM1(x) (((x) & 0x3) << 24) > +#define DSCSTS_XFERCNT_MASK 0xFFFFFF > +#define DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) > +#define GET_DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) > + > +#define DSCCTL_STRMID_SOFN(x) (((x) & 0xFFFF) << 14) > +#define DSCCTL_IOC BIT11 > +#define DSCCTL_ISP BIT10 > +#define DSCCTL_TRBCTL_MASK (0x3F << 4) > +#define DSCCTL_TRBCTL(x) (((x) & 0x3F) << 4) > +#define DSCCTL_LST BIT1 > +#define DSCCTL_HWO BIT0 > +#define TRBCTL_NORMAL 1 > +#define TRBCTL_SETUP 2 > +#define TRBCTL_STATUS_2 3 > +#define TRBCTL_STATUS_3 4 > +#define TRBCTL_CTLDATA_1ST 5 > +#define TRBCTL_ISOC_1ST 6 > +#define TRBCTL_ISOC 7 > +#define TRBCTL_LINK 8 > +#define TRBCTL_NORMAL_ZLP 9 > + > + > +#define UE_DIR_IN 0x80 > +#define UE_DIR_OUT 0 > +#define UE_SET_DIR(a, d) ((a) | (((d) & 1) << 7)) > +#define UE_GET_DIR(a) ((a) & 0x80) > +#define UE_ADDR 0x0F > +#define UE_GET_ADDR(a) ((a) & UE_ADDR) > + > +#define UT_GET_DIR(a) ((a) & 0x80) > +#define UT_WRITE 0x00 > +#define UT_READ 0x80 > + > +#define UT_GET_TYPE(a) ((a) & 0x60) > +#define UT_STANDARD 0x00 > +#define UT_CLASS 0x20 > +#define UT_VENDOR 0x40 > + > +#define UT_GET_RECIPIENT(a) ((a) & 0x1f) > +#define UT_DEVICE 0x00 > +#define UT_INTERFACE 0x01 > +#define UT_ENDPOINT 0x02 > +#define UT_OTHER 0x03 > + > +#define UR_GET_STATUS 0x00 > +#define UR_CLEAR_FEATURE 0x01 > +#define UR_SET_FEATURE 0x03 > +#define UR_SET_ADDRESS 0x05 > +#define UR_GET_DESCRIPTOR 0x06 > +#define UR_SET_DESCRIPTOR 0x07 > +#define UR_GET_CONFIG 0x08 > +#define UR_SET_CONFIG 0x09 > +#define UR_GET_INTERFACE 0x0A > +#define UR_SET_INTERFACE 0x0B > +#define UR_SYNCH_FRAME 0x0C > +#define UR_SET_SEL 0x30 > +#define UR_SET_ISOC_DELAY 0x31 > + Somewhere around here starts a bunch of redefinitions of things already described in MdePkg/Include/IndustryStandard/Usb.h. Can you please reuse that include file as much as possible. > +/* Feature numbers */ > +#define UF_ENDPOINT_HALT 0 > +#define UF_DEVICE_REMOTE_WAKEUP 1 > +#define UF_TEST_MODE 2 > +#define UF_DEVICE_B_HNP_ENABLE 3 > +#define UF_DEVICE_A_HNP_SUPPORT 4 > +#define UF_DEVICE_A_ALT_HNP_SUPPORT 5 > +#define UF_FUNCTION_SUSPEND 0 > +#define UF_U1_ENABLE 48 > +#define UF_U2_ENABLE 49 > +#define UF_LTM_ENABLE 50 > + > +#define UDESC_DEVICE 0x01 > +#define UDESC_CONFIG 0x02 > +#define UDESC_STRING 0x03 > +#define UDESC_INTERFACE 0x04 > +#define UDESC_ENDPOINT 0x05 > +#define UDESC_SS_USB_COMPANION 0x30 > +#define UDESC_DEVICE_QUALIFIER 0x06 > +#define UDESC_BOS 0x0f > +#define UDESC_DEVICE_CAPABILITY 0x10 > + > +#define STRING_LANGUAGE 0 > +#define STRING_MANUFACTURER 1 > +#define STRING_PRODUCT 2 > +#define STRING_SERIAL 3 > + > +#define CONFIG_VALUE 1 > + > +#define USB3_BULK_IN_EP 1 > +#define USB3_BULK_OUT_EP 1 > + > +#define USB_ENUM_ADB_PORT_VID 0x18D1 > +#define USB_ENUM_ADB_PORT_PID 0xD00D > +#define USB_ENUM_INTERFACE_ADB_SUBCLASS 0x42 > +#define USB_ENUM_INTERFACE_ADB_PROTOCOL 0x03 > + > +struct usb3_pcd; > + > +typedef enum pcd_state { > + USB3_STATE_UNCONNECTED, /* no host */ > + USB3_STATE_DEFAULT, > + USB3_STATE_ADDRESSED, > + USB3_STATE_CONFIGURED, > +} pcdstate_e; These structs also break the coding style completely. Will using IndustryStandard/Usb.h get rid of all of that? > + > +typedef enum ep0_state { > + EP0_IDLE, > + EP0_IN_DATA_PHASE, > + EP0_OUT_DATA_PHASE, > + EP0_IN_WAIT_NRDY, > + EP0_OUT_WAIT_NRDY, > + EP0_IN_STATUS_PHASE, > + EP0_OUT_STATUS_PHASE, > + EP0_STALL, > +} ep0state_e; > + > +typedef struct usb3_dma_desc { > + /** Buffer Pointer - Low address quadlet */ > + UINT32 bptl; > + > + /** Buffer Pointer - High address quadlet */ > + UINT32 bpth; > + > + /** Status quadlet. Fields defined in enum @ref desc_sts_data. */ > + UINT32 status; > + > + /** Control quadlet. Fields defined in enum @ref desc_ctl_data. */ > + UINT32 control; > +} usb3_dma_desc_t; > + > +typedef struct usb3_pcd_req { > + usb3_dma_desc_t *trb; > + UINT64 trbdma; > + > + UINT32 length; > + UINT32 actual; > + > + UINT64 *bufdma; > + int (*complete)(unsigned actual, int status); > +} usb3_pcd_req_t; > + > +typedef struct usb_device_request { > + UINT8 bmRequestType; > + UINT8 bRequest; > + UINT16 wValue; > + UINT16 wIndex; > + UINT16 wLength; > +} usb_device_request_t; > + > +#pragma pack(1) > +/** USB_DT_DEVICE: Device descriptor */ > +typedef struct usb_device_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT16 bcdUSB; > +#define USB_CLASS_COMM 0x02 > +#define USB_CLASS_VENDOR_SPEC 0xFF > +#define USB_SC_VENDOR_SPEC 0xFF > +#define USB_PR_VENDOR_SPEC 0xFF > + UINT8 bDeviceClass; > + UINT8 bDeviceSubClass; > + UINT8 bDeviceProtocol; > + UINT8 bMaxPacketSize0; > + UINT16 idVendor; > + UINT16 idProduct; > + UINT16 bcdDevice; > + UINT8 iManufacturer; > + UINT8 iProduct; > + UINT8 iSerialNumber; > + UINT8 bNumConfigurations; > +} usb_device_descriptor_t; > + > +/* USB_DT_CONFIG: Config descriptor */ > +typedef struct usb_config_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT16 wTotalLength; > + UINT8 bNumInterfaces; > +#define CONFIG_VALUE 1 > + UINT8 bConfigurationValue; > + UINT8 iConfiguration; > +#define USB_CONFIG_ATT_ONE (1 << 7) > + UINT8 bmAttributes; > +#define USB_CONFIG_VBUS_DRAW (0xFA) > + UINT8 bMaxPower; > +} usb_config_descriptor_t; > + > +/* USB_DT_DEVICE_QUALIFIER: Device Qualifier descriptor */ > +typedef struct usb_qualifier_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT16 bcdUSB; > + UINT8 bDeviceClass; > + UINT8 bDeviceSubClass; > + UINT8 bDeviceProtocol; > + UINT8 bMaxPacketSize0; > + UINT8 bNumConfigurations; > + UINT8 bRESERVED; > +} usb_qualifier_descriptor_t; > + > +/* USB_DT_INTERFACE: Interface descriptor */ > +typedef struct usb_interface_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT8 bInterfaceNumber; > + UINT8 bAlternateSetting; > + UINT8 bNumEndpoints; > + UINT8 bInterfaceClass; > + UINT8 bInterfaceSubClass; > + UINT8 bInterfaceProtocol; > + UINT8 iInterface; > +} usb_interface_descriptor_t; > + > +/* USB_DT_ENDPOINT: Endpoint descriptor */ > +typedef struct usb_endpoint_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT8 bEndpointAddress; > + UINT8 bmAttributes; > +#define USB_ENDPOINT_XFER_CONTROL 0x00 > +#define USB_ENDPOINT_XFER_ISOC 0x01 > +#define USB_ENDPOINT_XFER_BULK 0x02 > +#define USB_ENDPOINT_XFER_INT 0x03 > + UINT16 wMaxPacketSize; > + UINT8 bInterval; > +} usb_endpoint_descriptor_t; > + > +/* USB_DT_SS_ENDPOINT_COMP: SuperSpeed Endpoint Companion descriptor */ > +typedef struct usb_ss_ep_comp_descriptor { > + UINT8 bLength; > + UINT8 bDescriptorType; > + > + UINT8 bMaxBurst; > + UINT8 bmAttributes; > + UINT16 wBytesPerInterval; > +} usb_ss_ep_comp_descriptor_t; > + > +/* WUSB BOS Descriptor (Binary device Object Store) */ > +typedef struct wusb_bos_desc { > + UINT8 bLength; > + UINT8 bDescriptorType; > + UINT16 wTotalLength; > + UINT8 bNumDeviceCaps; > +} wusb_bos_desc_t; > + > +#define USB_DEVICE_CAPABILITY_20_EXTENSION 0x02 > +typedef struct usb_dev_cap_20_ext_desc { > + UINT8 bLength; > + UINT8 bDescriptorType; > + UINT8 bDevCapabilityType; > +#define USB_20_EXT_LPM 0x02 > + UINT32 bmAttributes; > +} usb_dev_cap_20_ext_desc_t; > + > +#define USB_DEVICE_CAPABILITY_SS_USB 0x03 > +typedef struct usb_dev_cap_ss_usb { > + UINT8 bLength; > + UINT8 bDescriptorType; > + UINT8 bDevCapabilityType; > +#define USB_DC_SS_USB_LTM_CAPABLE 0x02 > + UINT8 bmAttributes; > +#define USB_DC_SS_USB_SPEED_SUPPORT_LOW 0x01 > +#define USB_DC_SS_USB_SPEED_SUPPORT_FULL 0x02 > +#define USB_DC_SS_USB_SPEED_SUPPORT_HIGH 0x04 > +#define USB_DC_SS_USB_SPEED_SUPPORT_SS 0x08 > + UINT32 wSpeedsSupported; > + UINT8 bFunctionalitySupport; > + UINT8 bU1DevExitLat; > + UINT32 wU2DevExitLat; > +} usb_dev_cap_ss_usb_t; > + > +#define USB_DEVICE_CAPABILITY_CONTAINER_ID 0x04 > +typedef struct usb_dev_cap_container_id { > + UINT8 bLength; > + UINT8 bDescriptorType; > + UINT8 bDevCapabilityType; > + UINT8 bReserved; > + UINT8 containerID[16]; > +} usb_dev_cap_container_id_t; > +#pragma pack() > + > +typedef union usb_setup_pkt { > + usb_device_request_t req; > + UINT32 d32[2]; > + UINT8 d8[8]; > +} usb_setup_pkt_t; > + > +typedef struct usb3_pcd_ep { > + struct usb3_pcd *pcd; > + > + UINT8 EpInIdx; > + UINT8 EpOutIdx; > + UINT8 phys; > + > + //UINT8 phys; > + UINT8 num; > + UINT8 type; > + UINT8 maxburst; > + UINT16 maxpacket; > + /* Tx FIFO # for IN EPs */ > + UINT8 tx_fifo_num; > + > + /* The Transfer Resource Index from the Start Transfer command */ > + UINT8 tri_out; > + UINT8 tri_in; > + > + UINT8 stopped; > + /* Send ZLP */ > + UINT8 send_zlp; > + /* True if 3-stage control transfer */ > + UINT8 three_stage; > + /* True if transfer has been started on EP */ > + UINT8 xfer_started; > + /* EP direction 0 = OUT */ > + UINT8 is_in; > + /* True if endpoint is active */ > + UINT8 active; > + /* Initial data pid of bulk endpoint */ > + UINT8 data_pid_start; > + > + /* ep_desc (excluding ep0) */ > + usb3_dma_desc_t *ep_desc; > + > +#if 0 > + /* TRB descriptor must be aligned to 16 bytes */ > + UINT8 epx_desc[32]; > +#endif > + > + /* request (excluding ep0) */ > + usb3_pcd_req_t req; > +} usb3_pcd_ep_t; > + > +typedef struct usb3_pcd { > + //struct usb3_device *usb3_dev; > + > + INT32 link_state; > + pcdstate_e state; > + UINT8 new_config; > + ep0state_e ep0state; > + > + UINT32 eps_enabled; > + UINT32 ltm_enable; > + > + usb3_pcd_ep_t ep0; > + usb3_pcd_ep_t out_ep; > + usb3_pcd_ep_t in_ep; > + > + /* > + usb3_dev_global_regs_t *dev_global_regs; > + usb3_dev_ep_regs_t *out_ep_regs; > + usb3_dev_ep_regs_t *in_ep_regs; > + */ > + > + usb3_pcd_req_t ep0_req; > + > + UINT8 speed; > + > + usb3_dma_desc_t *ep0_setup_desc; > + usb3_dma_desc_t *ep0_in_desc; > + usb3_dma_desc_t *ep0_out_desc; > + > + /* TRB descriptor must be aligned to 16 bytes */ > +#if 0 > + UINT8 ep0_setup[32]; > + UINT8 ep0_in[32]; > + UINT8 ep0_out[32]; > + > + usb_setup_pkt_t ep0_setup_pkt[5]; > + > +#define USB3_STATUS_BUF_SIZE 512 > + UINT8 ep0_status_buf[USB3_STATUS_BUF_SIZE]; > + > +#define USB3_BULK_BUF_SIZE 2048 > + UINT8 ss_bulk_buf[USB3_BULK_BUF_SIZE]; > +#endif > + > + UINT32 file_type; > + UINT32 file_address; > + UINT32 file_capacity; > + UINT32 file_total_frame; > + UINT32 file_curr_frame; > + UINT32 file_next_frame; > + UINT32 file_received; > + UINT32 file_complete; > + > + UINT16 test_mode_nr; > + UINT16 test_mode; > +} usb3_pcd_t; > + > +struct usb_enum_port_param { > + UINT16 idVendor; > + UINT16 idProduct; > + UINT8 bInterfaceSubClass; > + UINT8 bInterfaceProtocol; > +}; > + > +#if 0 If not used, leave it out. > +typedef struct usb3_pcd_req { > + usb3_dma_desc_t *trb; > + UINT64 trbdma; > + > + UINT32 length; > + UINT32 actual; > + > + UINT64 *bufdma; > + int (*complete)(unsigned actual, int status); > +} usb3_pcd_req_t; > + > +#endif > + > +#endif /* __DW_USB3_DXE_H__ */ > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c > new file mode 100644 > index 000000000000..83d5e4736de0 > --- /dev/null > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c > @@ -0,0 +1,2434 @@ > +/** @file > + > + Copyright (c) 2018, Linaro Limited. All rights reserved. > + > + This program and the accompanying materials are licensed and made available > + under the terms and conditions of the BSD License which accompanies this > + distribution. The full text of the license may be found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > + > +**/ > + > +#include <IndustryStandard/Usb.h> > +#include <Library/ArmLib.h> > +#include <Library/BaseLib.h> > +#include <Library/BaseMemoryLib.h> > +#include <Library/CacheMaintenanceLib.h> > +#include <Library/DebugLib.h> > +#include <Library/DmaLib.h> > +#include <Library/IoLib.h> > +#include <Library/MemoryAllocationLib.h> > +#include <Library/TimerLib.h> > +#include <Library/UefiBootServicesTableLib.h> > +#include <Library/UefiDriverEntryPoint.h> > +#include <Library/UefiRuntimeServicesTableLib.h> > +#include <Protocol/DwUsb.h> > +#include <Protocol/DriverBinding.h> > +#include <Protocol/UsbDevice.h> > + > +#include "DwUsb3Dxe.h" > + > +#define FIFO_DIR_TX 0 > +#define FIFO_DIR_RX 1 > + > +#define TX_FIFO_ADDR 0 > +#define RX_FIFO_ADDR 0 > + > +#define RAM_WIDTH 8 > +#define RAM_TX0_DEPTH 2048 > +#define RAM_TX1_DEPTH 4096 > +#define RAM_RX_DEPTH 8192 Should these not also go into DwUsb3Dxe.h? > + > +#define USB_TYPE_LENGTH 16 > +#define USB_BLOCK_HIGH_SPEED_SIZE 512 > +#define DATA_SIZE 131072 > +#define CMD_SIZE 512 What do the above definitions mean? > +#define MATCH_CMD_LITERAL(Cmd, Buf) !AsciiStrnCmp (Cmd, Buf, sizeof (Cmd) - 1) This one does not appear to be used? > + > +#define ALIGN(x, a) (((x) + ((a) - 1)) & ~((a) - 1)) Use available macros in Base.h. > + > +// > +// The time between interrupt polls, in units of 100 nanoseconds > +// 10 Microseconds > +// > +#define DW_INTERRUPT_POLL_PERIOD 100 > + > +#define DWUSB3_EVENT_BUF_SIZE 256 > + > +// > +// Maxpacket size for EP0, defined by USB3 spec > +// > +#define USB3_MAX_EP0_SIZE 512 > + > +// > +// Maxpacket size for any EP, defined by USB3 spec > +// > +#define USB3_MAX_PACKET_SIZE 1024 > +#define USB2_HS_MAX_PACKET_SIZE 512 > +#define USB2_FS_MAX_PACKET_SIZE 64 > + > +#define USB3_STATE_UNCONNECTED 0 > +#define USB3_STATE_DEFAULT 1 > +#define USB3_STATE_ADDRESSED 2 > +#define USB3_STATE_CONFIGURED 3 > + > +#define USB3_STATUS_BUF_SIZE 512 If these are defined by USB3 spec, please add them to MdePkg/Include/IndustryStandard/Usb.h instead of defining them locally. > + > +#define GET_EVENTBUF_COUNT() \ > + (GEVNTCOUNT_EVNTCOUNT (MmioRead32 (GEVNTCOUNT (0)))) > +#define UPDATE_EVENTBUF_COUNT(x) \ > + (MmioWrite32 (GEVNTCOUNT (0), GEVNTCOUNT_EVNTCOUNT (x))) > + > +#define SET_DEVADDR(x) \ > + (MmioAndThenOr32 (DCFG, ~DCFG_DEVADDR_MASK, DCFG_DEVADDR (x))) Should these not also move to DwUsb3Dxe.h? > + > +EFI_GUID gDwUsbProtocolGuid = DW_USB_PROTOCOL_GUID; This looks less than ideal - this makes it impossible to include DwUsb2 and DwUsb3 drivers in the same image. > + > +STATIC DW_USB_PROTOCOL *DwUsb; > + > +STATIC usb3_pcd_t gPcd; This seems a very unfortunate choice for variable name. > +STATIC UINT32 *gEventBuf, *gEventPtr; > +STATIC struct usb_device_descriptor gDwUsb3DevDesc; > +STATIC VOID *gRxBuf; > + > +STATIC usb_setup_pkt_t *gEndPoint0SetupPacket; > +STATIC UINT8 *gEndPoint0StatusBuf; Although most if not all of these will disappear when converting to UEFI driver model, why do they have 'g' prefix in the first place? 'g' is for globally visible symbols. Variables that are "global" in this file only are better off using 'm'. > +STATIC USB_DEVICE_RX_CALLBACK mDataReceivedCallback; > +STATIC UINTN mDataBufferSize; > + > +struct usb_interface_descriptor intf = { > + sizeof (struct usb_interface_descriptor), > + UDESC_INTERFACE, > + 0, > + 0, > + 2, > + USB_CLASS_VENDOR_SPEC, > + 0x42, > + 0x03, > + 0 > +}; > + > +const struct usb_ss_ep_comp_descriptor ep_comp = { > + sizeof (struct usb_ss_ep_comp_descriptor), > + UDESC_SS_USB_COMPANION, > + 0, > + 0, > + 0 > +}; > + > +const struct usb_endpoint_descriptor hs_bulk_in = { > + sizeof (struct usb_endpoint_descriptor), > + UDESC_ENDPOINT, > + UE_DIR_IN | USB3_BULK_IN_EP, > + USB_ENDPOINT_XFER_BULK, > + 0x200, > + 0 > +}; > + > +const struct usb_endpoint_descriptor > +hs_bulk_out = { > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > + UDESC_ENDPOINT, /* bDescriptorType */ > + > + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > + 0x200, /* wMaxPacketSize: 512 of high-speed */ > + 1, /* bInterval */ > +}; > + > +const struct usb_endpoint_descriptor ss_bulk_in = { > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > + UDESC_ENDPOINT, /* bDescriptorType */ > + > + UE_DIR_IN | USB3_BULK_IN_EP, /* bEndpointAddress */ > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > + 0x400, /* wMaxPacketSize: 1024 of super-speed */ > + 0, /* bInterval */ > +}; > + > +const struct usb_endpoint_descriptor ss_bulk_out = { > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > + UDESC_ENDPOINT, /* bDescriptorType */ > + > + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > + 0x400, /* wMaxPacketSize: 1024 of super-speed */ > + 0, /* bInterval */ > +}; > + > +/** The BOS Descriptor */ What's a BOS? > + > +const struct usb_dev_cap_20_ext_desc cap1 = { > + sizeof(struct usb_dev_cap_20_ext_desc), /* bLength */ > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > + USB_DEVICE_CAPABILITY_20_EXTENSION, /* bDevCapabilityType */ > + 0x2, /* bmAttributes */ > +}; > + > +const struct usb_dev_cap_ss_usb > +cap2 = { > + sizeof(struct usb_dev_cap_ss_usb), /* bLength */ > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > + USB_DEVICE_CAPABILITY_SS_USB, /* bDevCapabilityType */ > + 0x0, /* bmAttributes */ > + (USB_DC_SS_USB_SPEED_SUPPORT_SS | > + USB_DC_SS_USB_SPEED_SUPPORT_HIGH), /* wSpeedsSupported */ > + 0x2, /* bFunctionalitySupport */ > + 0xa, /* bU1DevExitLat */ > + 0x100, /* wU2DevExitLat */ > +}; > + > +const struct usb_dev_cap_container_id > +cap3 = { > + sizeof(struct usb_dev_cap_container_id),/* bLength */ > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > + USB_DEVICE_CAPABILITY_CONTAINER_ID, /* bDevCapabilityType */ > + 0, /* bReserved */ > + {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, /* containerID */ > +}; > + > +const struct wusb_bos_desc > +bos = { > + sizeof(struct wusb_bos_desc), /* bLength */ > + UDESC_BOS, /* bDescriptorType */ > + (sizeof(struct wusb_bos_desc) + \ > + sizeof(cap1) + sizeof(cap2) + \ > + sizeof(cap3)), /* wTotalLength */ > + 3, /* bNumDeviceCaps */ > +}; > + > +STATIC struct usb_enum_port_param usb_port_activity_config = { > + .idVendor = USB_ENUM_ADB_PORT_VID, > + .idProduct = USB_ENUM_ADB_PORT_PID, > + .bInterfaceSubClass = USB_ENUM_INTERFACE_ADB_SUBCLASS, > + .bInterfaceProtocol = USB_ENUM_INTERFACE_ADB_PROTOCOL > +}; > + > +STATIC > +UINT32 > +DwUsb3GetEventBufEvent ( > + IN UINTN Size > + ) > +{ > + UINT32 Event; > + > + Event = *gEventPtr++; > + if ((UINT32)(UINTN)gEventPtr >= (UINT32)(UINTN)gEventBuf + Size) { > + gEventPtr = gEventBuf; > + } > + return Event; > +} > + > +STATIC > +VOID > +DwUsb3SetFifoSize ( > + IN UINT32 Addr, > + IN UINT32 Depth, > + IN UINT32 Dir, > + IN UINT32 FifoNum > + ) > +{ > + UINT32 Reg = 0; > + > + if (Dir == FIFO_DIR_TX) { > + Reg = GTXFIFOSIZ (FifoNum); > + } else if (Dir == FIFO_DIR_RX) { > + Reg = GRXFIFOSIZ (FifoNum); > + } else { > + ASSERT (0); > + } > + MmioWrite32 (Reg, FIFOSIZ_DEP (Depth) | FIFOSIZ_ADDR (Addr)); > +} > + > +STATIC > +UINT32 > +Handshake ( What does this function do? "Handshake" does not tell me enough to figure out. > + IN UINT32 Reg, > + IN UINT32 Mask, > + IN UINT32 Done > + ) > +{ > + UINT32 Timeout = 100000; > + > + do { > + if ((MmioRead32 (Reg) & Mask) == Done) { > + return 1; > + } > + MicroSecondDelay (1); > + } while (Timeout-- > 0); > + return 0; > +} > + > +STATIC > +VOID > +DwUsb3FillDesc ( > + IN usb3_dma_desc_t *desc, > + IN UINT64 dma_addr, > + IN UINT32 dma_len, > + IN UINT32 stream, > + IN UINT32 type, > + IN UINT32 ctrlbits, > + IN UINT32 own > + ) > +{ > + desc->bptl = (UINT32)(dma_addr & 0xFFFFFFFF); > + desc->bpth = (UINT32)(dma_addr >> 32); Could you possibly create LOW_32BIT and HIGH_32BIT macros for MdePkg/Base.h instead? We are already duplicating these operations all over the place. Including all over MdeModulePkg. > + desc->status = DSCSTS_XFERCNT (dma_len); > + if (type) { > + desc->control = DSCCTL_TRBCTL (type); > + } > + desc->control |= DSCCTL_STRMID_SOFN (stream) | ctrlbits; > + ArmDataSynchronizationBarrier (); Please use MemoryFence() instead. This controller is not ARM-specific. (DSB? That sounds excessive. DMB should suffice?) > + /* must execute this operation at last */ > + if (own) { > + desc->control |= DSCCTL_HWO; > + } > + ArmDataSynchronizationBarrier (); As above. > +} > + > +STATIC > +VOID > +DwUsb3DepStartNewCfg ( > + IN UINT32 EpIdx, > + IN UINT32 RsrcIdx > + ) > +{ > + /* start the command */ > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_XFER_RSRC_IDX (RsrcIdx) | DEPCMD_CMDTYPE (CMDTYPE_START_NEW_CFG) | \ > + DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +STATIC > +VOID > +DwUsb3DepCfg ( Please no abbreviations in function names. This name tells me nothing about what the function does (and there is no comment describing its function). > + IN UINT32 EpIdx, > + IN UINT32 DepCfg0, > + IN UINT32 DepCfg1, > + IN UINT32 DepCfg2 > + ) > +{ > + MmioWrite32 (DEPCMDPAR2 (EpIdx), DepCfg2); > + MmioWrite32 (DEPCMDPAR1 (EpIdx), DepCfg1); > + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepCfg0); > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_CMDTYPE (CMDTYPE_SET_EP_CFG) | DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +STATIC > +VOID > +DwUsb3DepXferCfg ( Dep<something>TransferConfig. Please address throughout. > + IN UINT32 EpIdx, > + IN UINT32 DepStrmCfg > + ) > +{ > + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepStrmCfg); > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_CMDTYPE (CMDTYPE_SET_XFER_CFG) | DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +STATIC > +UINT8 > +DwUsb3DepStartXfer ( > + IN UINT32 EpIdx, > + IN UINT64 DmaAddr, > + IN UINT32 StreamOrUf > + ) > +{ > + UINT32 Data; > + > + MmioWrite32 (DEPCMDPAR1 (EpIdx), (UINT32)DmaAddr); > + MmioWrite32 (DEPCMDPAR0 (EpIdx), (UINT32)(DmaAddr >> 32)); > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_STR_NUM_OR_UF (StreamOrUf) | DEPCMD_CMDTYPE (CMDTYPE_START_XFER) | \ > + DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > + Data = MmioRead32 (DEPCMD (EpIdx)); > + return GET_DEPCMD_XFER_RSRC_IDX(Data); > +} > + > +STATIC > +VOID > +DwUsb3DepStopXfer ( > + IN UINT32 EpIdx, > + IN UINT32 Tri > + ) > +{ > + MmioWrite32 (DEPCMDPAR2 (EpIdx), 0); > + MmioWrite32 (DEPCMDPAR1 (EpIdx), 0); > + MmioWrite32 (DEPCMDPAR0 (EpIdx), 0); > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_END_XFER) | \ > + DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +VOID > +DwUsb3DepUpdateXfer ( > + IN UINT32 EpIdx, > + IN UINT32 Tri > + ) > +{ > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_UPDATE_XFER) | \ > + DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +STATIC > +VOID > +DwUsb3DepClearStall ( > + IN UINTN EpIdx > + ) > +{ > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_CMDTYPE (CMDTYPE_CLR_STALL) | DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > + > +STATIC > +VOID > +DwUsb3DepSetStall ( > + IN UINTN EpIdx > + ) > +{ > + MmioWrite32 ( > + DEPCMD (EpIdx), > + DEPCMD_CMDTYPE (CMDTYPE_SET_STALL) | DEPCMD_CMDACT > + ); > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > +} > + > +STATIC > +VOID > +DwUsb3EnableEp ( EnableEndpoint > + IN UINT32 EpIdx, EndpointIndex. > + IN usb3_pcd_ep_t *ep Endpoint. > + ) > +{ > + UINT32 Dalepena; What? > + > + Dalepena = MmioRead32 (DALEPENA); > + /* If the EP is already enabled, skip to set it again. */ endpoint > + if (Dalepena & (1 << EpIdx)) { > + return; > + } > + Dalepena |= 1 << EpIdx; > + MmioWrite32 (DALEPENA, Dalepena); > +} > + > +STATIC > +VOID > +DwUsb3Ep0Activate ( (EP0 is special, so may not need expanding.) > + IN OUT usb3_pcd_t *pcd > + ) > +{ > + /* issue DEPCFG command to EP0 OUT */ ... except for in comments. > + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 0); > + DwUsb3DepCfg ( > + EP_OUT_IDX (0), > + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512), > + EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY, > + 0 > + ); > + /* issue DEPSTRMCFG command to EP0 OUT */ > + DwUsb3DepXferCfg (EP_OUT_IDX (0), 1); // one stream > + /* issue DEPCFG command to EP0 IN */ > + DwUsb3DepCfg ( > + EP_IN_IDX (0), > + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512) | \ > + EPCFG0_TXFNUM (pcd->ep0.tx_fifo_num), > + EPCFG1_XFER_NRDY | EPCFG1_XFER_CMPL | EPCFG1_EP_DIR_IN, > + 0 > + ); > + /* issue DEPSTRMCFG command to EP0 IN */ > + DwUsb3DepXferCfg (EP_IN_IDX (0), 1); // one stream > + pcd->ep0.active = 1; > +} > + > +STATIC > +VOID > +DwUsb3EpActivate ( > + IN OUT usb3_pcd_t *pcd, > + IN OUT usb3_pcd_ep_t *ep > + ) > +{ > + UINT32 EpIdx, DepCfg0, DepCfg1; > + if (ep->is_in) { > + EpIdx = EP_IN_IDX (ep->num); > + } else { > + EpIdx = EP_OUT_IDX (ep->num); > + } > + > + /* Start a new configurate when enable the first EP. */ > + if (!pcd->eps_enabled) { > + pcd->eps_enabled = 1; > + /* Issue DEPCFG command to physical EP1 (logical EP0 IN) first. > + * It resets the core's Tx FIFO mapping table. > + */ > + DepCfg0 = EPCFG0_EPTYPE (EPTYPE_CONTROL); > + DepCfg0 |= EPCFG0_CFG_ACTION (CFG_ACTION_MODIFY); > + DepCfg1 = EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY | EPCFG1_EP_DIR_IN; > + > + switch (pcd->speed) { > + case USB_SPEED_SUPER: > + DepCfg0 |= EPCFG0_MPS (512); > + break; > + case USB_SPEED_HIGH: > + case USB_SPEED_FULL: > + DepCfg0 |= EPCFG0_MPS (64); > + break; > + case USB_SPEED_LOW: > + DepCfg0 |= EPCFG0_MPS (8); No live coding of values. Add #defines for those speed setting values. > + break; > + default: > + ASSERT (0); > + break; > + } > + DwUsb3DepCfg (EP_IN_IDX (0), DepCfg0, DepCfg1, 0); What does that final 0 do? > + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 2); And that 2? Please address throughout. > + } > + /* issue DEPCFG command to EP */ > + DepCfg0 = EPCFG0_EPTYPE (ep->type); > + DepCfg0 |= EPCFG0_MPS (ep->maxpacket); > + if (ep->is_in) { > + DepCfg0 |= EPCFG0_TXFNUM (ep->tx_fifo_num); > + } > + DepCfg0 |= EPCFG0_BRSTSIZ (ep->maxburst); > + DepCfg1 = EPCFG1_EP_NUM (ep->num); > + if (ep->is_in) { > + DepCfg1 |= EPCFG1_EP_DIR_IN; > + } else { > + DepCfg1 |= EPCFG1_XFER_CMPL; > + } > + DwUsb3DepCfg (EpIdx, DepCfg0, DepCfg1, 0); > + /* issue DEPSTRMCFG command to EP */ > + DwUsb3DepXferCfg (EpIdx, 1); > + DwUsb3EnableEp (EpIdx, ep); > + ep->active = 1; > +} > + > +STATIC > +VOID > +DwUsb3Ep0OutStart ( > + IN usb3_pcd_t *pcd > + ) > +{ > + usb3_dma_desc_t *desc; > + > + /* Get the SETUP packet DMA Descriptor (TRB) */ > + desc = pcd->ep0_setup_desc; > + > + /* DMA Descriptor setup */ > + DwUsb3FillDesc ( > + desc, > + (UINT64)gEndPoint0SetupPacket, > + pcd->ep0.maxpacket, > + 0, > + TRBCTL_SETUP, > + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, > + 1 > + ); > + > + /* issue DEPSTRTXFER command to EP0 OUT */ > + pcd->ep0.tri_out = DwUsb3DepStartXfer (EP_OUT_IDX (0), (UINT64)desc, 0); > +} > + > +STATIC > +VOID > +DwUsb3Init ( > + VOID > + ) > +{ > + UINT32 Data, Addr; Address. > + usb3_pcd_t *pcd = &gPcd; > + > + /* soft reset the usb core */ > + do { > + MmioAndThenOr32 (DCTL, ~DCTL_RUN_STOP, DCTL_CSFTRST); > + > + do { > + MicroSecondDelay (1000); > + Data = MmioRead32 (DCTL); > + } while (Data & DCTL_CSFTRST); > + // > + // wait for at least 3 PHY clocks > + // > + MicroSecondDelay (1000); > + } while (0); > + > + pcd->link_state = 0; > + > + /* TI PHY: Set Turnaround Time = 9 (8-bit UTMI+ / ULPI) */ > + MmioAndThenOr32 ( > + GUSB2PHYCFG (0), > + ~GUSB2PHYCFG_USBTRDTIM_MASK, > + GUSB2PHYCFG_USBTRDTIM (9) How long is 9? I will stop my review here and give a full review on v2 when the overarching comments have been addressed. / Leif ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver 2018-10-04 16:32 ` Leif Lindholm @ 2018-10-22 2:51 ` Haojian Zhuang 0 siblings, 0 replies; 9+ messages in thread From: Haojian Zhuang @ 2018-10-22 2:51 UTC (permalink / raw) To: Leif Lindholm; +Cc: edk2-devel@lists.01.org, Ard Biesheuvel On Fri, 5 Oct 2018 at 00:32, Leif Lindholm <leif.lindholm@linaro.org> wrote: > > On Mon, Aug 20, 2018 at 06:31:25PM +0800, Haojian Zhuang wrote: > > Add Designware USB 3.0 device driver. It's focus on USB device > > functionality, not USB Host functionality. The USB driver is > > mainly used for Android Fastboot application. > > > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org> > > --- > > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec | 44 + > > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf | 52 + > > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h | 632 +++++ > > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c | 2434 ++++++++++++++++++++ > > 4 files changed, 3162 insertions(+) > > > > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > > new file mode 100644 > > index 000000000000..038e4881a948 > > --- /dev/null > > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > > @@ -0,0 +1,44 @@ > > +#/** @file > > +# Framework Module Development Environment Industry Standards > > Please change this to reflect this package rather than EmbeddedPkg one > (which appears to have stolen it from an old version of MdePkg.dec :) > OK > > +# > > +# This Package provides headers and libraries that conform to EFI/PI Industry standards. > > +# Copyright (c) 2007, Intel Corporation. All rights reserved.<BR> > > +# Copyright (c) 2012-2014, ARM Ltd. All rights reserved.<BR> > > I see no need to keep these copyright statements. > OK > > +# Copyright (c) 2018, Linaro. All rights reserved.<BR> > > +# > > +# This program and the accompanying materials are licensed and made available under > > +# the terms and conditions of the BSD License which accompanies this distribution. > > +# The full text of the license may be found at > > +# http://opensource.org/licenses/bsd-license.php > > +# > > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > > +# > > +#**/ > > + > > +[Defines] > > + DEC_SPECIFICATION = 0x00010019 > > + PACKAGE_NAME = DwUsb3DxePkg > > + PACKAGE_GUID = 9b7aa6fe-405b-4955-af1f-5faf183aedec > > + PACKAGE_VERSION = 0.1 > > + > > + > > +################################################################################ > > +# > > +# Include Section - list of Include Paths that are provided by this package. > > +# Comments are used for Keywords and Module Types. > > +# > > +# Supported Module Types: > > +# BASE SEC PEI_CORE PEIM DXE_CORE DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER DXE_SAL_DRIVER UEFI_DRIVER UEFI_APPLICATION > > +# > > +################################################################################ > > + > > +[Guids.common] > > + gDwUsb3DxeTokenSpaceGuid = { 0x098a50d3, 0x92e2, 0x461a, { 0xb6, 0xba, 0xf8, 0x5d, 0x9d, 0x89, 0x49, 0xf5 }} > > + > > +[Protocols.common] > > + gDwUsbProtocolGuid = { 0x109fa264, 0x7811, 0x4862, { 0xa9, 0x73, 0x4a, 0xb2, 0xef, 0x2e, 0xe2, 0xff }} > > Please make this file a common .dec for DwUsb and DwUsb3 instead of > duplicating guid information. > (If we weren't moving it to edk2-platforms, it would still have been > better to add it to EmbeddedPkg.dec.) > OK > > + > > +[PcdsFixedAtBuild.common] > > + # DwUsb Driver PCDs > > + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress|0x0|UINT32|0x00000001 > > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf > > new file mode 100644 > > index 000000000000..510b51a34de7 > > --- /dev/null > > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf > > @@ -0,0 +1,52 @@ > > +#/** @file > > +# > > +# Copyright (c) 2018, Linaro Limited. All rights reserved. > > +# > > +# This program and the accompanying materials > > +# are licensed and made available under the terms and conditions of the BSD License > > +# which accompanies this distribution. The full text of the license may be found at > > +# http://opensource.org/licenses/bsd-license.php > > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > > +# > > +# > > +#**/ > > + > > +[Defines] > > + INF_VERSION = 0x00010019 > > + BASE_NAME = DwUsb3Dxe > > + FILE_GUID = 0879cd34-c399-4315-9891-56024072e711 > > + MODULE_TYPE = UEFI_DRIVER > > + VERSION_STRING = 1.0 > > + ENTRY_POINT = DwUsb3EntryPoint > > + > > +[Sources.common] > > + DwUsb3Dxe.c > > + > > +[LibraryClasses] > > + CacheMaintenanceLib > > + DebugLib > > + DmaLib > > + IoLib > > + MemoryAllocationLib > > + TimerLib > > + UefiBootServicesTableLib > > + UefiDriverEntryPoint > > + UsbSerialNumberLib > > I don't see this library anywhere in the tree? > I'll add it into the patch series for the next revision. > > + > > +[Protocols] > > + gEfiDriverBindingProtocolGuid > > + gUsbDeviceProtocolGuid > > + > > +[Packages] > > + ArmPkg/ArmPkg.dec > > + EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > > + EmbeddedPkg/EmbeddedPkg.dec > > + MdePkg/MdePkg.dec > > + MdeModulePkg/MdeModulePkg.dec > > + > > +[Pcd] > > + gDwUsb3DxeTokenSpaceGuid.PcdDwUsb3DxeBaseAddress > > + > > +[Depex] > > + TRUE > > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h > > new file mode 100644 > > index 000000000000..92b610553169 > > --- /dev/null > > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h > > @@ -0,0 +1,632 @@ > > +/** @file > > + > > + Copyright (c) 2017, Linaro Limited. All rights reserved. > > + > > + This program and the accompanying materials > > + are licensed and made available under the terms and conditions of the BSD License > > + which accompanies this distribution. The full text of the license may be found at > > + http://opensource.org/licenses/bsd-license.php > > + > > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > > + > > +**/ > > + > > +#ifndef __DW_USB3_DXE_H__ > > +#define __DW_USB3_DXE_H__ > > DW -> DESIGNWARE (throughout). > OK > > + > > +#define DW_USB3_BASE FixedPcdGet32 (PcdDwUsb3DxeBaseAddress) > > + > > +#define GSBUCFG0 (DW_USB3_BASE + 0xC100) > > +#define GCTL (DW_USB3_BASE + 0xC110) > > + > > +#define GCTL_PWRDNSCALE_MASK (0x1FFF << 19) > > +#define GCTL_PWRDNSCALE(x) (((x) & 0x1FFF) << 19) > > +#define GCTL_U2RSTECN BIT16 > > +#define GCTL_PRTCAPDIR_MASK (BIT13 | BIT12) > > +#define GCTL_PRTCAPDIR_HOST BIT12 > > +#define GCTL_PRTCAPDIR_DEVICE BIT13 > > +#define GCTL_PRTCAPDIR_OTG (BIT13 | BIT12) > > +#define GCTL_U2EXIT_LFPS BIT2 > > + > > +#define GUSB2PHYCFG(x) (DW_USB3_BASE + 0xC200 + (((x) & 0xF) << 2)) > > + > > +#define GUSB2PHYCFG_USBTRDTIM_MASK (0xF << 10) > > +#define GUSB2PHYCFG_USBTRDTIM(x) (((x) & 0xF) << 10) > > +#define GUSB2PHYCFG_SUSPHY BIT6 > > + > > +#define GUSB3PIPECTL(x) (DW_USB3_BASE + 0xC2C0 + (((x) & 0x3) << 2)) > > + > > +#define PIPECTL_DELAYP1TRANS BIT18 > > +#define PIPECTL_SUSPEND_EN BIT17 > > +#define PIPECTL_LFPS_FILTER BIT9 > > +#define PIPECTL_TX_DEMPH_MASK (0x3 << 1) > > +#define PIPECTL_TX_DEMPH(x) (((x) & 0x3) << 1) > > + > > +#define GTXFIFOSIZ(x) (DW_USB3_BASE + 0xC300 + (((x) & 0x1F) << 2)) > > +#define GRXFIFOSIZ(x) (DW_USB3_BASE + 0xC380 + (((x) & 0x1F) << 2)) > > + > > +#define FIFOSIZ_ADDR(x) (((x) & 0xFFFF) << 16) > > +#define FIFOSIZ_DEP(x) ((x) & 0xFFFF) > > + > > +#define GEVNTADRL(x) (DW_USB3_BASE + 0xC400 + (((x) & 0x1F) << 2)) > > +#define GEVNTADRH(x) (DW_USB3_BASE + 0xC404 + (((x) & 0x1F) << 2)) > > +#define GEVNTSIZ(x) (DW_USB3_BASE + 0xC408 + (((x) & 0x1F) << 2)) > > + > > +#define GEVNTSIZ_EVNTINTMASK BIT31 > > +#define GEVNTSIZ_EVNTSIZ_MASK (0xFFFF) > > +#define GEVNTSIZ_EVNTSIZ(x) ((x) & 0xFFFF) > > + > > +#define GEVNTCOUNT(x) (DW_USB3_BASE + 0xC40C + (((x) & 0x1F) << 2)) > > +#define GEVNTCOUNT_EVNTCOUNT_MASK (0xFFFF) > > +#define GEVNTCOUNT_EVNTCOUNT(x) ((x) & 0xFFFF) > > + > > +// Non-Endpoint specific event flag > > +#define GEVNT_INTTYPE_MASK (0x7F << 1) > > +#define GEVNT_INTTYPE(x) (((x) & 0x7F) << 1) > > +#define EVENT_I2C_INT 4 > > +#define EVENT_CARKIT_INT 3 > > +#define EVENT_OTG_INT 1 > > +#define EVENT_DEV_INT 0 > > + > > +#define GEVNT_NON_EP BIT0 > > +// Endpoint specific event flag > > +#define GEVNT_DEPEVT_INTTYPE_MASK (0xF << 6) > > +#define GEVNT_DEPEVT_INTTYPE(x) (((x) & 0xF) << 6) > > +#define GEVNT_DEPEVT_INTTYPE_EPCMD_CMPL (7 << 6) > > +#define GEVNT_DEPEVT_INTTYPE_STRM_EVT (6 << 6) > > +#define GEVNT_DEPEVT_INTTYPE_FIFOXRUN (4 << 6) > > +#define GEVNT_DEPEVT_INTTYPE_XFER_NRDY (3 << 6) > > +#define GEVNT_DEPEVT_INTTYPE_XFER_IN_PROG (2 << 6) > > +#define GEVNT_DEPEVT_INTTYPE_XFER_CMPL (1 << 6) > > +#define GEVNT_DEPEVT_EPNUM_MASK (0x1F << 1) > > +#define GEVNT_DEPEVT_EPNUM_SHIFT 1 > > +#define GEVNT_DEPEVT_EPNUM(x) (((x) & 0x1F) << 1) > > +// Devices specific event flag > > +#define GEVNT_DEVT_MASK (0xF << 8) > > +#define GEVNT_DEVT_SHIFT 8 > > +#define GEVNT_DEVT(x) (((x) & 0xF) << 8) > > +#define GEVNT_DEVT_INACT_TIMEOUT_RCVD (0x15 << 8) > > +#define GEVNT_DEVT_VNDR_DEV_TST_RCVD (0x14 << 8) > > +#define GEVNT_DEVT_OVERFLOW (0x13 << 8) > > +#define GEVNT_DEVT_CMD_CMPL (0x12 << 8) > > +#define GEVNT_DEVT_ERRATICERR (0x11 << 8) > > +#define GEVNT_DEVT_SOF (0x7 << 8) > > +#define GEVNT_DEVT_EOPF (0x6 << 8) > > +#define GEVNT_DEVT_HIBER_REQ (0x5 << 8) > > +#define GEVNT_DEVT_WKUP (0x4 << 8) > > +#define GEVNT_DEVT_ULST_CHNG (0x3 << 8) > > +#define GEVNT_DEVT_CONNDONE (0x2 << 8) > > +#define GEVNT_DEVT_USBRESET (0x1 << 8) > > +#define GEVNT_DEVT_DISCONN (0x0 << 8) > > + > > +#define DCFG (DW_USB3_BASE + 0xC700) > > + > > +#define DCFG_NUMP_MASK (0x1F << 17) > > +#define DCFG_NUMP(x) (((x) & 0x1F) << 17) > > +#define DCFG_DEVADDR_MASK (0x7F << 3) > > +#define DCFG_DEVADDR(x) (((x) & 0x7F) << 3) > > +#define DCFG_DEVSPD_MASK (0x7) > > +#define DCFG_DEVSPD(x) ((x) & 0x7) > > +#define DEVSPD_HS_PHY_30MHZ_OR_60MHZ 0 > > +#define DEVSPD_FS_PHY_30MHZ_OR_60MHZ 1 > > +#define DEVSPD_LS_PHY_6MHZ 2 > > +#define DEVSPD_FS_PHY_48MHZ 3 > > +#define DEVSPD_SS_PHY_125MHZ_OR_250MHZ 4 > > + > > +#define DCTL (DW_USB3_BASE + 0xC704) > > + > > +#define DCTL_RUN_STOP BIT31 > > +#define DCTL_CSFTRST BIT30 > > +#define DCTL_INIT_U2_EN BIT12 > > +#define DCTL_ACCEPT_U2_EN BIT11 > > +#define DCTL_INIT_U1_EN BIT10 > > +#define DCTL_ACCEPT_U1_EN BIT9 > > + > > +#define DEVTEN (DW_USB3_BASE + 0xC708) > > +#define DEVTEN_CONNECTDONEEN BIT2 > > +#define DEVTEN_USBRSTEN BIT1 > > +#define DEVTEN_DISCONNEN BIT0 > > + > > +#define DSTS (DW_USB3_BASE + 0xC70C) > > +#define DSTS_GET_DEVSPD(x) ((x) & 0x7) > > + > > +#define DALEPENA (DW_USB3_BASE + 0xC720) > > + > > +#define DEPCMDPAR2(x) (DW_USB3_BASE + 0xC800 + ((x) & 0x1F) * 0x10) > > +#define DEPCMDPAR1(x) (DW_USB3_BASE + 0xC804 + ((x) & 0x1F) * 0x10) > > +#define DEPCMDPAR0(x) (DW_USB3_BASE + 0xC808 + ((x) & 0x1F) * 0x10) > > +#define DEPCMD(x) (DW_USB3_BASE + 0xc80C + ((x) & 0x1F) * 0x10) > > + > > +#define DEPCMD_COMMANDPARAM_MASK (0xFFFF << 16) > > +#define DEPCMD_COMMANDPARAM(x) (((x) & 0xFFFF) << 16) > > +/* Stream Number or uFrame (input) */ > > +#define DEPCMD_STR_NUM_OR_UF_MASK (0xFFFF << 16) > > +#define DEPCMD_STR_NUM_OR_UF(x) (((x) & 0xFFFF) << 16) > > +/* Transfer Resource Index (output) */ > > +#define DEPCMD_XFER_RSRC_IDX_SHIFT 16 > > +#define DEPCMD_XFER_RSRC_IDX_MASK (0x7F << 16) > > +#define DEPCMD_XFER_RSRC_IDX(x) (((x) & 0x7F) << 16) > > +#define GET_DEPCMD_XFER_RSRC_IDX(x) (((x) >> 16) & 0x7F) > > +#define DEPCMD_CMDACT BIT10 > > +#define DEPCMD_CMDTYPE_MASK 0xFF > > +#define DEPCMD_CMDTYPE(x) ((x) & 0xFF) > > + > > +/* EP registers range as: OUT0, IN0, OUT1, IN1, ... */ > > +#define EP_OUT_IDX(x) ((x) * 2) > > +#define EP_IN_IDX(x) (((x) * 2) + 1) > > + > > +#define CMDTYPE_SET_EP_CFG 1 > > +#define CMDTYPE_SET_XFER_CFG 2 > > +#define CMDTYPE_GET_EP_STATE 3 > > +#define CMDTYPE_SET_STALL 4 > > +#define CMDTYPE_CLR_STALL 5 > > +#define CMDTYPE_START_XFER 6 > > +#define CMDTYPE_UPDATE_XFER 7 > > +#define CMDTYPE_END_XFER 8 > > +#define CMDTYPE_START_NEW_CFG 9 > > + > > +#define EPTYPE_CONTROL 0 > > +#define EPTYPE_ISOC 1 > > +#define EPTYPE_BULK 2 > > +#define EPTYPE_INTR 3 > > + > > +#define CFG_ACTION_INIT 0 > > +#define CFG_ACTION_RESTORE 1 > > +#define CFG_ACTION_MODIFY 2 > > + > > +#define EPCFG0_CFG_ACTION_MASK (0x3 << 30) > > +#define EPCFG0_CFG_ACTION(x) (((x) & 0x3) << 30) > > +#define EPCFG0_BRSTSIZ_MASK (0xF << 22) > > +#define EPCFG0_BRSTSIZ(x) (((x) & 0xF) << 22) > > +#define EPCFG0_TXFNUM_MASK (0x1F << 17) > > +#define EPCFG0_TXFNUM(x) (((x) & 0x1F) << 17) > > +#define EPCFG0_MPS_MASK (0x7FF << 3) > > +#define EPCFG0_MPS(x) (((x) & 0x7FF) << 3) > > +#define EPCFG0_EPTYPE_MASK (0x3 << 1) > > +#define EPCFG0_EPTYPE_SHIFT 1 > > +#define EPCFG0_EPTYPE(x) (((x) & 0x3) << 1) > > + > > +/* Endpoint Number */ > > +#define EPCFG1_EP_NUM_MASK (0xF << 26) > > +#define EPCFG1_EP_NUM(x) (((x) & 0xF) << 26) > > +/* Endpoint Direction */ > > +#define EPCFG1_EP_DIR_IN BIT25 > > +/* Stream Not Ready */ > > +#define EPCFG1_XFER_NRDY BIT10 > > +/* XferInProgress Enable */ > > +#define EPCFG1_XFER_IN_PROG BIT9 > > +/* Stream Completed */ > > +#define EPCFG1_XFER_CMPL BIT8 > > + > > +#define USB_SPEED_UNKNOWN 0 > > +#define USB_SPEED_LOW 1 > > +#define USB_SPEED_FULL 2 > > +#define USB_SPEED_HIGH 3 > > +#define USB_SPEED_VARIABLE 4 > > +#define USB_SPEED_SUPER 5 > > + > > +// DMA registers > > +#define DSCSTS_TRBRSP_MASK (0xF << 28) > > +#define DSCSTS_TRBRSP(x) (((x) & 0xF) << 28) > > +#define GET_DSCSTS_TRBRSP(x) (((x) >> 28) & 0xF) > > +#define TRBRSP_MISSED_ISOC_IN 1 > > +#define TRBRSP_SETUP_PEND 2 > > +#define TRBRSP_XFER_IN_PROG 4 > > +#define DSCSTS_PCM1_MASK (0x3 << 24) > > +#define DSCSTS_PCM1(x) (((x) & 0x3) << 24) > > +#define DSCSTS_XFERCNT_MASK 0xFFFFFF > > +#define DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) > > +#define GET_DSCSTS_XFERCNT(x) ((x) & 0xFFFFFF) > > + > > +#define DSCCTL_STRMID_SOFN(x) (((x) & 0xFFFF) << 14) > > +#define DSCCTL_IOC BIT11 > > +#define DSCCTL_ISP BIT10 > > +#define DSCCTL_TRBCTL_MASK (0x3F << 4) > > +#define DSCCTL_TRBCTL(x) (((x) & 0x3F) << 4) > > +#define DSCCTL_LST BIT1 > > +#define DSCCTL_HWO BIT0 > > +#define TRBCTL_NORMAL 1 > > +#define TRBCTL_SETUP 2 > > +#define TRBCTL_STATUS_2 3 > > +#define TRBCTL_STATUS_3 4 > > +#define TRBCTL_CTLDATA_1ST 5 > > +#define TRBCTL_ISOC_1ST 6 > > +#define TRBCTL_ISOC 7 > > +#define TRBCTL_LINK 8 > > +#define TRBCTL_NORMAL_ZLP 9 > > + > > + > > +#define UE_DIR_IN 0x80 > > +#define UE_DIR_OUT 0 > > +#define UE_SET_DIR(a, d) ((a) | (((d) & 1) << 7)) > > +#define UE_GET_DIR(a) ((a) & 0x80) > > +#define UE_ADDR 0x0F > > +#define UE_GET_ADDR(a) ((a) & UE_ADDR) > > + > > +#define UT_GET_DIR(a) ((a) & 0x80) > > +#define UT_WRITE 0x00 > > +#define UT_READ 0x80 > > + > > +#define UT_GET_TYPE(a) ((a) & 0x60) > > +#define UT_STANDARD 0x00 > > +#define UT_CLASS 0x20 > > +#define UT_VENDOR 0x40 > > + > > +#define UT_GET_RECIPIENT(a) ((a) & 0x1f) > > +#define UT_DEVICE 0x00 > > +#define UT_INTERFACE 0x01 > > +#define UT_ENDPOINT 0x02 > > +#define UT_OTHER 0x03 > > + > > +#define UR_GET_STATUS 0x00 > > +#define UR_CLEAR_FEATURE 0x01 > > +#define UR_SET_FEATURE 0x03 > > +#define UR_SET_ADDRESS 0x05 > > +#define UR_GET_DESCRIPTOR 0x06 > > +#define UR_SET_DESCRIPTOR 0x07 > > +#define UR_GET_CONFIG 0x08 > > +#define UR_SET_CONFIG 0x09 > > +#define UR_GET_INTERFACE 0x0A > > +#define UR_SET_INTERFACE 0x0B > > +#define UR_SYNCH_FRAME 0x0C > > +#define UR_SET_SEL 0x30 > > +#define UR_SET_ISOC_DELAY 0x31 > > + > > Somewhere around here starts a bunch of redefinitions of things > already described in MdePkg/Include/IndustryStandard/Usb.h. > Can you please reuse that include file as much as possible. > OK > > +/* Feature numbers */ > > +#define UF_ENDPOINT_HALT 0 > > +#define UF_DEVICE_REMOTE_WAKEUP 1 > > +#define UF_TEST_MODE 2 > > +#define UF_DEVICE_B_HNP_ENABLE 3 > > +#define UF_DEVICE_A_HNP_SUPPORT 4 > > +#define UF_DEVICE_A_ALT_HNP_SUPPORT 5 > > +#define UF_FUNCTION_SUSPEND 0 > > +#define UF_U1_ENABLE 48 > > +#define UF_U2_ENABLE 49 > > +#define UF_LTM_ENABLE 50 > > + > > +#define UDESC_DEVICE 0x01 > > +#define UDESC_CONFIG 0x02 > > +#define UDESC_STRING 0x03 > > +#define UDESC_INTERFACE 0x04 > > +#define UDESC_ENDPOINT 0x05 > > +#define UDESC_SS_USB_COMPANION 0x30 > > +#define UDESC_DEVICE_QUALIFIER 0x06 > > +#define UDESC_BOS 0x0f > > +#define UDESC_DEVICE_CAPABILITY 0x10 > > + > > +#define STRING_LANGUAGE 0 > > +#define STRING_MANUFACTURER 1 > > +#define STRING_PRODUCT 2 > > +#define STRING_SERIAL 3 > > + > > +#define CONFIG_VALUE 1 > > + > > +#define USB3_BULK_IN_EP 1 > > +#define USB3_BULK_OUT_EP 1 > > + > > +#define USB_ENUM_ADB_PORT_VID 0x18D1 > > +#define USB_ENUM_ADB_PORT_PID 0xD00D > > +#define USB_ENUM_INTERFACE_ADB_SUBCLASS 0x42 > > +#define USB_ENUM_INTERFACE_ADB_PROTOCOL 0x03 > > + > > +struct usb3_pcd; > > + > > +typedef enum pcd_state { > > + USB3_STATE_UNCONNECTED, /* no host */ > > + USB3_STATE_DEFAULT, > > + USB3_STATE_ADDRESSED, > > + USB3_STATE_CONFIGURED, > > +} pcdstate_e; > > These structs also break the coding style completely. > Will using IndustryStandard/Usb.h get rid of all of that? > OK > > + > > +typedef enum ep0_state { > > + EP0_IDLE, > > + EP0_IN_DATA_PHASE, > > + EP0_OUT_DATA_PHASE, > > + EP0_IN_WAIT_NRDY, > > + EP0_OUT_WAIT_NRDY, > > + EP0_IN_STATUS_PHASE, > > + EP0_OUT_STATUS_PHASE, > > + EP0_STALL, > > +} ep0state_e; > > + > > +typedef struct usb3_dma_desc { > > + /** Buffer Pointer - Low address quadlet */ > > + UINT32 bptl; > > + > > + /** Buffer Pointer - High address quadlet */ > > + UINT32 bpth; > > + > > + /** Status quadlet. Fields defined in enum @ref desc_sts_data. */ > > + UINT32 status; > > + > > + /** Control quadlet. Fields defined in enum @ref desc_ctl_data. */ > > + UINT32 control; > > +} usb3_dma_desc_t; > > + > > +typedef struct usb3_pcd_req { > > + usb3_dma_desc_t *trb; > > + UINT64 trbdma; > > + > > + UINT32 length; > > + UINT32 actual; > > + > > + UINT64 *bufdma; > > + int (*complete)(unsigned actual, int status); > > +} usb3_pcd_req_t; > > + > > +typedef struct usb_device_request { > > + UINT8 bmRequestType; > > + UINT8 bRequest; > > + UINT16 wValue; > > + UINT16 wIndex; > > + UINT16 wLength; > > +} usb_device_request_t; > > + > > +#pragma pack(1) > > +/** USB_DT_DEVICE: Device descriptor */ > > +typedef struct usb_device_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT16 bcdUSB; > > +#define USB_CLASS_COMM 0x02 > > +#define USB_CLASS_VENDOR_SPEC 0xFF > > +#define USB_SC_VENDOR_SPEC 0xFF > > +#define USB_PR_VENDOR_SPEC 0xFF > > + UINT8 bDeviceClass; > > + UINT8 bDeviceSubClass; > > + UINT8 bDeviceProtocol; > > + UINT8 bMaxPacketSize0; > > + UINT16 idVendor; > > + UINT16 idProduct; > > + UINT16 bcdDevice; > > + UINT8 iManufacturer; > > + UINT8 iProduct; > > + UINT8 iSerialNumber; > > + UINT8 bNumConfigurations; > > +} usb_device_descriptor_t; > > + > > +/* USB_DT_CONFIG: Config descriptor */ > > +typedef struct usb_config_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT16 wTotalLength; > > + UINT8 bNumInterfaces; > > +#define CONFIG_VALUE 1 > > + UINT8 bConfigurationValue; > > + UINT8 iConfiguration; > > +#define USB_CONFIG_ATT_ONE (1 << 7) > > + UINT8 bmAttributes; > > +#define USB_CONFIG_VBUS_DRAW (0xFA) > > + UINT8 bMaxPower; > > +} usb_config_descriptor_t; > > + > > +/* USB_DT_DEVICE_QUALIFIER: Device Qualifier descriptor */ > > +typedef struct usb_qualifier_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT16 bcdUSB; > > + UINT8 bDeviceClass; > > + UINT8 bDeviceSubClass; > > + UINT8 bDeviceProtocol; > > + UINT8 bMaxPacketSize0; > > + UINT8 bNumConfigurations; > > + UINT8 bRESERVED; > > +} usb_qualifier_descriptor_t; > > + > > +/* USB_DT_INTERFACE: Interface descriptor */ > > +typedef struct usb_interface_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT8 bInterfaceNumber; > > + UINT8 bAlternateSetting; > > + UINT8 bNumEndpoints; > > + UINT8 bInterfaceClass; > > + UINT8 bInterfaceSubClass; > > + UINT8 bInterfaceProtocol; > > + UINT8 iInterface; > > +} usb_interface_descriptor_t; > > + > > +/* USB_DT_ENDPOINT: Endpoint descriptor */ > > +typedef struct usb_endpoint_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT8 bEndpointAddress; > > + UINT8 bmAttributes; > > +#define USB_ENDPOINT_XFER_CONTROL 0x00 > > +#define USB_ENDPOINT_XFER_ISOC 0x01 > > +#define USB_ENDPOINT_XFER_BULK 0x02 > > +#define USB_ENDPOINT_XFER_INT 0x03 > > + UINT16 wMaxPacketSize; > > + UINT8 bInterval; > > +} usb_endpoint_descriptor_t; > > + > > +/* USB_DT_SS_ENDPOINT_COMP: SuperSpeed Endpoint Companion descriptor */ > > +typedef struct usb_ss_ep_comp_descriptor { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + > > + UINT8 bMaxBurst; > > + UINT8 bmAttributes; > > + UINT16 wBytesPerInterval; > > +} usb_ss_ep_comp_descriptor_t; > > + > > +/* WUSB BOS Descriptor (Binary device Object Store) */ > > +typedef struct wusb_bos_desc { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + UINT16 wTotalLength; > > + UINT8 bNumDeviceCaps; > > +} wusb_bos_desc_t; > > + > > +#define USB_DEVICE_CAPABILITY_20_EXTENSION 0x02 > > +typedef struct usb_dev_cap_20_ext_desc { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + UINT8 bDevCapabilityType; > > +#define USB_20_EXT_LPM 0x02 > > + UINT32 bmAttributes; > > +} usb_dev_cap_20_ext_desc_t; > > + > > +#define USB_DEVICE_CAPABILITY_SS_USB 0x03 > > +typedef struct usb_dev_cap_ss_usb { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + UINT8 bDevCapabilityType; > > +#define USB_DC_SS_USB_LTM_CAPABLE 0x02 > > + UINT8 bmAttributes; > > +#define USB_DC_SS_USB_SPEED_SUPPORT_LOW 0x01 > > +#define USB_DC_SS_USB_SPEED_SUPPORT_FULL 0x02 > > +#define USB_DC_SS_USB_SPEED_SUPPORT_HIGH 0x04 > > +#define USB_DC_SS_USB_SPEED_SUPPORT_SS 0x08 > > + UINT32 wSpeedsSupported; > > + UINT8 bFunctionalitySupport; > > + UINT8 bU1DevExitLat; > > + UINT32 wU2DevExitLat; > > +} usb_dev_cap_ss_usb_t; > > + > > +#define USB_DEVICE_CAPABILITY_CONTAINER_ID 0x04 > > +typedef struct usb_dev_cap_container_id { > > + UINT8 bLength; > > + UINT8 bDescriptorType; > > + UINT8 bDevCapabilityType; > > + UINT8 bReserved; > > + UINT8 containerID[16]; > > +} usb_dev_cap_container_id_t; > > +#pragma pack() > > + > > +typedef union usb_setup_pkt { > > + usb_device_request_t req; > > + UINT32 d32[2]; > > + UINT8 d8[8]; > > +} usb_setup_pkt_t; > > + > > +typedef struct usb3_pcd_ep { > > + struct usb3_pcd *pcd; > > + > > + UINT8 EpInIdx; > > + UINT8 EpOutIdx; > > + UINT8 phys; > > + > > + //UINT8 phys; > > + UINT8 num; > > + UINT8 type; > > + UINT8 maxburst; > > + UINT16 maxpacket; > > + /* Tx FIFO # for IN EPs */ > > + UINT8 tx_fifo_num; > > + > > + /* The Transfer Resource Index from the Start Transfer command */ > > + UINT8 tri_out; > > + UINT8 tri_in; > > + > > + UINT8 stopped; > > + /* Send ZLP */ > > + UINT8 send_zlp; > > + /* True if 3-stage control transfer */ > > + UINT8 three_stage; > > + /* True if transfer has been started on EP */ > > + UINT8 xfer_started; > > + /* EP direction 0 = OUT */ > > + UINT8 is_in; > > + /* True if endpoint is active */ > > + UINT8 active; > > + /* Initial data pid of bulk endpoint */ > > + UINT8 data_pid_start; > > + > > + /* ep_desc (excluding ep0) */ > > + usb3_dma_desc_t *ep_desc; > > + > > +#if 0 > > + /* TRB descriptor must be aligned to 16 bytes */ > > + UINT8 epx_desc[32]; > > +#endif > > + > > + /* request (excluding ep0) */ > > + usb3_pcd_req_t req; > > +} usb3_pcd_ep_t; > > + > > +typedef struct usb3_pcd { > > + //struct usb3_device *usb3_dev; > > + > > + INT32 link_state; > > + pcdstate_e state; > > + UINT8 new_config; > > + ep0state_e ep0state; > > + > > + UINT32 eps_enabled; > > + UINT32 ltm_enable; > > + > > + usb3_pcd_ep_t ep0; > > + usb3_pcd_ep_t out_ep; > > + usb3_pcd_ep_t in_ep; > > + > > + /* > > + usb3_dev_global_regs_t *dev_global_regs; > > + usb3_dev_ep_regs_t *out_ep_regs; > > + usb3_dev_ep_regs_t *in_ep_regs; > > + */ > > + > > + usb3_pcd_req_t ep0_req; > > + > > + UINT8 speed; > > + > > + usb3_dma_desc_t *ep0_setup_desc; > > + usb3_dma_desc_t *ep0_in_desc; > > + usb3_dma_desc_t *ep0_out_desc; > > + > > + /* TRB descriptor must be aligned to 16 bytes */ > > +#if 0 > > + UINT8 ep0_setup[32]; > > + UINT8 ep0_in[32]; > > + UINT8 ep0_out[32]; > > + > > + usb_setup_pkt_t ep0_setup_pkt[5]; > > + > > +#define USB3_STATUS_BUF_SIZE 512 > > + UINT8 ep0_status_buf[USB3_STATUS_BUF_SIZE]; > > + > > +#define USB3_BULK_BUF_SIZE 2048 > > + UINT8 ss_bulk_buf[USB3_BULK_BUF_SIZE]; > > +#endif > > + > > + UINT32 file_type; > > + UINT32 file_address; > > + UINT32 file_capacity; > > + UINT32 file_total_frame; > > + UINT32 file_curr_frame; > > + UINT32 file_next_frame; > > + UINT32 file_received; > > + UINT32 file_complete; > > + > > + UINT16 test_mode_nr; > > + UINT16 test_mode; > > +} usb3_pcd_t; > > + > > +struct usb_enum_port_param { > > + UINT16 idVendor; > > + UINT16 idProduct; > > + UINT8 bInterfaceSubClass; > > + UINT8 bInterfaceProtocol; > > +}; > > + > > +#if 0 > > If not used, leave it out. > OK > > +typedef struct usb3_pcd_req { > > + usb3_dma_desc_t *trb; > > + UINT64 trbdma; > > + > > + UINT32 length; > > + UINT32 actual; > > + > > + UINT64 *bufdma; > > + int (*complete)(unsigned actual, int status); > > +} usb3_pcd_req_t; > > + > > +#endif > > + > > +#endif /* __DW_USB3_DXE_H__ */ > > diff --git a/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c > > new file mode 100644 > > index 000000000000..83d5e4736de0 > > --- /dev/null > > +++ b/EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c > > @@ -0,0 +1,2434 @@ > > +/** @file > > + > > + Copyright (c) 2018, Linaro Limited. All rights reserved. > > + > > + This program and the accompanying materials are licensed and made available > > + under the terms and conditions of the BSD License which accompanies this > > + distribution. The full text of the license may be found at > > + http://opensource.org/licenses/bsd-license.php > > + > > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > > + > > +**/ > > + > > +#include <IndustryStandard/Usb.h> > > +#include <Library/ArmLib.h> > > +#include <Library/BaseLib.h> > > +#include <Library/BaseMemoryLib.h> > > +#include <Library/CacheMaintenanceLib.h> > > +#include <Library/DebugLib.h> > > +#include <Library/DmaLib.h> > > +#include <Library/IoLib.h> > > +#include <Library/MemoryAllocationLib.h> > > +#include <Library/TimerLib.h> > > +#include <Library/UefiBootServicesTableLib.h> > > +#include <Library/UefiDriverEntryPoint.h> > > +#include <Library/UefiRuntimeServicesTableLib.h> > > +#include <Protocol/DwUsb.h> > > +#include <Protocol/DriverBinding.h> > > +#include <Protocol/UsbDevice.h> > > + > > +#include "DwUsb3Dxe.h" > > + > > +#define FIFO_DIR_TX 0 > > +#define FIFO_DIR_RX 1 > > + > > +#define TX_FIFO_ADDR 0 > > +#define RX_FIFO_ADDR 0 > > + > > +#define RAM_WIDTH 8 > > +#define RAM_TX0_DEPTH 2048 > > +#define RAM_TX1_DEPTH 4096 > > +#define RAM_RX_DEPTH 8192 > > Should these not also go into DwUsb3Dxe.h? > OK > > + > > +#define USB_TYPE_LENGTH 16 > > +#define USB_BLOCK_HIGH_SPEED_SIZE 512 > > +#define DATA_SIZE 131072 > > +#define CMD_SIZE 512 > > What do the above definitions mean? > I'll add comment. > > +#define MATCH_CMD_LITERAL(Cmd, Buf) !AsciiStrnCmp (Cmd, Buf, sizeof (Cmd) - 1) > > This one does not appear to be used? > OK > > + > > +#define ALIGN(x, a) (((x) + ((a) - 1)) & ~((a) - 1)) > > Use available macros in Base.h. > OK > > + > > +// > > +// The time between interrupt polls, in units of 100 nanoseconds > > +// 10 Microseconds > > +// > > +#define DW_INTERRUPT_POLL_PERIOD 100 > > + > > +#define DWUSB3_EVENT_BUF_SIZE 256 > > + > > +// > > +// Maxpacket size for EP0, defined by USB3 spec > > +// > > +#define USB3_MAX_EP0_SIZE 512 > > + > > +// > > +// Maxpacket size for any EP, defined by USB3 spec > > +// > > +#define USB3_MAX_PACKET_SIZE 1024 > > +#define USB2_HS_MAX_PACKET_SIZE 512 > > +#define USB2_FS_MAX_PACKET_SIZE 64 > > + > > +#define USB3_STATE_UNCONNECTED 0 > > +#define USB3_STATE_DEFAULT 1 > > +#define USB3_STATE_ADDRESSED 2 > > +#define USB3_STATE_CONFIGURED 3 > > + > > +#define USB3_STATUS_BUF_SIZE 512 > > If these are defined by USB3 spec, please add them to > MdePkg/Include/IndustryStandard/Usb.h instead of defining them > locally. > OK > > + > > +#define GET_EVENTBUF_COUNT() \ > > + (GEVNTCOUNT_EVNTCOUNT (MmioRead32 (GEVNTCOUNT (0)))) > > +#define UPDATE_EVENTBUF_COUNT(x) \ > > + (MmioWrite32 (GEVNTCOUNT (0), GEVNTCOUNT_EVNTCOUNT (x))) > > + > > +#define SET_DEVADDR(x) \ > > + (MmioAndThenOr32 (DCFG, ~DCFG_DEVADDR_MASK, DCFG_DEVADDR (x))) > > Should these not also move to DwUsb3Dxe.h? > OK > > + > > +EFI_GUID gDwUsbProtocolGuid = DW_USB_PROTOCOL_GUID; > > This looks less than ideal - this makes it impossible to include > DwUsb2 and DwUsb3 drivers in the same image. > On the view of hardware, DwUsb2 and DwUsb3 doesn't exist at the same time. > > + > > +STATIC DW_USB_PROTOCOL *DwUsb; > > + > > +STATIC usb3_pcd_t gPcd; > > This seems a very unfortunate choice for variable name. > OK. I'll rename it. > > +STATIC UINT32 *gEventBuf, *gEventPtr; > > +STATIC struct usb_device_descriptor gDwUsb3DevDesc; > > +STATIC VOID *gRxBuf; > > + > > +STATIC usb_setup_pkt_t *gEndPoint0SetupPacket; > > +STATIC UINT8 *gEndPoint0StatusBuf; > > Although most if not all of these will disappear when converting to > UEFI driver model, why do they have 'g' prefix in the first place? > 'g' is for globally visible symbols. Variables that are "global" in > this file only are better off using 'm'. OK. I'll fix it. > > > +STATIC USB_DEVICE_RX_CALLBACK mDataReceivedCallback; > > +STATIC UINTN mDataBufferSize; > > + > > +struct usb_interface_descriptor intf = { > > + sizeof (struct usb_interface_descriptor), > > + UDESC_INTERFACE, > > + 0, > > + 0, > > + 2, > > + USB_CLASS_VENDOR_SPEC, > > + 0x42, > > + 0x03, > > + 0 > > +}; > > + > > +const struct usb_ss_ep_comp_descriptor ep_comp = { > > + sizeof (struct usb_ss_ep_comp_descriptor), > > + UDESC_SS_USB_COMPANION, > > + 0, > > + 0, > > + 0 > > +}; > > + > > +const struct usb_endpoint_descriptor hs_bulk_in = { > > + sizeof (struct usb_endpoint_descriptor), > > + UDESC_ENDPOINT, > > + UE_DIR_IN | USB3_BULK_IN_EP, > > + USB_ENDPOINT_XFER_BULK, > > + 0x200, > > + 0 > > +}; > > + > > +const struct usb_endpoint_descriptor > > +hs_bulk_out = { > > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > > + UDESC_ENDPOINT, /* bDescriptorType */ > > + > > + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ > > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > > + 0x200, /* wMaxPacketSize: 512 of high-speed */ > > + 1, /* bInterval */ > > +}; > > + > > +const struct usb_endpoint_descriptor ss_bulk_in = { > > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > > + UDESC_ENDPOINT, /* bDescriptorType */ > > + > > + UE_DIR_IN | USB3_BULK_IN_EP, /* bEndpointAddress */ > > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > > + 0x400, /* wMaxPacketSize: 1024 of super-speed */ > > + 0, /* bInterval */ > > +}; > > + > > +const struct usb_endpoint_descriptor ss_bulk_out = { > > + sizeof(struct usb_endpoint_descriptor), /* bLength */ > > + UDESC_ENDPOINT, /* bDescriptorType */ > > + > > + UE_DIR_OUT | USB3_BULK_OUT_EP, /* bEndpointAddress */ > > + USB_ENDPOINT_XFER_BULK, /* bmAttributes */ > > + 0x400, /* wMaxPacketSize: 1024 of super-speed */ > > + 0, /* bInterval */ > > +}; > > + > > +/** The BOS Descriptor */ > > What's a BOS? > Let me add comment on it. > > + > > +const struct usb_dev_cap_20_ext_desc cap1 = { > > + sizeof(struct usb_dev_cap_20_ext_desc), /* bLength */ > > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > > + USB_DEVICE_CAPABILITY_20_EXTENSION, /* bDevCapabilityType */ > > + 0x2, /* bmAttributes */ > > +}; > > + > > +const struct usb_dev_cap_ss_usb > > +cap2 = { > > + sizeof(struct usb_dev_cap_ss_usb), /* bLength */ > > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > > + USB_DEVICE_CAPABILITY_SS_USB, /* bDevCapabilityType */ > > + 0x0, /* bmAttributes */ > > + (USB_DC_SS_USB_SPEED_SUPPORT_SS | > > + USB_DC_SS_USB_SPEED_SUPPORT_HIGH), /* wSpeedsSupported */ > > + 0x2, /* bFunctionalitySupport */ > > + 0xa, /* bU1DevExitLat */ > > + 0x100, /* wU2DevExitLat */ > > +}; > > + > > +const struct usb_dev_cap_container_id > > +cap3 = { > > + sizeof(struct usb_dev_cap_container_id),/* bLength */ > > + UDESC_DEVICE_CAPABILITY, /* bDescriptorType */ > > + USB_DEVICE_CAPABILITY_CONTAINER_ID, /* bDevCapabilityType */ > > + 0, /* bReserved */ > > + {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, /* containerID */ > > +}; > > + > > +const struct wusb_bos_desc > > +bos = { > > + sizeof(struct wusb_bos_desc), /* bLength */ > > + UDESC_BOS, /* bDescriptorType */ > > + (sizeof(struct wusb_bos_desc) + \ > > + sizeof(cap1) + sizeof(cap2) + \ > > + sizeof(cap3)), /* wTotalLength */ > > + 3, /* bNumDeviceCaps */ > > +}; > > + > > +STATIC struct usb_enum_port_param usb_port_activity_config = { > > + .idVendor = USB_ENUM_ADB_PORT_VID, > > + .idProduct = USB_ENUM_ADB_PORT_PID, > > + .bInterfaceSubClass = USB_ENUM_INTERFACE_ADB_SUBCLASS, > > + .bInterfaceProtocol = USB_ENUM_INTERFACE_ADB_PROTOCOL > > +}; > > + > > +STATIC > > +UINT32 > > +DwUsb3GetEventBufEvent ( > > + IN UINTN Size > > + ) > > +{ > > + UINT32 Event; > > + > > + Event = *gEventPtr++; > > + if ((UINT32)(UINTN)gEventPtr >= (UINT32)(UINTN)gEventBuf + Size) { > > + gEventPtr = gEventBuf; > > + } > > + return Event; > > +} > > + > > +STATIC > > +VOID > > +DwUsb3SetFifoSize ( > > + IN UINT32 Addr, > > + IN UINT32 Depth, > > + IN UINT32 Dir, > > + IN UINT32 FifoNum > > + ) > > +{ > > + UINT32 Reg = 0; > > + > > + if (Dir == FIFO_DIR_TX) { > > + Reg = GTXFIFOSIZ (FifoNum); > > + } else if (Dir == FIFO_DIR_RX) { > > + Reg = GRXFIFOSIZ (FifoNum); > > + } else { > > + ASSERT (0); > > + } > > + MmioWrite32 (Reg, FIFOSIZ_DEP (Depth) | FIFOSIZ_ADDR (Addr)); > > +} > > + > > +STATIC > > +UINT32 > > +Handshake ( > > What does this function do? "Handshake" does not tell me enough to > figure out. > > > + IN UINT32 Reg, > > + IN UINT32 Mask, > > + IN UINT32 Done > > + ) > > +{ > > + UINT32 Timeout = 100000; > > + > > + do { > > + if ((MmioRead32 (Reg) & Mask) == Done) { > > + return 1; > > + } > > + MicroSecondDelay (1); > > + } while (Timeout-- > 0); > > + return 0; > > +} > > + > > +STATIC > > +VOID > > +DwUsb3FillDesc ( > > + IN usb3_dma_desc_t *desc, > > + IN UINT64 dma_addr, > > + IN UINT32 dma_len, > > + IN UINT32 stream, > > + IN UINT32 type, > > + IN UINT32 ctrlbits, > > + IN UINT32 own > > + ) > > +{ > > + desc->bptl = (UINT32)(dma_addr & 0xFFFFFFFF); > > + desc->bpth = (UINT32)(dma_addr >> 32); > > Could you possibly create LOW_32BIT and HIGH_32BIT macros for > MdePkg/Base.h instead? We are already duplicating these operations all > over the place. Including all over MdeModulePkg. > OK > > + desc->status = DSCSTS_XFERCNT (dma_len); > > + if (type) { > > + desc->control = DSCCTL_TRBCTL (type); > > + } > > + desc->control |= DSCCTL_STRMID_SOFN (stream) | ctrlbits; > > + ArmDataSynchronizationBarrier (); > > Please use MemoryFence() instead. > This controller is not ARM-specific. > > (DSB? That sounds excessive. DMB should suffice?) > OK > > + /* must execute this operation at last */ > > + if (own) { > > + desc->control |= DSCCTL_HWO; > > + } > > + ArmDataSynchronizationBarrier (); > > As above. > OK > > +} > > + > > +STATIC > > +VOID > > +DwUsb3DepStartNewCfg ( > > + IN UINT32 EpIdx, > > + IN UINT32 RsrcIdx > > + ) > > +{ > > + /* start the command */ > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_XFER_RSRC_IDX (RsrcIdx) | DEPCMD_CMDTYPE (CMDTYPE_START_NEW_CFG) | \ > > + DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3DepCfg ( > > Please no abbreviations in function names. This name tells me nothing > about what the function does (and there is no comment describing its > function). > OK > > + IN UINT32 EpIdx, > > + IN UINT32 DepCfg0, > > + IN UINT32 DepCfg1, > > + IN UINT32 DepCfg2 > > + ) > > +{ > > + MmioWrite32 (DEPCMDPAR2 (EpIdx), DepCfg2); > > + MmioWrite32 (DEPCMDPAR1 (EpIdx), DepCfg1); > > + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepCfg0); > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_CMDTYPE (CMDTYPE_SET_EP_CFG) | DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3DepXferCfg ( > > Dep<something>TransferConfig. > Please address throughout. > > > + IN UINT32 EpIdx, > > + IN UINT32 DepStrmCfg > > + ) > > +{ > > + MmioWrite32 (DEPCMDPAR0 (EpIdx), DepStrmCfg); > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_CMDTYPE (CMDTYPE_SET_XFER_CFG) | DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +STATIC > > +UINT8 > > +DwUsb3DepStartXfer ( > > + IN UINT32 EpIdx, > > + IN UINT64 DmaAddr, > > + IN UINT32 StreamOrUf > > + ) > > +{ > > + UINT32 Data; > > + > > + MmioWrite32 (DEPCMDPAR1 (EpIdx), (UINT32)DmaAddr); > > + MmioWrite32 (DEPCMDPAR0 (EpIdx), (UINT32)(DmaAddr >> 32)); > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_STR_NUM_OR_UF (StreamOrUf) | DEPCMD_CMDTYPE (CMDTYPE_START_XFER) | \ > > + DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > + Data = MmioRead32 (DEPCMD (EpIdx)); > > + return GET_DEPCMD_XFER_RSRC_IDX(Data); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3DepStopXfer ( > > + IN UINT32 EpIdx, > > + IN UINT32 Tri > > + ) > > +{ > > + MmioWrite32 (DEPCMDPAR2 (EpIdx), 0); > > + MmioWrite32 (DEPCMDPAR1 (EpIdx), 0); > > + MmioWrite32 (DEPCMDPAR0 (EpIdx), 0); > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_END_XFER) | \ > > + DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +VOID > > +DwUsb3DepUpdateXfer ( > > + IN UINT32 EpIdx, > > + IN UINT32 Tri > > + ) > > +{ > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_XFER_RSRC_IDX (Tri) | DEPCMD_CMDTYPE (CMDTYPE_UPDATE_XFER) | \ > > + DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3DepClearStall ( > > + IN UINTN EpIdx > > + ) > > +{ > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_CMDTYPE (CMDTYPE_CLR_STALL) | DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > + > > +STATIC > > +VOID > > +DwUsb3DepSetStall ( > > + IN UINTN EpIdx > > + ) > > +{ > > + MmioWrite32 ( > > + DEPCMD (EpIdx), > > + DEPCMD_CMDTYPE (CMDTYPE_SET_STALL) | DEPCMD_CMDACT > > + ); > > + Handshake (DEPCMD (EpIdx), DEPCMD_CMDACT, 0); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3EnableEp ( > > EnableEndpoint > OK > > + IN UINT32 EpIdx, > > EndpointIndex. > OK > > + IN usb3_pcd_ep_t *ep > > Endpoint. > OK > > + ) > > +{ > > + UINT32 Dalepena; > > What? > > > + > > + Dalepena = MmioRead32 (DALEPENA); > > + /* If the EP is already enabled, skip to set it again. */ > > endpoint > > > + if (Dalepena & (1 << EpIdx)) { > > + return; > > + } > > + Dalepena |= 1 << EpIdx; > > + MmioWrite32 (DALEPENA, Dalepena); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3Ep0Activate ( > > (EP0 is special, so may not need expanding.) > > > + IN OUT usb3_pcd_t *pcd > > + ) > > +{ > > + /* issue DEPCFG command to EP0 OUT */ > > ... except for in comments. > > > + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 0); > > + DwUsb3DepCfg ( > > + EP_OUT_IDX (0), > > + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512), > > + EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY, > > + 0 > > + ); > > + /* issue DEPSTRMCFG command to EP0 OUT */ > > + DwUsb3DepXferCfg (EP_OUT_IDX (0), 1); // one stream > > + /* issue DEPCFG command to EP0 IN */ > > + DwUsb3DepCfg ( > > + EP_IN_IDX (0), > > + EPCFG0_EPTYPE (EPTYPE_CONTROL) | EPCFG0_MPS (512) | \ > > + EPCFG0_TXFNUM (pcd->ep0.tx_fifo_num), > > + EPCFG1_XFER_NRDY | EPCFG1_XFER_CMPL | EPCFG1_EP_DIR_IN, > > + 0 > > + ); > > + /* issue DEPSTRMCFG command to EP0 IN */ > > + DwUsb3DepXferCfg (EP_IN_IDX (0), 1); // one stream > > + pcd->ep0.active = 1; > > +} > > + > > +STATIC > > +VOID > > +DwUsb3EpActivate ( > > + IN OUT usb3_pcd_t *pcd, > > + IN OUT usb3_pcd_ep_t *ep > > + ) > > +{ > > + UINT32 EpIdx, DepCfg0, DepCfg1; > > + if (ep->is_in) { > > + EpIdx = EP_IN_IDX (ep->num); > > + } else { > > + EpIdx = EP_OUT_IDX (ep->num); > > + } > > + > > + /* Start a new configurate when enable the first EP. */ > > + if (!pcd->eps_enabled) { > > + pcd->eps_enabled = 1; > > + /* Issue DEPCFG command to physical EP1 (logical EP0 IN) first. > > + * It resets the core's Tx FIFO mapping table. > > + */ > > + DepCfg0 = EPCFG0_EPTYPE (EPTYPE_CONTROL); > > + DepCfg0 |= EPCFG0_CFG_ACTION (CFG_ACTION_MODIFY); > > + DepCfg1 = EPCFG1_XFER_CMPL | EPCFG1_XFER_NRDY | EPCFG1_EP_DIR_IN; > > + > > + switch (pcd->speed) { > > + case USB_SPEED_SUPER: > > + DepCfg0 |= EPCFG0_MPS (512); > > + break; > > + case USB_SPEED_HIGH: > > + case USB_SPEED_FULL: > > + DepCfg0 |= EPCFG0_MPS (64); > > + break; > > + case USB_SPEED_LOW: > > + DepCfg0 |= EPCFG0_MPS (8); > > No live coding of values. > Add #defines for those speed setting values. > > > + break; > > + default: > > + ASSERT (0); > > + break; > > + } > > + DwUsb3DepCfg (EP_IN_IDX (0), DepCfg0, DepCfg1, 0); > > What does that final 0 do? > > > + DwUsb3DepStartNewCfg (EP_OUT_IDX (0), 2); > > And that 2? > > Please address throughout. > > > + } > > + /* issue DEPCFG command to EP */ > > + DepCfg0 = EPCFG0_EPTYPE (ep->type); > > + DepCfg0 |= EPCFG0_MPS (ep->maxpacket); > > + if (ep->is_in) { > > + DepCfg0 |= EPCFG0_TXFNUM (ep->tx_fifo_num); > > + } > > + DepCfg0 |= EPCFG0_BRSTSIZ (ep->maxburst); > > + DepCfg1 = EPCFG1_EP_NUM (ep->num); > > + if (ep->is_in) { > > + DepCfg1 |= EPCFG1_EP_DIR_IN; > > + } else { > > + DepCfg1 |= EPCFG1_XFER_CMPL; > > + } > > + DwUsb3DepCfg (EpIdx, DepCfg0, DepCfg1, 0); > > + /* issue DEPSTRMCFG command to EP */ > > + DwUsb3DepXferCfg (EpIdx, 1); > > + DwUsb3EnableEp (EpIdx, ep); > > + ep->active = 1; > > +} > > + > > +STATIC > > +VOID > > +DwUsb3Ep0OutStart ( > > + IN usb3_pcd_t *pcd > > + ) > > +{ > > + usb3_dma_desc_t *desc; > > + > > + /* Get the SETUP packet DMA Descriptor (TRB) */ > > + desc = pcd->ep0_setup_desc; > > + > > + /* DMA Descriptor setup */ > > + DwUsb3FillDesc ( > > + desc, > > + (UINT64)gEndPoint0SetupPacket, > > + pcd->ep0.maxpacket, > > + 0, > > + TRBCTL_SETUP, > > + DSCCTL_IOC | DSCCTL_ISP | DSCCTL_LST, > > + 1 > > + ); > > + > > + /* issue DEPSTRTXFER command to EP0 OUT */ > > + pcd->ep0.tri_out = DwUsb3DepStartXfer (EP_OUT_IDX (0), (UINT64)desc, 0); > > +} > > + > > +STATIC > > +VOID > > +DwUsb3Init ( > > + VOID > > + ) > > +{ > > + UINT32 Data, Addr; > > Address. > > > + usb3_pcd_t *pcd = &gPcd; > > + > > + /* soft reset the usb core */ > > + do { > > + MmioAndThenOr32 (DCTL, ~DCTL_RUN_STOP, DCTL_CSFTRST); > > + > > + do { > > + MicroSecondDelay (1000); > > + Data = MmioRead32 (DCTL); > > + } while (Data & DCTL_CSFTRST); > > + // > > + // wait for at least 3 PHY clocks > > + // > > + MicroSecondDelay (1000); > > + } while (0); > > + > > + pcd->link_state = 0; > > + > > + /* TI PHY: Set Turnaround Time = 9 (8-bit UTMI+ / ULPI) */ > > + MmioAndThenOr32 ( > > + GUSB2PHYCFG (0), > > + ~GUSB2PHYCFG_USBTRDTIM_MASK, > > + GUSB2PHYCFG_USBTRDTIM (9) > > How long is 9? > > I will stop my review here and give a full review on v2 when the > overarching comments have been addressed. > OK Best Regards Haojian ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1 0/2] add DwUsb3Dxe driver 2018-08-20 10:31 [PATCH v1 0/2] add DwUsb3Dxe driver Haojian Zhuang 2018-08-20 10:31 ` [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol Haojian Zhuang 2018-08-20 10:31 ` [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver Haojian Zhuang @ 2018-10-04 14:15 ` Leif Lindholm 2018-10-22 2:09 ` Haojian Zhuang 2 siblings, 1 reply; 9+ messages in thread From: Leif Lindholm @ 2018-10-04 14:15 UTC (permalink / raw) To: Haojian Zhuang; +Cc: edk2-devel, Ard Biesheuvel Hi Haojian, I will start with a few high-level requests: - Could you rework this for inclusion in edk2-platforms instead? Silicon/Synopsys I guess? - Could you submit Usb2 and Usb3 support in a single set for v2? - Can you convert these to UEFI driver model with NonDiscoverableDeviceRegistrationLib instead of a hard-coded base address? I will provide detailed feedback on the individual patches separately, so please wait for that before creating a v2. / Leif On Mon, Aug 20, 2018 at 06:31:23PM +0800, Haojian Zhuang wrote: > Changelog: > v1: > * Add Designware USB 3.0 device driver. > > Haojian Zhuang (2): > EmbeddedPkg: add DwUsb protocol > EmbeddedPkg/Drivers: add DwUsb3Dxe driver > > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c | 2434 +++++++++++++++++++++++++++ > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec | 44 + > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h | 632 +++++++ > EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf | 52 + > EmbeddedPkg/Include/Protocol/DwUsb.h | 81 + > 5 files changed, 3243 insertions(+) > create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.c > create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.dec > create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.h > create mode 100644 EmbeddedPkg/Drivers/DwUsb3Dxe/DwUsb3Dxe.inf > create mode 100644 EmbeddedPkg/Include/Protocol/DwUsb.h > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1 0/2] add DwUsb3Dxe driver 2018-10-04 14:15 ` [PATCH v1 0/2] " Leif Lindholm @ 2018-10-22 2:09 ` Haojian Zhuang 0 siblings, 0 replies; 9+ messages in thread From: Haojian Zhuang @ 2018-10-22 2:09 UTC (permalink / raw) To: Leif Lindholm; +Cc: edk2-devel@lists.01.org, Ard Biesheuvel On Thu, 4 Oct 2018 at 22:15, Leif Lindholm <leif.lindholm@linaro.org> wrote: > > Hi Haojian, > > I will start with a few high-level requests: > - Could you rework this for inclusion in edk2-platforms instead? > Silicon/Synopsys I guess? Sure. I'll rework it into edk2-platforms. > - Could you submit Usb2 and Usb3 support in a single set for v2? No problem. > - Can you convert these to UEFI driver model with > NonDiscoverableDeviceRegistrationLib instead of a hard-coded base > address? > The only problem is related on Usb Device. Could I append a new type "UsbDevice" into NonDiscoverableDeviceRegistrationLib? Best Regards Haojian ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-10-22 2:51 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-08-20 10:31 [PATCH v1 0/2] add DwUsb3Dxe driver Haojian Zhuang 2018-08-20 10:31 ` [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol Haojian Zhuang 2018-10-04 14:49 ` Leif Lindholm 2018-10-22 2:39 ` Haojian Zhuang 2018-08-20 10:31 ` [PATCH v1 2/2] EmbeddedPkg/Drivers: add DwUsb3Dxe driver Haojian Zhuang 2018-10-04 16:32 ` Leif Lindholm 2018-10-22 2:51 ` Haojian Zhuang 2018-10-04 14:15 ` [PATCH v1 0/2] " Leif Lindholm 2018-10-22 2:09 ` Haojian Zhuang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox