]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
dt-bindings: mfd: samsung,s2mps11: Add comment about interrupts properties
authorAndré Draszik <andre.draszik@linaro.org>
Fri, 27 Jun 2025 09:15:25 +0000 (10:15 +0100)
committerLee Jones <lee@kernel.org>
Thu, 24 Jul 2025 10:27:12 +0000 (11:27 +0100)
Document why the binding uses oneOf when specifying just one of the
interrupt properties is supposed to be enough.

dtschema's fixups.py has special treatment of the interrupts and
interrupts-extended properties, but that appears to work at the top
level only. Elsewhere, an explicit oneOf is required.

Signed-off-by: André Draszik <andre.draszik@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20250627-s2mpg10-binding-comment-v1-1-f37e5187f0fd@linaro.org
Signed-off-by: Lee Jones <lee@kernel.org>
Documentation/devicetree/bindings/mfd/samsung,s2mps11.yaml

index d6b9e29147965b6d8eef786b0fb5b5f198ab69ab..31d544a9c05cad878d10a0ae9b99631f08eb04a8 100644 (file)
@@ -81,6 +81,9 @@ allOf:
         samsung,s2mps11-acokb-ground: false
         samsung,s2mps11-wrstbi-ground: false
 
+      # oneOf is required, because dtschema's fixups.py doesn't handle this
+      # nesting here. Its special treatment to allow either interrupt property
+      # when only one is specified in the binding works at the top level only.
       oneOf:
         - required: [interrupts]
         - required: [interrupts-extended]