[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-9.0? 1/2] hw/misc/applesmc: Do not call DeviceReset() fro
From: |
Peter Maydell |
Subject: |
Re: [PATCH-for-9.0? 1/2] hw/misc/applesmc: Do not call DeviceReset() from DeviceRealize() |
Date: |
Mon, 8 Apr 2024 11:30:43 +0100 |
On Mon, 8 Apr 2024 at 10:52, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> QDev core layer always call DeviceReset() after DeviceRealize(),
> no need to do it manually. Remove the extra call.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> hw/misc/applesmc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/misc/applesmc.c b/hw/misc/applesmc.c
> index 72300d0cbc..8e65816da6 100644
> --- a/hw/misc/applesmc.c
> +++ b/hw/misc/applesmc.c
> @@ -342,7 +342,6 @@ static void applesmc_isa_realize(DeviceState *dev, Error
> **errp)
> }
>
> QLIST_INIT(&s->data_def);
> - qdev_applesmc_isa_reset(dev);
> }
>
> static Property applesmc_isa_properties[] = {
> --
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
thanks
-- PMM