On Thu, Aug 24, 2023 at 07:14:22PM -0300, Daniel Henrique Barboza wrote: > After the introduction of riscv_cpu_options[] all properties in > riscv_cpu_extensions[] are booleans. This check is now obsolete. > > Signed-off-by: Daniel Henrique Barboza <dbarb...@ventanamicro.com> > Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> > --- > target/riscv/cpu.c | 14 ++++---------- > 1 file changed, 4 insertions(+), 10 deletions(-) > > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c > index 6a4f95991d..6aac3ba1d1 100644 > --- a/target/riscv/cpu.c > +++ b/target/riscv/cpu.c > @@ -1970,17 +1970,11 @@ static void riscv_cpu_add_user_properties(Object *obj) > * Set the default to disabled for every extension > * unknown to KVM and error out if the user attempts > * to enable any of them. > - * > - * We're giving a pass for non-bool properties since they're > - * not related to the availability of extensions and can be > - * safely ignored as is. > */ > - if (prop->info == &qdev_prop_bool) { > - object_property_add(obj, prop->name, "bool", > - NULL, cpu_set_cfg_unavailable, > - NULL, (void *)prop->name); > - continue; > - } > + object_property_add(obj, prop->name, "bool", > + NULL, cpu_set_cfg_unavailable, > + NULL, (void *)prop->name); > + continue; > } > #endif > qdev_property_add_static(dev, prop); > -- > 2.41.0 > >
Reviewed-by: Andrew Jones <ajo...@ventanamicro.com>