Re: [PATCH v6 15/33] hw/isa/piix3: Create power management controller in host device

[Philippe Mathieu-Daudé]

On 9/1/23 18:23, Bernhard Beschow wrote:
> The power management controller is an integral part of PIIX3 (function
> 3). So create it as part of the south bridge.
>
> Note that the ACPI function is optional in QEMU. This is why it gets
> object_initialize_child()'ed in realize rather than in instance_init.
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
> Message-Id: <20221022150508.26830-14-shentey@gmail.com>
> ---
>   include/hw/southbridge/piix.h |  6 ++++++
>   hw/i386/pc_piix.c             | 24 ++++++++++++++----------
>   hw/isa/piix3.c                | 14 ++++++++++++++
>   hw/isa/Kconfig                |  1 +
>   4 files changed, 35 insertions(+), 10 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>