diff mbox series

dt-bindings: interrupt-controller: Fix compatible used in ti,pruss-intc

Message ID 20210607184257.2010276-1-sudeep.holla@arm.com
State Accepted, archived
Headers show
Series dt-bindings: interrupt-controller: Fix compatible used in ti,pruss-intc | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 1 warnings, 8 lines checked
robh/dt-meta-schema success
robh/dtbs-check success

Commit Message

Sudeep Holla June 7, 2021, 6:42 p.m. UTC
As per soc/ti/ti,pruss.yaml schema, only ti,am4376-pruss0 and
ti,am4376-pruss1 are valid compatibles. Replace ti,am4376-pruss with
ti,am4376-pruss1 based on example in soc/ti/ti,pruss.yaml

This fixes the below warning with 'make DT_CHECKER_FLAGS=-m dt_binding_check':

    interrupt-controller/ti,pruss-intc.example.dt.yaml:0:0: /example-1/pruss@0:
    failed to match any schema with compatible: ['ti,am4376-pruss']

Cc: Rob Herring <robh+dt@kernel.org>
Cc: Suman Anna <s-anna@ti.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
 .../devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Suman Anna June 16, 2021, 12:46 a.m. UTC | #1
On 6/7/21 1:42 PM, Sudeep Holla wrote:
> As per soc/ti/ti,pruss.yaml schema, only ti,am4376-pruss0 and
> ti,am4376-pruss1 are valid compatibles. Replace ti,am4376-pruss with
> ti,am4376-pruss1 based on example in soc/ti/ti,pruss.yaml
> 
> This fixes the below warning with 'make DT_CHECKER_FLAGS=-m dt_binding_check':
> 
>     interrupt-controller/ti,pruss-intc.example.dt.yaml:0:0: /example-1/pruss@0:
>     failed to match any schema with compatible: ['ti,am4376-pruss']
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Suman Anna <s-anna@ti.com>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

Thanks for catching and fixing this Sudeep.

Acked-by: Suman Anna <s-anna@ti.com>

> ---
>  .../devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
> index 9731dd4421a1..051beb45d998 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
> @@ -134,7 +134,7 @@ additionalProperties: false
>      /* AM4376 PRU-ICSS */
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>      pruss@0 {
> -        compatible = "ti,am4376-pruss";
> +        compatible = "ti,am4376-pruss1";
>          reg = <0x0 0x40000>;
>          #address-cells = <1>;
>          #size-cells = <1>;
>
Rob Herring (Arm) June 16, 2021, 5:35 p.m. UTC | #2
On Mon, 07 Jun 2021 19:42:57 +0100, Sudeep Holla wrote:
> As per soc/ti/ti,pruss.yaml schema, only ti,am4376-pruss0 and
> ti,am4376-pruss1 are valid compatibles. Replace ti,am4376-pruss with
> ti,am4376-pruss1 based on example in soc/ti/ti,pruss.yaml
> 
> This fixes the below warning with 'make DT_CHECKER_FLAGS=-m dt_binding_check':
> 
>     interrupt-controller/ti,pruss-intc.example.dt.yaml:0:0: /example-1/pruss@0:
>     failed to match any schema with compatible: ['ti,am4376-pruss']
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Suman Anna <s-anna@ti.com>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> ---
>  .../devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
index 9731dd4421a1..051beb45d998 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/ti,pruss-intc.yaml
@@ -134,7 +134,7 @@  additionalProperties: false
     /* AM4376 PRU-ICSS */
     #include <dt-bindings/interrupt-controller/arm-gic.h>
     pruss@0 {
-        compatible = "ti,am4376-pruss";
+        compatible = "ti,am4376-pruss1";
         reg = <0x0 0x40000>;
         #address-cells = <1>;
         #size-cells = <1>;