On Wed, 18 Sep 2024 09:42:05 +0800 Bibo Mao <maob...@loongson.cn> wrote:
> Macro definition is added for acpi sleep control register, ged emulation > driver can use the macro , also it can be used in FDT table if ged is > exposed with FDT table. > > Signed-off-by: Bibo Mao <maob...@loongson.cn> Reviewed-by: Igor Mammedov <imamm...@redhat.com> > --- > hw/acpi/generic_event_device.c | 6 +++--- > include/hw/acpi/generic_event_device.h | 7 +++++-- > 2 files changed, 8 insertions(+), 5 deletions(-) > > diff --git a/hw/acpi/generic_event_device.c b/hw/acpi/generic_event_device.c > index 15b4c3ebbf..d00f5a6c1c 100644 > --- a/hw/acpi/generic_event_device.c > +++ b/hw/acpi/generic_event_device.c > @@ -201,9 +201,9 @@ static void ged_regs_write(void *opaque, hwaddr addr, > uint64_t data, > > switch (addr) { > case ACPI_GED_REG_SLEEP_CTL: > - slp_typ = (data >> 2) & 0x07; > - slp_en = (data >> 5) & 0x01; > - if (slp_en && slp_typ == 5) { > + slp_typ = (data >> ACPI_GED_SLP_TYP_POS) & ACPI_GED_SLP_TYP_MASK; > + slp_en = !!(data & ACPI_GED_SLP_EN); > + if (slp_en && slp_typ == ACPI_GED_SLP_TYP_S5) { > qemu_system_shutdown_request(SHUTDOWN_CAUSE_GUEST_SHUTDOWN); > } > return; > diff --git a/include/hw/acpi/generic_event_device.h > b/include/hw/acpi/generic_event_device.h > index 40af3550b5..d2dac87b4a 100644 > --- a/include/hw/acpi/generic_event_device.h > +++ b/include/hw/acpi/generic_event_device.h > @@ -81,8 +81,11 @@ OBJECT_DECLARE_SIMPLE_TYPE(AcpiGedState, ACPI_GED) > /* ACPI_GED_REG_RESET value for reset*/ > #define ACPI_GED_RESET_VALUE 0x42 > > -/* ACPI_GED_REG_SLEEP_CTL.SLP_TYP value for S5 (aka poweroff) */ > -#define ACPI_GED_SLP_TYP_S5 0x05 > +/* [ACPI 5.0 Chapter 4.8.3.7] Sleep Control and Status Register */ > +#define ACPI_GED_SLP_TYP_POS 0x2 /* SLP_TYPx Bit Offset */ > +#define ACPI_GED_SLP_TYP_MASK 0x07 /* SLP_TYPx 3-bit mask */ > +#define ACPI_GED_SLP_TYP_S5 0x05 /* System _S5 State (Soft Off) */ > +#define ACPI_GED_SLP_EN 0x20 /* SLP_EN write-only bit */ > > #define GED_DEVICE "GED" > #define AML_GED_EVT_REG "EREG"