[PATCH v2 1/1] lib: sbi: Allow programmable counters to monitor cycle/instret events for Andes PMU

[Atish Patra]

On 3/28/25 1:41 AM, Leo Yu-Chi Liang wrote:
> Referencing commit 0c304b661965
> ("lib: sbi: Allow programmable counters to monitor cycle/instret events")
> to support this functionality for Andes PMU.
> 
> Signed-off-by: Leo Yu-Chi Liang <ycliang at andestech.com>
> Reviewed-by: Samuel Holland <samuel.holland at sifive.com>
> ---
> Change since v1:
> 	- Modify the comment
> ---
>   lib/sbi/sbi_pmu.c | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/sbi/sbi_pmu.c b/lib/sbi/sbi_pmu.c
> index b9e84543..575799eb 100644
> --- a/lib/sbi/sbi_pmu.c
> +++ b/lib/sbi/sbi_pmu.c
> @@ -719,12 +719,13 @@ static int pmu_ctr_find_hw(struct sbi_pmu_hart_state *phs,
>   		return SBI_EINVAL;
>   
>   	/**
> -	 * If Sscof is present try to find the programmable counter for
> -	 * cycle/instret as well.
> +	 * If Sscofpmf or Andes PMU is present, try to find
> +	 * the programmable counter for cycle/instret as well.
>   	 */
>   	fixed_ctr = pmu_ctr_find_fixed_hw(event_idx);
>   	if (fixed_ctr >= 0 &&
> -	    !sbi_hart_has_extension(scratch, SBI_HART_EXT_SSCOFPMF))
> +	    !sbi_hart_has_extension(scratch, SBI_HART_EXT_SSCOFPMF) &&
> +	    !sbi_hart_has_extension(scratch, SBI_HART_EXT_XANDESPMU))
>   		return pmu_fixed_ctr_update_inhibit_bits(fixed_ctr, flags);
>   
>   	if (sbi_hart_priv_version(scratch) >= SBI_HART_PRIV_VER_1_11)

Reviewed-by: Atish Patra <atishp at rivosinc.com>