[PATCH v2 1/4] lib: utils/timer: mtimer: add separate T-Head C9xx CLINT mtimer compatible

[Samuel Holland]

On 2023-10-08 7:16 AM, Inochi Amaoto wrote:
> T-HEAD allows soc vendor to map the mtimer and mswi of C9xx clint on
> different address, which may cause a misuse if use the same compatible
> string, add a new timer compatible string to avoid this.
> 
> Signed-off-by: Inochi Amaoto <inochiama at outlook.com>
> Link: https://lore.kernel.org/linux-riscv/6e48cbe5e60f9ada2fd1fe58e803e127f1a678e5.1696433229.git.unicorn_wang@outlook.com/
> ---
>  lib/utils/timer/fdt_timer_mtimer.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/lib/utils/timer/fdt_timer_mtimer.c b/lib/utils/timer/fdt_timer_mtimer.c
> index 9eaa11d..0ac9aed 100644
> --- a/lib/utils/timer/fdt_timer_mtimer.c
> +++ b/lib/utils/timer/fdt_timer_mtimer.c
> @@ -146,6 +146,8 @@ static const struct fdt_match timer_mtimer_match[] = {
>  	{ .compatible = "riscv,clint0", .data = &sifive_clint_quirks },
>  	{ .compatible = "sifive,clint0", .data = &sifive_clint_quirks },
>  	{ .compatible = "thead,c900-clint", .data = &thead_clint_quirks },
> +	{ .compatible = "thead,c900-aclint-mtimer",
> +	  .data = &thead_clint_quirks },

This is wrong. When using separate mtimer/mswi nodes, there is no MMIO offset.
So you need a new quirks structure for this compatible string.

Regards,
Samuel

>  	{ .compatible = "riscv,aclint-mtimer" },
>  	{ },
>  };