On Tue, 27 Sept 2022 at 11:12, Jean-Philippe Brucker <jean-phili...@linaro.org> wrote: > The node name of the gpio-key devicetree node should be "key-poweroff": > > gpio-keys: 'poweroff' does not match any of the regexes: > '^(button|event|key|switch|(button|event|key|switch)-[a-z0-9-]+|[a-z0-9-]+-(button|event|key|switch))$', > 'pinctrl-[0-9]+' > From schema: linux/Documentation/devicetree/bindings/input/gpio-keys.yaml > > Signed-off-by: Jean-Philippe Brucker <jean-phili...@linaro.org>
This restriction only went into the DT documentation in July (kernel commit 5eb5652250). Please don't retrospectively make perfectly valid working DTs non-valid. I don't see any reason to change QEMU here. More generally, the set of things you might want the validator to warn about for a fresh new human-written DTB doesn't necessarily correspond to the set of things you want to enforce for a pre-existing code-generated DTB. For the former it makes much more sense to impose "coding style" and "naming convention" type rules. thanks -- PMM