On 2021-10-04 22:07, Rob Herring wrote:
On Mon, Oct 04, 2021 at 04:46:19PM +0530, Souradeep Chowdhury wrote:
Added the property for EUD(Embedded USB Debug) connector.Added
the "reg" and "interrupts" property which is needed for EUD.
You are going to need a better explanation of this h/w.
Ack. Will update this with the detailed hardware description
in the next version.
Signed-off-by: Souradeep Chowdhury <schowdhu@xxxxxxxxxxxxxx>
---
.../devicetree/bindings/connector/usb-connector.yaml | 15
+++++++++++++++
1 file changed, 15 insertions(+)
diff --git
a/Documentation/devicetree/bindings/connector/usb-connector.yaml
b/Documentation/devicetree/bindings/connector/usb-connector.yaml
index 7eb8659..908129f 100644
--- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
+++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
@@ -30,6 +30,21 @@ properties:
- const: samsung,usb-connector-11pin
- const: usb-b-connector
+ - items:
+ - enum:
+ - qcom,sc7280-usb-connector-eud
+ - const: qcom,usb-connector-eud
+ - const: usb-c-connector
+
+ reg:
+ items:
+ - description: EUD Base Register Region
+ - description: EUD Mode Manager Region
A connector node represents the physical connector on a board. That
can't really be an MMIO peripheral. Maybe you need a node for EUD and
then it should have a connector child node? Don't really know without
understanding this h/w.
As per the previous discussion on the EUD, it was agreed upon to map EUD
as a type C connector and use Role-Switch to change the USB role instead
of extcon interface that was being used previously. The link for the
same
is as follows:-
https://lore.kernel.org/lkml/5db1a666-62ec-c850-6626-ad33d337b452@xxxxxxxxxxxxxx/
+
+ interrupts:
+ description:
+ EUD interrupt
+
label:
description: Symbolic name for the connector.
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member
of Code Aurora Forum, hosted by The Linux Foundation
