[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/8] hw/arm/virt: Fix devicetree warning about the gpio-ke
From: |
Eric Auger |
Subject: |
Re: [PATCH v2 4/8] hw/arm/virt: Fix devicetree warning about the gpio-key node |
Date: |
Tue, 27 Sep 2022 14:48:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 |
On 9/27/22 12:03, Jean-Philippe Brucker 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-philippe@linaro.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> hw/arm/virt.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index 8605f5058a..6805c57530 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -932,12 +932,12 @@ static void create_gpio_keys(char *fdt, DeviceState
> *pl061_dev,
> qemu_fdt_add_subnode(fdt, "/gpio-keys");
> qemu_fdt_setprop_string(fdt, "/gpio-keys", "compatible", "gpio-keys");
>
> - qemu_fdt_add_subnode(fdt, "/gpio-keys/poweroff");
> - qemu_fdt_setprop_string(fdt, "/gpio-keys/poweroff",
> + qemu_fdt_add_subnode(fdt, "/gpio-keys/key-poweroff");
> + qemu_fdt_setprop_string(fdt, "/gpio-keys/key-poweroff",
> "label", "GPIO Key Poweroff");
> - qemu_fdt_setprop_cell(fdt, "/gpio-keys/poweroff", "linux,code",
> + qemu_fdt_setprop_cell(fdt, "/gpio-keys/key-poweroff", "linux,code",
> KEY_POWER);
> - qemu_fdt_setprop_cells(fdt, "/gpio-keys/poweroff",
> + qemu_fdt_setprop_cells(fdt, "/gpio-keys/key-poweroff",
> "gpios", phandle, 3, 0);
> }
>
- Re: [PATCH v2 6/8] hw/arm/virt: Fix devicetree warning about the SMMU node, (continued)
- [PATCH v2 3/8] hw/arm/virt: Use "msi-map" devicetree property for PCI, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 2/8] hw/arm/virt: Fix devicetree warning about the GIC node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 4/8] hw/arm/virt: Fix devicetree warning about the gpio-key node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 1/8] hw/arm/virt: Fix devicetree warning about the root node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 7/8] hw/arm/virt: Fix devicetree warnings about the virtio-iommu node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 8/8] hw/arm/virt: Fix devicetree warnings about node names, Jean-Philippe Brucker, 2022/09/27
- Re: [PATCH v2 0/8] hw/arm/virt: Fix dt-schema warnings, Peter Maydell, 2022/09/29