mbox series

[v2,0/4] usb: host: add TPL support to ehci ohci and dwc2

Message ID 20220621152350.145745-1-fabrice.gasnier@foss.st.com
Headers show
Series usb: host: add TPL support to ehci ohci and dwc2 | expand

Message

Fabrice Gasnier June 21, 2022, 3:23 p.m. UTC 
The Target Peripheral List (TPL) support is used to identify targeted
devices during Embedded Host (EH) compliance testing. Add TPL support
setting from device tree on ehci-platform, ohci-platform and dwc2 drivers.
Also document TPL support in DWC2 dt-bindings.

Changes in v2:
- Document TPL support in DWC2 dt-bindings
- Spell TPL and EH
- Add collected ack

Amelie Delaunay (3):
  usb: host: ohci-platform: add TPL support
  usb: host: ehci-platform: add TPL support
  usb: dwc2: host: add TPL support

Fabrice Gasnier (1):
  dt-bindings: usb: dwc2: document TPL support

 Documentation/devicetree/bindings/usb/dwc2.yaml | 3 +++
 drivers/usb/dwc2/hcd.c                          | 3 +++
 drivers/usb/host/ehci-platform.c                | 2 ++
 drivers/usb/host/ohci-platform.c                | 3 +++
 4 files changed, 11 insertions(+)

Comments

Minas Harutyunyan June 22, 2022, 5:32 a.m. UTC | #1 
On 6/21/2022 7:23 PM, Fabrice Gasnier wrote:
> From: Amelie Delaunay <amelie.delaunay@foss.st.com>
> 
> The Target Peripheral List (TPL) is used to identify targeted devices
> during Embedded Host compliance testing. The user can add "tpl-support"
> in the device tree to enable it.
> 
> Signed-off-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>

Acked-by: Minas Harutyunyan <hminas@synopsys.com>

> ---
> Changes in v2:
> - added dt-bindings precursor patch
> - update commit message to clarify TPL and EH meaning
> ---
>   drivers/usb/dwc2/hcd.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/usb/dwc2/hcd.c b/drivers/usb/dwc2/hcd.c
> index f63a27d11fac8..4567f3c24d225 100644
> --- a/drivers/usb/dwc2/hcd.c
> +++ b/drivers/usb/dwc2/hcd.c
> @@ -52,6 +52,7 @@
>   
>   #include <linux/usb/hcd.h>
>   #include <linux/usb/ch11.h>
> +#include <linux/usb/of.h>
>   
>   #include "core.h"
>   #include "hcd.h"
> @@ -5339,6 +5340,8 @@ int dwc2_hcd_init(struct dwc2_hsotg *hsotg)
>   	/* Don't support SG list at this point */
>   	hcd->self.sg_tablesize = 0;
>   
> +	hcd->tpl_support = of_usb_host_tpl_support(hsotg->dev->of_node);
> +
>   	if (!IS_ERR_OR_NULL(hsotg->uphy))
>   		otg_set_host(hsotg->uphy->otg, &hcd->self);
>