[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/12] hw/acpi/piix4: change smm_enabled from int to bool
From: |
Ani Sinha |
Subject: |
Re: [PATCH 02/12] hw/acpi/piix4: change smm_enabled from int to bool |
Date: |
Mon, 30 May 2022 10:26:03 +0530 |
On Sat, May 28, 2022 at 2:49 PM Mark Cave-Ayland
<mark.cave-ayland@ilande.co.uk> wrote:
>
> This is in preparation for conversion to a qdev property.
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
other than the comment below,
Reviewed-by: Ani Sinha <ani@anisinha.ca>
> ---
> hw/acpi/piix4.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
> index bf20fa139b..fcfaafc175 100644
> --- a/hw/acpi/piix4.c
> +++ b/hw/acpi/piix4.c
> @@ -74,7 +74,7 @@ struct PIIX4PMState {
>
> qemu_irq irq;
> qemu_irq smi_irq;
> - int smm_enabled;
> + bool smm_enabled;
For the sake of consistency, I would also change the signature of
piix4_pm_init(), that is, change simm_enabled from int to bool.
We are good in pc_init1 since x86_machine_is_smm_enabled() returns
bool. In piix4_create() in isa, we pass integer 0 which we might want
to make boolean.
> bool smm_compat;
> Notifier machine_ready;
> Notifier powerdown_notifier;
> --
> 2.20.1
>
- [PATCH 00/12] hw/acpi/piix4: remove legacy piix4_pm_init() function, Mark Cave-Ayland, 2022/05/28
- [PATCH 06/12] hw/acpi/piix4: rename piix4_pm_init() to piix4_pm_initfn(), Mark Cave-Ayland, 2022/05/28
- [PATCH 10/12] hw/i386/pc_piix: create PIIX4_PM device directly instead of using piix4_pm_initfn(), Mark Cave-Ayland, 2022/05/28
- [PATCH 12/12] hw/acpi/piix4: remove unused piix4_pm_initfn() function, Mark Cave-Ayland, 2022/05/28
- [PATCH 01/12] hw/acpi/piix4: move xen_enabled() logic from piix4_pm_init() to piix4_pm_realize(), Mark Cave-Ayland, 2022/05/28
- [PATCH 02/12] hw/acpi/piix4: change smm_enabled from int to bool, Mark Cave-Ayland, 2022/05/28
- Re: [PATCH 02/12] hw/acpi/piix4: change smm_enabled from int to bool,
Ani Sinha <=
- [PATCH 03/12] hw/acpi/piix4: convert smm_enabled bool to qdev property, Mark Cave-Ayland, 2022/05/28
- [PATCH 11/12] hw/isa/piix4.c: create PIIX4_PM device directly instead of using piix4_pm_initfn(), Mark Cave-Ayland, 2022/05/28
- [PATCH 08/12] hw/acpi/piix4: use qdev gpio to wire up sci_irq, Mark Cave-Ayland, 2022/05/28
- [PATCH 09/12] hw/acpi/piix4: use qdev gpio to wire up smi_irq, Mark Cave-Ayland, 2022/05/28
- [PATCH 07/12] hw/acpi/piix4: introduce piix4_pm_init() instance init function, Mark Cave-Ayland, 2022/05/28