[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 23/24] sifive_e: Register "revb" as class property
From: |
Alistair Francis |
Subject: |
Re: [PATCH 23/24] sifive_e: Register "revb" as class property |
Date: |
Tue, 22 Sep 2020 09:45:21 -0700 |
On Mon, Sep 21, 2020 at 3:30 PM Eduardo Habkost <ehabkost@redhat.com> wrote:
>
> Class properties make QOM introspection simpler and easier, as
> they don't require an object to be instantiated.
>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Alistair Francis <Alistair.Francis@wdc.com>
> Cc: Sagar Karandikar <sagark@eecs.berkeley.edu>
> Cc: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
> Cc: qemu-riscv@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
> hw/riscv/sifive_e.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/hw/riscv/sifive_e.c b/hw/riscv/sifive_e.c
> index 40bbf530d49..9fe0c12df64 100644
> --- a/hw/riscv/sifive_e.c
> +++ b/hw/riscv/sifive_e.c
> @@ -137,11 +137,6 @@ static void sifive_e_machine_instance_init(Object *obj)
> SiFiveEState *s = RISCV_E_MACHINE(obj);
>
> s->revb = false;
> - object_property_add_bool(obj, "revb", sifive_e_machine_get_revb,
> - sifive_e_machine_set_revb);
> - object_property_set_description(obj, "revb",
> - "Set on to tell QEMU that it should
> model "
> - "the revB HiFive1 board");
> }
>
> static void sifive_e_machine_class_init(ObjectClass *oc, void *data)
> @@ -152,6 +147,12 @@ static void sifive_e_machine_class_init(ObjectClass *oc,
> void *data)
> mc->init = sifive_e_machine_init;
> mc->max_cpus = 1;
> mc->default_cpu_type = SIFIVE_E_CPU;
> +
> + object_class_property_add_bool(oc, "revb", sifive_e_machine_get_revb,
> + sifive_e_machine_set_revb);
> + object_class_property_set_description(oc, "revb",
> + "Set on to tell QEMU that it
> should model "
> + "the revB HiFive1 board");
> }
>
> static const TypeInfo sifive_e_machine_typeinfo = {
> --
> 2.26.2
>
>
- Re: [PATCH 12/24] s390x: Register all CPU properties as class properties, (continued)
- [PATCH 14/24] i386: Register feature bit properties as class properties, Eduardo Habkost, 2020/09/21
- [PATCH 19/24] xlnx-zcu102: Register properties as class properties, Eduardo Habkost, 2020/09/21
- [PATCH 15/24] arm/virt: Register most properties as class properties, Eduardo Habkost, 2020/09/21
- [PATCH 16/24] virt: Register "its" as class property, Eduardo Habkost, 2020/09/21
- [PATCH 21/24] vga-pci: Register "big-endian-framebuffer" as class property, Eduardo Habkost, 2020/09/21
- [PATCH 23/24] sifive_e: Register "revb" as class property, Eduardo Habkost, 2020/09/21
- Re: [PATCH 23/24] sifive_e: Register "revb" as class property,
Alistair Francis <=
- [PATCH 17/24] cpu/core: Register core-id and nr-threads as class properties, Eduardo Habkost, 2020/09/21
- [PATCH 18/24] arm/cpu64: Register "aarch64" as class property, Eduardo Habkost, 2020/09/21
- [PATCH 20/24] machine: Register "memory-backend" as class property, Eduardo Habkost, 2020/09/21
- [PATCH 24/24] sifive_u: Register "start-in-flash" as class property, Eduardo Habkost, 2020/09/21
- [PATCH 22/24] i440fx: Register i440FX-pcihost properties as class properties, Eduardo Habkost, 2020/09/21
- Re: [PATCH 00/24] qom: Convert some properties to class properties, Li Qiang, 2020/09/21