On Tue, Jun 16, 2026 at 6:42 AM Marc-André Lureau <[email protected]> wrote: > > "pmu_num" is u8, "pmu_mask" is u32, so is "pmp-granularity" > > Signed-off-by: Marc-André Lureau <[email protected]>
Reviewed-by: Alistair Francis <[email protected]> Alistair > --- > target/riscv/cpu.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c > index a63961bb713..98c9ebd791c 100644 > --- a/target/riscv/cpu.c > +++ b/target/riscv/cpu.c > @@ -1302,7 +1302,7 @@ static void prop_pmu_num_get(Object *obj, Visitor *v, > const char *name, > } > > static const PropertyInfo prop_pmu_num = { > - .type = "int8", > + .type = "uint8", > .description = "pmu-num", > .get = prop_pmu_num_get, > .set = prop_pmu_num_set, > @@ -1338,13 +1338,13 @@ static void prop_pmu_mask_set(Object *obj, Visitor > *v, const char *name, > static void prop_pmu_mask_get(Object *obj, Visitor *v, const char *name, > void *opaque, Error **errp) > { > - uint8_t pmu_mask = RISCV_CPU(obj)->cfg.pmu_mask; > + uint32_t pmu_mask = RISCV_CPU(obj)->cfg.pmu_mask; > > - visit_type_uint8(v, name, &pmu_mask, errp); > + visit_type_uint32(v, name, &pmu_mask, errp); > } > > static const PropertyInfo prop_pmu_mask = { > - .type = "int8", > + .type = "uint32", > .description = "pmu-mask", > .get = prop_pmu_mask_get, > .set = prop_pmu_mask_set, > @@ -1486,6 +1486,7 @@ static void prop_pmp_granularity_get(Object *obj, > Visitor *v, const char *name, > } > > static const PropertyInfo prop_pmp_granularity = { > + .type = "uint32", > .description = "pmp-granularity", > .get = prop_pmp_granularity_get, > .set = prop_pmp_granularity_set, > > -- > 2.54.0 > >
