[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/21] hw/i386/pc_sysfw: Use qdev_is_realized() instead of QO
From: |
Zhao Liu |
Subject: |
Re: [PATCH 02/21] hw/i386/pc_sysfw: Use qdev_is_realized() instead of QOM API |
Date: |
Thu, 22 Feb 2024 17:16:35 +0800 |
On Fri, Feb 16, 2024 at 12:02:53PM +0100, Philippe Mathieu-Daudé wrote:
> Date: Fri, 16 Feb 2024 12:02:53 +0100
> From: Philippe Mathieu-Daudé <philmd@linaro.org>
> Subject: [PATCH 02/21] hw/i386/pc_sysfw: Use qdev_is_realized() instead of
> QOM API
> X-Mailer: git-send-email 2.41.0
>
> Prefer QDev API for QDev objects, avoid the underlying QOM layer.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> hw/i386/pc_sysfw.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
>
> diff --git a/hw/i386/pc_sysfw.c b/hw/i386/pc_sysfw.c
> index c8d9e71b88..3efabbbab2 100644
> --- a/hw/i386/pc_sysfw.c
> +++ b/hw/i386/pc_sysfw.c
> @@ -107,17 +107,15 @@ void pc_system_flash_cleanup_unused(PCMachineState
> *pcms)
> {
> char *prop_name;
> int i;
> - Object *dev_obj;
>
> assert(PC_MACHINE_GET_CLASS(pcms)->pci_enabled);
>
> for (i = 0; i < ARRAY_SIZE(pcms->flash); i++) {
> - dev_obj = OBJECT(pcms->flash[i]);
> - if (!object_property_get_bool(dev_obj, "realized", &error_abort)) {
> + if (!qdev_is_realized(DEVICE(pcms->flash[i]))) {
> prop_name = g_strdup_printf("pflash%d", i);
> object_property_del(OBJECT(pcms), prop_name);
> g_free(prop_name);
> - object_unparent(dev_obj);
> + object_unparent(OBJECT(pcms->flash[i]));
> pcms->flash[i] = NULL;
> }
> }
> --
> 2.41.0
>
>
- [PATCH 00/21] hw: More QDev cleanups, Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 01/21] hw/i386/pc: Do not use C99 mixed-declarations style, Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 02/21] hw/i386/pc_sysfw: Use qdev_is_realized() instead of QOM API, Philippe Mathieu-Daudé, 2024/02/16
- Re: [PATCH 02/21] hw/i386/pc_sysfw: Use qdev_is_realized() instead of QOM API,
Zhao Liu <=
- [PATCH 03/21] hw/ppc/spapr_cpu: Use qdev_is_realized() instead of QOM API, Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 04/21] hw/tricore/testboard: Use qdev_new() instead of QOM basic API, Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 05/21] hw/ppc/pnv_bmc: Use qdev_new() instead of QOM API, Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 06/21] hw: Replace DEVICE(object_new) -> qdev_new(), Philippe Mathieu-Daudé, 2024/02/16
- [PATCH 07/21] target: Replace DEVICE(object_new) -> qdev_new(), Philippe Mathieu-Daudé, 2024/02/16