On Fri, 6 Dec 2024 at 16:02, Alex Bennée <alex.ben...@linaro.org> wrote: > > As we are about to add more physical and virtual timers lets make it > clear what each timer does. > > Signed-off-by: Alex Bennée <alex.ben...@linaro.org> > Cc: qemu-sta...@nongnu.org > --- > target/arm/gtimer.h | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/target/arm/gtimer.h b/target/arm/gtimer.h > index b992941bef..de016e6da3 100644 > --- a/target/arm/gtimer.h > +++ b/target/arm/gtimer.h > @@ -10,11 +10,11 @@ > #define TARGET_ARM_GTIMER_H > > enum { > - GTIMER_PHYS = 0, > - GTIMER_VIRT = 1, > - GTIMER_HYP = 2, > - GTIMER_SEC = 3, > - GTIMER_HYPVIRT = 4, > + GTIMER_PHYS = 0, /* EL1 physical timer */ > + GTIMER_VIRT = 1, /* EL1 virtual timer */ > + GTIMER_HYP = 2, /* EL2 physical timer */ > + GTIMER_SEC = 3, /* EL3 physical timer */ > + GTIMER_HYPVIRT = 4, /* EL2 virtual timer */ > #define NUM_GTIMERS 5 > };
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM