Thread (18 messages) 18 messages, 4 authors, 2022-05-20

Re: [PATCH 3/3] ARM: dts: lan966x: Add UDPHS support

From: Herve Codina <herve.codina@bootlin.com>
Date: 2022-05-20 12:37:40
Also in: linux-arm-kernel, linux-clk, linux-usb, lkml

Hi Krzysztof, Sergei

On Fri, 13 May 2022 14:54:26 +0200
Krzysztof Kozlowski [off-list ref] wrote:
On 13/05/2022 12:58, Herve Codina wrote:
quoted
Add UDPHS (the USB High Speed Device Port controller) support.
The UDPHS IP present in the lan966x SOC is the same as the one
present in the SAMA5D3 SOC

Signed-off-by: Herve Codina <herve.codina@bootlin.com>
---
 arch/arm/boot/dts/lan966x.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)
diff --git a/arch/arm/boot/dts/lan966x.dtsi b/arch/arm/boot/dts/lan966x.dtsi
index 7d2869648050..4c09f3166d27 100644
--- a/arch/arm/boot/dts/lan966x.dtsi
+++ b/arch/arm/boot/dts/lan966x.dtsi
@@ -211,6 +211,17 @@ can0: can@e081c000 {
 			status = "disabled";
 		};
 
+		udc: udphs@e0808000 {  
Generic node names, so it looks like usb. For example HCD schema
requires it. I am not sure which bindings are used here, but anyway once
they might require usb...
HCD are related to the Host controller.
Here we are talking about a device.

In existing bindings related to USB device (or OTG as an OTG can be a
host or a device) on several SOCs, we can find:
- usb1: gadget@fffa4000
- usb_otg: usb@1c13000
- usb: usb@47400000
- udc: usb@13040000
- usb_otg_hs: usb_otg_hs@4a0ab000


So I will change to
  udc: usb@e0808000

Is that ok for you ?

Regards,
Herve

-- 
Hervé Codina, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help