On Fri, 6 Dec 2024 at 16:02, Alex Bennée <alex.bennee@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.bennee@linaro.org>
> Cc: qemu-stable@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.maydell@linaro.org>
thanks
-- PMM