From: Leif Lindholm <leif.lindholm@linaro.org>
To: Haojian Zhuang <haojian.zhuang@linaro.org>
Cc: edk2-devel@lists.01.org, Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: Re: [PATCH v1 1/2] EmbeddedPkg: add DwUsb protocol
Date: Thu, 4 Oct 2018 15:49:04 +0100 [thread overview]
Message-ID: <20181004144904.sdanz2vstp4hwhwk@bivouac.eciton.net> (raw)
In-Reply-To: <1534761085-26972-2-git-send-email-haojian.zhuang@linaro.org>
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
>
next prev parent reply other threads:[~2018-10-04 14:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181004144904.sdanz2vstp4hwhwk@bivouac.eciton.net \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox