[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/12] hw/acpi/piix4: convert smm_enabled bool to qdev proper
From: |
Ani Sinha |
Subject: |
Re: [PATCH 03/12] hw/acpi/piix4: convert smm_enabled bool to qdev property |
Date: |
Mon, 30 May 2022 10:48:56 +0530 |
On Sat, May 28, 2022 at 2:49 PM Mark Cave-Ayland
<mark.cave-ayland@ilande.co.uk> wrote:
>
> This allows the smm_enabled value to be set using a standard qdev property
> instead
> of being referenced directly in piix4_pm_init().
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Ani Sinha <ani@anisinha.ca>
> ---
> hw/acpi/piix4.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
> index fcfaafc175..2735ff375e 100644
> --- a/hw/acpi/piix4.c
> +++ b/hw/acpi/piix4.c
> @@ -547,6 +547,7 @@ I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t
> smb_io_base,
> pci_dev = pci_new(devfn, TYPE_PIIX4_PM);
> dev = DEVICE(pci_dev);
> qdev_prop_set_uint32(dev, "smb_io_base", smb_io_base);
> + qdev_prop_set_bit(dev, "smm-enabled", smm_enabled);
> if (piix4_pm) {
> *piix4_pm = dev;
> }
> @@ -554,7 +555,6 @@ I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t
> smb_io_base,
> s = PIIX4_PM(dev);
> s->irq = sci_irq;
> s->smi_irq = smi_irq;
> - s->smm_enabled = smm_enabled;
>
> pci_realize_and_unref(pci_dev, bus, &error_fatal);
>
> @@ -664,6 +664,7 @@ static Property piix4_pm_properties[] = {
> DEFINE_PROP_BOOL("memory-hotplug-support", PIIX4PMState,
> acpi_memory_hotplug.is_enabled, true),
> DEFINE_PROP_BOOL("smm-compat", PIIX4PMState, smm_compat, false),
> + DEFINE_PROP_BOOL("smm-enabled", PIIX4PMState, smm_enabled, false),
> DEFINE_PROP_BOOL("x-not-migrate-acpi-index", PIIX4PMState,
> not_migrate_acpi_index, false),
> DEFINE_PROP_END_OF_LIST(),
> --
> 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
- [PATCH 03/12] hw/acpi/piix4: convert smm_enabled bool to qdev property, Mark Cave-Ayland, 2022/05/28
- Re: [PATCH 03/12] hw/acpi/piix4: convert smm_enabled bool to qdev property,
Ani Sinha <=
- [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
- [PATCH 05/12] hw/acpi/piix4: alter piix4_pm_init() to return PIIX4PMState, Mark Cave-Ayland, 2022/05/28