Messages in this thread | | | Date | Wed, 20 Jul 2022 19:14:13 +0200 | Subject | Re: [PATCH v1 2/6] dt-bindings: mediatek,mt6779-keypad: use unevaluatedProperties | From | Krzysztof Kozlowski <> |
| |
On 20/07/2022 16:48, Mattijs Korpershoek wrote: > writing-bindings.rst states: >> - 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
| |