[PATCH 3/3] dt-bindings: clock: qcom,sm8450-video: Require required-opps on X1E80100
From: Krzysztof Kozlowski
Date: Wed Jul 08 2026 - 05:50:19 EST
Reverse the logic within "if:then:" when expecting "required-opps"
property to have negative list of only variants which do not require
these. VIDEOCC on SM8650 does not need "required-opps" as explained in
the commit 6720e8dbcb1b ("dt-bindings: clock: qcom: Drop required-opps
in required on sm8650 videocc"). Requirement on VIDEOCC on SM8475 is
not known.
Reverting the logic in "if:" part has two impacts:
1. Adds requirement of "required-opps" on VIDEOCC on Hamoa (X1E80100),
to match hardware expectation, similarly to existing its variant
X1P42100 (already expressed in DTS).
2. Reduces the chances of forgetting to update constraints when adding
new compatibles (like for X1E80100).
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
---
.../devicetree/bindings/clock/qcom,sm8450-videocc.yaml | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml
index 5d77029bfaf8..be5105569cfc 100644
--- a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml
@@ -65,14 +65,11 @@ allOf:
- if:
properties:
compatible:
- contains:
- enum:
- - qcom,glymur-videocc
- - qcom,kaanapali-videocc
- - qcom,sm8450-videocc
- - qcom,sm8550-videocc
- - qcom,sm8750-videocc
- - qcom,x1p42100-videocc
+ not:
+ contains:
+ enum:
+ - qcom,sm8475-videocc
+ - qcom,sm8650-videocc
then:
required:
- required-opps
--
2.53.0