Thread (23 messages) 23 messages, 4 authors, 2022-07-26

Re: [PATCH v1 2/6] dt-bindings: mediatek,mt6779-keypad: use unevaluatedProperties

From: Krzysztof Kozlowski <hidden>
Date: 2022-07-20 17:14:26
Also in: linux-arm-kernel, linux-devicetree, linux-mediatek, lkml

On 20/07/2022 16:48, Mattijs Korpershoek wrote:
writing-bindings.rst states:
quoted
- If schema includes other schema (e.g. /schemas/i2c/i2c-controller.yaml) use
  "unevaluatedProperties:false". In other cases, usually use
  "additionalProperties:false".
mt6779-keypad includes matrix-keymap.yaml so replace additionalProperties:false
by unevaluatedProperties:false.
This is not sufficient explanation. You now allow all properties from
matrix-keymap.yaml, which might be desired or might be not (e.g. they
are not valid for this device). Please investigate it and mention the
outcome.

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help