Re: [PATCH v1 5/9] dt-bindings: interrupt-controller: Add SpacemiT K1 PLIC

From: Conor Dooley
Date: Tue Jun 18 2024 - 11:46:02 EST


On Mon, Jun 17, 2024 at 01:20:50AM +0800, Yangyu Chen wrote:
> Add compatible string for SpacemiT K1 PLIC.
>
> Signed-off-by: Yangyu Chen <cyy@xxxxxxxxxxxx>
> ---
> .../bindings/interrupt-controller/sifive,plic-1.0.0.yaml | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/sifive,plic-1.0.0.yaml b/Documentation/devicetree/bindings/interrupt-controller/sifive,plic-1.0.0.yaml
> index 709b2211276b..3d0426b56904 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/sifive,plic-1.0.0.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/sifive,plic-1.0.0.yaml
> @@ -59,6 +59,7 @@ properties:
> - enum:
> - canaan,k210-plic
> - sifive,fu540-c000-plic
> + - spacemit,k1-plic
> - starfive,jh7100-plic
> - starfive,jh7110-plic
> - const: sifive,plic-1.0.0
> @@ -71,7 +72,9 @@ properties:
> - thead,th1520-plic
> - const: thead,c900-plic
> - items:
> - - const: sifive,plic-1.0.0
> + - enum:
> + - sifive,plic-1.0.0
> + - spacemit,k1-plic

Why in two places? This isn't sifive IP, so I think we should go with
this one here.

> - const: riscv,plic0
> deprecated: true
> description: For the QEMU virt machine only
> --
> 2.45.1
>

Attachment: signature.asc
Description: PGP signature