| Message ID | 20240113-pmi632-typec-v2-2-182d9aa0a5b3@linaro.org |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | usb: typec: qcom-pmic-typec: enable support for PMI632 PMIC | expand |
On 13/01/2024 21:55, Dmitry Baryshkov wrote: > The PMI632 PMIC has the same Type-C register block as the PM8150B. > However this PMIC doesn't support USB Power Delivery. As such it doesn't > have the second region used by the existing pm8150b bindings. Add if > clauses to handle the PMI632 usecase. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> ... > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - qcom,pmi632-typec > + then: > + properties: > + interrupts: > + maxItems: 8 > + interrupt-names: > + maxItems: 8 > + vdd-pdphy-supply: false > + else: > + properties: > + interrupts: > + minItems: 16 > + interrupt-names: > + maxItems: 16 In both if:else: narrow also "reg". Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/usb/qcom,pmic-typec.yaml b/Documentation/devicetree/bindings/usb/qcom,pmic-typec.yaml index 55df3129a0bc..e9f44750d580 100644 --- a/Documentation/devicetree/bindings/usb/qcom,pmic-typec.yaml +++ b/Documentation/devicetree/bindings/usb/qcom,pmic-typec.yaml @@ -15,6 +15,7 @@ description: properties: compatible: enum: + - qcom,pmi632-typec - qcom,pm8150b-typec connector: @@ -24,9 +25,11 @@ properties: reg: description: Type-C port and pdphy SPMI register base offsets + minItems: 1 maxItems: 2 interrupts: + minItems: 8 items: - description: Type-C CC attach notification, VBUS error, tCCDebounce done - description: Type-C VCONN powered @@ -46,6 +49,7 @@ properties: - description: Power Domain Fast Role Swap event interrupt-names: + minItems: 8 items: - const: or-rid-detect-change - const: vpd-detect @@ -81,7 +85,29 @@ required: - interrupts - interrupt-names - vdd-vbus-supply - - vdd-pdphy-supply + +allOf: + - if: + properties: + compatible: + contains: + enum: + - qcom,pmi632-typec + then: + properties: + interrupts: + maxItems: 8 + interrupt-names: + maxItems: 8 + vdd-pdphy-supply: false + else: + properties: + interrupts: + minItems: 16 + interrupt-names: + maxItems: 16 + required: + - vdd-pdphy-supply additionalProperties: false
The PMI632 PMIC has the same Type-C register block as the PM8150B. However this PMIC doesn't support USB Power Delivery. As such it doesn't have the second region used by the existing pm8150b bindings. Add if clauses to handle the PMI632 usecase. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- .../devicetree/bindings/usb/qcom,pmic-typec.yaml | 28 +++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-)