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

Krzysztof Kozlowski krzysztof.kozlowski at linaro.org
Wed Jul 20 10:14:13 PDT 2022


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



More information about the Linux-mediatek mailing list