[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v3 8/8] hw/arm/virt: Add gpio-keys node for Powero
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH v3 8/8] hw/arm/virt: Add gpio-keys node for Poweroff using DT |
Date: |
Fri, 27 Nov 2015 17:16:53 +0000 |
On 16 November 2015 at 13:23, <address@hidden> wrote:
> From: Shannon Zhao <address@hidden>
>
> Add a gpio-keys node. This is used for Poweroff for the systems which
> use DT not ACPI.
>
> Signed-off-by: Shannon Zhao <address@hidden>
> Signed-off-by: Shannon Zhao <address@hidden>
> Tested-by: Wei Huang <address@hidden>
> ---
> hw/arm/virt.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index cb1d45b..3394262 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -561,6 +561,7 @@ static void create_gpio(const VirtBoardInfo *vbi,
> qemu_irq *pic)
>
> pl061_dev = sysbus_create_simple("pl061", base, pic[irq]);
>
> + uint32_t phandle = qemu_fdt_alloc_phandle(vbi->fdt);
> nodename = g_strdup_printf("/address@hidden" PRIx64, base);
> qemu_fdt_add_subnode(vbi->fdt, nodename);
> qemu_fdt_setprop_sized_cells(vbi->fdt, nodename, "reg",
> @@ -573,6 +574,20 @@ static void create_gpio(const VirtBoardInfo *vbi,
> qemu_irq *pic)
> GIC_FDT_IRQ_FLAGS_LEVEL_HI);
> qemu_fdt_setprop_cell(vbi->fdt, nodename, "clocks", vbi->clock_phandle);
> qemu_fdt_setprop_string(vbi->fdt, nodename, "clock-names", "apb_pclk");
> + qemu_fdt_setprop_cell(vbi->fdt, nodename, "phandle", phandle);
> +
> + qemu_fdt_add_subnode(vbi->fdt, "/gpio-keys");
> + qemu_fdt_setprop_string(vbi->fdt, "/gpio-keys", "compatible",
> "gpio-keys");
> + qemu_fdt_setprop_cell(vbi->fdt, "/gpio-keys", "#size-cells", 0);
> + qemu_fdt_setprop_cell(vbi->fdt, "/gpio-keys", "#address-cells", 1);
> + qemu_fdt_setprop(vbi->fdt, "/gpio-keys", "autorepeat", NULL, 0);
Why would we want the power button to autorepeat ?
> +
> + qemu_fdt_add_subnode(vbi->fdt, "/gpio-keys/poweroff");
> + qemu_fdt_setprop_string(vbi->fdt, "/gpio-keys/poweroff",
> + "label", "GPIO Key Poweroff");
> + qemu_fdt_setprop_cell(vbi->fdt, "/gpio-keys/poweroff", "linux,code",
> 116);
This could use a comment /* KEY_POWER */ to indicate what 116 is.
(Alternatively, see whether we can #include "standard-headers/linux/input.h"
which #defines KEY_POWER.)
> + qemu_fdt_setprop_cells(vbi->fdt, "/gpio-keys/poweroff",
> + "gpios", phandle, 3, 0);
>
> /* connect powerdown request */
> qemu_register_powerdown_notifier(&virt_system_powerdown_notifier);
thanks
-- PMM
- [Qemu-arm] [PATCH v3 0/8] Add system_powerdown support on ARM for ACPI and DT, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 7/8] hw/arm/virt: Add QEMU powerdown notifier and hook it to GPIO Pin 3, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 3/8] hw/arm/virt-acpi-build: Add power button device in ACPI DSDT table, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 1/8] hw/arm/virt: Add a GPIO controller, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 2/8] hw/arm/virt-acpi-build: Add GPIO controller in ACPI DSDT table, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 4/8] hw/acpi/aml-build: Add GPIO Connection Descriptor, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 5/8] hw/acpi/aml-build: Add a wrapper for GPIO Interrupt Connection, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 6/8] hw/arm/virt-acpi-build: Add _E03 for Power Button, shannon . zhao, 2015/11/16
- [Qemu-arm] [PATCH v3 8/8] hw/arm/virt: Add gpio-keys node for Poweroff using DT, shannon . zhao, 2015/11/16
- Re: [Qemu-arm] [PATCH v3 8/8] hw/arm/virt: Add gpio-keys node for Poweroff using DT,
Peter Maydell <=
- Re: [Qemu-arm] [PATCH v3 0/8] Add system_powerdown support on ARM for ACPI and DT, Shannon Zhao, 2015/11/16