Skip to content

Commit 96ecc71

Browse files
andredlag-linaro
authored andcommitted
dt-bindings: mfd: samsung,s2mps11: Add comment about interrupts properties
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 <[email protected]> Reviewed-by: Krzysztof Kozlowski <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Lee Jones <[email protected]>
1 parent c371040 commit 96ecc71

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

Documentation/devicetree/bindings/mfd/samsung,s2mps11.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,9 @@ allOf:
8181
samsung,s2mps11-acokb-ground: false
8282
samsung,s2mps11-wrstbi-ground: false
8383

84+
# oneOf is required, because dtschema's fixups.py doesn't handle this
85+
# nesting here. Its special treatment to allow either interrupt property
86+
# when only one is specified in the binding works at the top level only.
8487
oneOf:
8588
- required: [interrupts]
8689
- required: [interrupts-extended]

0 commit comments

Comments
 (0)