Re: [PATCH 1/3] target/arm: document the architectural names of our GTIMERs

[Peter Maydell]

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