Re: [PATCH v3 3/6] dt-bindings: interrupt-controller: thead,c900-aclint-sswi: Add SpacemiT K3

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dt-bindings: missing SpacemiT K3 in text description of supported variants
--

commit e153d8cef506aa80d048bbfcecc93b53a351d62b
Author: Junhui Liu <junhui.liu@pigmoral.tech>

dt-bindings: interrupt-controller: thead,c900-aclint-sswi: Add SpacemiT K3

This commit adds support for the SpacemiT K3 ACLINT SSWI. It updates the
compatible strings list to include spacemit,k3-aclint-sswi.

> diff --git a/Documentation/devicetree/bindings/interrupt-controller/thead,c900-aclint-sswi.yaml b/Documentation/devicetree/bindings/interrupt-controller/thead,c900-aclint-sswi.yaml
> index d02c6886283af..0b34ed5b4639d 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/thead,c900-aclint-sswi.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/thead,c900-aclint-sswi.yaml
> @@ -24,12 +24,13 @@ description:

[Severity: Low]
Does the text description of supported variants need to be updated to match
the new compatible string?

Looking at the full description block in
Documentation/devicetree/bindings/interrupt-controller/thead,c900-aclint-sswi.yaml:

  Following variants of the SSWI ACLINT supported, using dedicated
  compatible string
  - THEAD C900
  - MIPS P8700

Should the SpacemiT K3 variant be added to this list so the documentation
stays in sync with the new schema properties?

>  properties:
>    compatible:
>      oneOf:
> +      - enum:
> +          - mips,p8700-aclint-sswi
> +          - spacemit,k3-aclint-sswi

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260918-k3-aclint-v3-0-cf4fd4050e14@pigmoral.tech?part=3