| Message ID | 20230824221440.484675-3-dbarboza@ventanamicro.com |
|---|---|
| State | New |
| Headers | show |
| Series | riscv: 'max' CPU, detect user choice in TCG | expand |
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 <dbarboza@ventanamicro.com> > Reviewed-by: Alistair Francis <alistair.francis@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 <ajones@ventanamicro.com>
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);