1. Patchew
  2. linux
  3. [PATCH v2 0/2] Add ICU support for Renesas RZ/V2N SoC
  4. View patch

[PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC

Prabhakar posted 2 patches 2 months, 2 weeks ago
[PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Prabhakar 2 months, 2 weeks ago 
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.
Although the ICU closely matches the design found on the RZ/V2H(P) family,
it differs in its register layout, particularly in the reduced set of
ECCRAM related registers. These variations require a distinct compatible
string so that software can correctly match and handle the RZ/V2N
implementation.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
v1->v2:
- Dropped using RZ/V2H compatible as a fallback.
---
 .../bindings/interrupt-controller/renesas,rzv2h-icu.yaml         | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml b/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
index 3f99c8645767..cb244b8f5e1c 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
@@ -22,6 +22,7 @@ properties:
   compatible:
     enum:
       - renesas,r9a09g047-icu # RZ/G3E
+      - renesas,r9a09g056-icu # RZ/V2N
       - renesas,r9a09g057-icu # RZ/V2H(P)
 
   '#interrupt-cells':
-- 
2.52.0
Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Conor Dooley 2 months, 2 weeks ago 
Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable
Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Rob Herring (Arm) 2 months, 2 weeks ago 
On Thu, 27 Nov 2025 16:24:46 +0000, Prabhakar wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> 
> Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.
> Although the ICU closely matches the design found on the RZ/V2H(P) family,
> it differs in its register layout, particularly in the reduced set of
> ECCRAM related registers. These variations require a distinct compatible
> string so that software can correctly match and handle the RZ/V2N
> implementation.
> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
> v1->v2:
> - Dropped using RZ/V2H compatible as a fallback.
> ---
>  .../bindings/interrupt-controller/renesas,rzv2h-icu.yaml         | 1 +
>  1 file changed, 1 insertion(+)
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'audio-codec@1,0' does not match any of the regexes: '^pinctrl-[0-9]+$', '^slim@[0-9a-f]+$'
	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): #address-cells: 1 was expected
	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dmas' is a required property
	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dma-names' is a required property
	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20251127162447.320971-2-prabhakar.mahadev-lad.rj@bp.renesas.com

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Thomas Gleixner 2 months ago 
On Thu, Nov 27 2025 at 11:16, Rob Herring wrote:
> On Thu, 27 Nov 2025 16:24:46 +0000, Prabhakar wrote:
>> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>> 
>> Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.
>> Although the ICU closely matches the design found on the RZ/V2H(P) family,
>> it differs in its register layout, particularly in the reduced set of
>> ECCRAM related registers. These variations require a distinct compatible
>> string so that software can correctly match and handle the RZ/V2N
>> implementation.
>> 
>> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>> ---
>> v1->v2:
>> - Dropped using RZ/V2H compatible as a fallback.
>> ---
>>  .../bindings/interrupt-controller/renesas,rzv2h-icu.yaml         | 1 +
>>  1 file changed, 1 insertion(+)
>> 
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'audio-codec@1,0' does not match any of the regexes: '^pinctrl-[0-9]+$', '^slim@[0-9a-f]+$'
> 	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): #address-cells: 1 was expected
> 	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dmas' is a required property
> 	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dma-names' is a required property
> 	from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']

Confused. How is the change in

          bindings/interrupt-controller/renesas,rzv2h-icu.yaml

related to the errors your bot found>

Thanks,

        tglx
Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Lad, Prabhakar 1 month, 4 weeks ago 
Hi Thomas,

On Fri, Dec 12, 2025 at 1:27 AM Thomas Gleixner <tglx@linutronix.de> wrote:
>
> On Thu, Nov 27 2025 at 11:16, Rob Herring wrote:
> > On Thu, 27 Nov 2025 16:24:46 +0000, Prabhakar wrote:
> >> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> >>
> >> Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.
> >> Although the ICU closely matches the design found on the RZ/V2H(P) family,
> >> it differs in its register layout, particularly in the reduced set of
> >> ECCRAM related registers. These variations require a distinct compatible
> >> string so that software can correctly match and handle the RZ/V2N
> >> implementation.
> >>
> >> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> >> ---
> >> v1->v2:
> >> - Dropped using RZ/V2H compatible as a fallback.
> >> ---
> >>  .../bindings/interrupt-controller/renesas,rzv2h-icu.yaml         | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >
> > My bot found errors running 'make dt_binding_check' on your patch:
> >
> > yamllint warnings/errors:
> >
> > dtschema/dtc warnings/errors:
> > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'audio-codec@1,0' does not match any of the regexes: '^pinctrl-[0-9]+$', '^slim@[0-9a-f]+$'
> >       from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): #address-cells: 1 was expected
> >       from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dmas' is a required property
> >       from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/slimbus/slimbus.example.dtb: slim@28080000 (qcom,slim-ngd-v1.5.0): 'dma-names' is a required property
> >       from schema $id: http://devicetree.org/schemas/slimbus/qcom,slim-ngd.yaml
> > Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> > Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c263000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> > Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
> > Documentation/devicetree/bindings/thermal/thermal-sensor.example.dtb: /example-0/soc/thermal-sensor@c265000: failed to match any schema with compatible: ['qcom,sdm845-tsens', 'qcom,tsens-v2']
>
> Confused. How is the change in
>
>           bindings/interrupt-controller/renesas,rzv2h-icu.yaml
>
> related to the errors your bot found>
>
My understanding is that the bot printed out all the errors there were
present in the tree while running the validation.

Cheers,
Prabhakar
Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by Geert Uytterhoeven 2 months, 2 weeks ago 
On Thu, 27 Nov 2025 at 17:24, Prabhakar <prabhakar.csengg@gmail.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.
> Although the ICU closely matches the design found on the RZ/V2H(P) family,
> it differs in its register layout, particularly in the reduced set of
> ECCRAM related registers. These variations require a distinct compatible
> string so that software can correctly match and handle the RZ/V2N
> implementation.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
> v1->v2:
> - Dropped using RZ/V2H compatible as a fallback.

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert


--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
[tip: irq/drivers] dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC
Posted by tip-bot2 for Lad Prabhakar 1 month, 3 weeks ago 
The following commit has been merged into the irq/drivers branch of tip:

Commit-ID:     c71869c61d7cf2070421cdcc4dd64c4c860df49d
Gitweb:        https://git.kernel.org/tip/c71869c61d7cf2070421cdcc4dd64c4c860df49d
Author:        Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
AuthorDate:    Thu, 27 Nov 2025 16:24:46 
Committer:     Thomas Gleixner <tglx@linutronix.de>
CommitterDate: Mon, 15 Dec 2025 22:44:31 +01:00

dt-bindings: interrupt-controller: renesas,rzv2h-icu: Document RZ/V2N SoC

Document the Interrupt Control Unit (ICU) used on the Renesas RZ/V2N SoC.

Although the ICU closely matches the design found on the RZ/V2H(P) family,
it differs in its register layout, particularly in the reduced set of
ECCRAM related registers. These variations require a distinct compatible
string so that software can correctly match and handle the RZ/V2N
implementation.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Link: https://patch.msgid.link/20251127162447.320971-2-prabhakar.mahadev-lad.rj@bp.renesas.com
---
 Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml b/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
index 3f99c86..cb244b8 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/renesas,rzv2h-icu.yaml
@@ -22,6 +22,7 @@ properties:
   compatible:
     enum:
       - renesas,r9a09g047-icu # RZ/G3E
+      - renesas,r9a09g056-icu # RZ/V2N
       - renesas,r9a09g057-icu # RZ/V2H(P)
 
   '#interrupt-cells':

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.