On Fri, Aug 04, 2023 at 03:50:07PM +0200, Neil Armstrong wrote: > On SM8450 and SM8550 based platforms, the Type-C plug orientation is given on a > GPIO line set by the PMIC. > > Document this optional Type-C connector property, and take the > assumption an active level represents an inverted/flipped orientation. > > Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx> > --- > .../devicetree/bindings/soc/qcom/qcom,pmic-glink.yaml | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,pmic-glink.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,pmic-glink.yaml > index bceb479f74c5..1b0a00b19a54 100644 > --- a/Documentation/devicetree/bindings/soc/qcom/qcom,pmic-glink.yaml > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,pmic-glink.yaml > @@ -35,6 +35,11 @@ properties: > '#size-cells': > const: 0 > > + orientation-gpios: > + description: An input gpio for Type-C connector orientation, used to detect orientation > + of the Type-C connector. GPIO active level means "CC2" or Reversed/Flipped orientation. This is a per-connector/port property, as such be part of the connector sub nodes rather than as a property of the shared pmic_glink entity. Regards, Bjorn > + maxItems: 1 > + > patternProperties: > '^connector@\d$': > $ref: /schemas/connector/usb-connector.yaml# > @@ -44,6 +49,19 @@ patternProperties: > required: > - compatible > > +allOf: > + - if: > + not: > + properties: > + compatible: > + contains: > + enum: > + - qcom,sm8450-pmic-glink > + - qcom,sm8550-pmic-glink > + then: > + properties: > + orientation-gpios: false > + > additionalProperties: false > > examples: > > -- > 2.34.1 >