[PATCH v2 2/4] lib: utils/ipi: mswi: add separate T-Head C9xx CLINT mswi compatible

Inochi Amaoto inochiama at outlook.com
Sun Oct 8 17:19:06 PDT 2023


>On 2023-10-08 7:16 AM, Inochi Amaoto wrote:
>> Like the mtimer of T-HEAD C9xx clint, the mswi also needs new compatible
>> string to avoid misuse.
>>
>> Signed-off-by: Inochi Amaoto <inochiama at outlook.com>
>> Link: https://lore.kernel.org/linux-riscv/1f6b82a1864477a51db33d3f295889ff985b497b.1696433229.git.unicorn_wang@outlook.com/
>> ---
>>  lib/utils/ipi/fdt_ipi_mswi.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/utils/ipi/fdt_ipi_mswi.c b/lib/utils/ipi/fdt_ipi_mswi.c
>> index e3a6378..f4b2de4 100644
>> --- a/lib/utils/ipi/fdt_ipi_mswi.c
>> +++ b/lib/utils/ipi/fdt_ipi_mswi.c
>> @@ -56,6 +56,7 @@ static const struct fdt_match ipi_mswi_match[] = {
>>  	{ .compatible = "riscv,clint0", .data = &clint_offset },
>>  	{ .compatible = "sifive,clint0", .data = &clint_offset },
>>  	{ .compatible = "thead,c900-clint", .data = &clint_offset },
>> +	{ .compatible = "thead,c900-aclint-mswi", .data = &clint_offset },
>
>This should omit .data, like for riscv,aclint-mswi below, so it gets the MMIO
>region base and size directly from the DT node.
>
>Regards,
>Samuel
>

Yes, I will omit it. It seems I have misunderstand the aclint.

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



More information about the opensbi mailing list