diff mbox series

dt-bindings: usb: qcom,dwc3: document extcon property

Message ID 20230712235902.98557-1-david@ixit.cz (mailing list archive)
State Not Applicable
Headers show
Series dt-bindings: usb: qcom,dwc3: document extcon property | expand

Commit Message

David Heidelberg July 12, 2023, 11:59 p.m. UTC
The driver has support for it already. Only missing bit is dt-binding entry.

Signed-off-by: David Heidelberg <david@ixit.cz>
---
 Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Krzysztof Kozlowski July 13, 2023, 6:37 a.m. UTC | #1
On 13/07/2023 01:59, David Heidelberg wrote:
> The driver has support for it already. Only missing bit is dt-binding entry.

No, because the child node already has it. We do not need to extcons for
one USB. Also another reason is that it should be deprecated.

Best regards,
Krzysztof
Bjorn Andersson July 14, 2023, 3:20 a.m. UTC | #2
On Thu, Jul 13, 2023 at 08:37:55AM +0200, Krzysztof Kozlowski wrote:
> On 13/07/2023 01:59, David Heidelberg wrote:
> > The driver has support for it already. Only missing bit is dt-binding entry.
> 
> No, because the child node already has it. We do not need to extcons for
> one USB. Also another reason is that it should be deprecated.
> 

In the current design of the Linux dwc3 driver we actually need both
nodes to be extcon targets, as both device drivers needs to react to the
role change. So I think the patch accurately describes the intended
design of today.

That said, we do not want to do this for the Type-C case, where we use a
full port node. Combined with other reasons, we really should fix the
dwc3 driver model so that the core propagates the role changes to the
glue drivers, and as such I agree that we shouldn't pick this patch.

Regards,
Bjorn
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
index 4875c5b7d5b5..7099b583d21c 100644
--- a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
+++ b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
@@ -107,6 +107,10 @@  properties:
     minItems: 1
     maxItems: 4
 
+  extcon:
+    description:
+      Phandle to the extcon device providing the cable state.
+
   qcom,select-utmi-as-pipe-clk:
     description:
       If present, disable USB3 pipe_clk requirement.