mbox series

[v2,0/2] AM64: Add USB support

Message ID 20210310112758.1982-1-a-govindraju@ti.com
Headers show
Series AM64: Add USB support | expand

Message

Aswath Govindraju March 10, 2021, 11:27 a.m. UTC
The following series of patches, add USB support for AM642 evm.

USB test logs,
https://pastebin.ubuntu.com/p/Qrbb7SGv8N/

Changes since v1:
- Rebased the patches on top of ti-k3-dts-next
- Added test logs

Aswath Govindraju (2):
  arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem
  arm64: dts: ti: k3-am642-evm: Add USB support

 arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++
 arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++
 2 files changed, 48 insertions(+)

Comments

Kishon Vijay Abraham I March 15, 2021, 5:54 a.m. UTC | #1
On 10/03/21 4:57 pm, Aswath Govindraju wrote:
> The following series of patches, add USB support for AM642 evm.

> 

> USB test logs,

> https://pastebin.ubuntu.com/p/Qrbb7SGv8N/

> 

> Changes since v1:

> - Rebased the patches on top of ti-k3-dts-next

> - Added test logs


Would prefer simple data transfer tests in the log but other than that
for the patches itself

Reviewed-by: Kishon Vijay Abraham I <kishon@ti.com>

> 

> Aswath Govindraju (2):

>   arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem

>   arm64: dts: ti: k3-am642-evm: Add USB support

> 

>  arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++

>  arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++

>  2 files changed, 48 insertions(+)

>
Nishanth Menon March 16, 2021, 12:21 p.m. UTC | #2
On 16:57-20210310, Aswath Govindraju wrote:
> The following series of patches, add USB support for AM642 evm.

> 

> USB test logs,

> https://pastebin.ubuntu.com/p/Qrbb7SGv8N/

> 

> Changes since v1:

> - Rebased the patches on top of ti-k3-dts-next

> - Added test logs

> 

> Aswath Govindraju (2):

>   arm64: dts: ti: k3-am64-main: Add DT node for USB subsystem

>   arm64: dts: ti: k3-am642-evm: Add USB support

> 

>  arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 30 ++++++++++++++++++++++++

>  arch/arm64/boot/dts/ti/k3-am642-evm.dts  | 18 ++++++++++++++

>  2 files changed, 48 insertions(+)

> 

> -- 

> 2.17.1

> 


	I see:
	/workdir/arch/arm64/boot/dts/ti/k3-am642-evm.dt.yaml: cdns-usb@f900000: compatible: 'oneOf' conditional failed, one must be fixed:

I assume there is a reason for this? Can you capture that in the commit
message?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D