[PATCH v3 1/3] dt-bindings: arm: convert nested if-else construct to allOf
From: Nikunj Kela <hidden>
Date: 2023-08-11 17:57:54
Also in:
linux-arm-kernel, linux-arm-msm, lkml
Subsystem:
open firmware and flattened device tree bindings, system control & power/management interface (scpi/scmi) message protocol drivers, the rest · Maintainers:
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sudeep Holla, Linus Torvalds
Nested if-else construct is not scalable therefore, convert it to allOf:if-else. Signed-off-by: Nikunj Kela <redacted> Suggested-by: Krzysztof Kozlowski <redacted> Reviewed-by: Krzysztof Kozlowski <redacted> --- .../bindings/firmware/arm,scmi.yaml | 55 +++++++++---------- 1 file changed, 27 insertions(+), 28 deletions(-)
diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
index b138f3d23df8..8d54ea768d38 100644
--- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
+++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml@@ -284,35 +284,34 @@ $defs: required: - compatible -if: - properties: - compatible: - contains: - const: arm,scmi -then: - properties: - interrupts: false - interrupt-names: false - - required: - - mboxes - - shmem - -else: - if: - properties: - compatible: - contains: - enum: - - arm,scmi-smc - - arm,scmi-smc-param - then: - required: - - arm,smc-id - - shmem +allOf: + - if: + properties: + compatible: + contains: + const: arm,scmi + then: + properties: + interrupts: false + interrupt-names: false + + required: + - mboxes + - shmem + + - if: + properties: + compatible: + contains: + enum: + - arm,scmi-smc + - arm,scmi-smc-param + then: + required: + - arm,smc-id + - shmem - else: - if: + - if: properties: compatible: contains:
--
2.17.1