diff mbox series

media: dt-bindings: adv7180: Document the 'interrupts' property

Message ID 20250109000503.45264-1-festevam@gmail.com
State New
Headers show
Series media: dt-bindings: adv7180: Document the 'interrupts' property | expand

Commit Message

Fabio Estevam Jan. 9, 2025, 12:05 a.m. UTC 
From: Fabio Estevam <festevam@denx.de>

The ADV7180 family of chips have an INTRQ pin that can be connected
to a SoC GPIO.

Allow the 'interrupts' property to be described to fix the following
dt-schema warning:

camera@21: 'interrupt-parent', 'interrupts' do not match any of the regexes: 'pinctrl-[0-9]+'

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/media/i2c/adv7180.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Rob Herring (Arm) Jan. 10, 2025, 4:09 p.m. UTC | #1 
On Wed, Jan 08, 2025 at 09:05:03PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> The ADV7180 family of chips have an INTRQ pin that can be connected
> to a SoC GPIO.
> 
> Allow the 'interrupts' property to be described to fix the following
> dt-schema warning:
> 
> camera@21: 'interrupt-parent', 'interrupts' do not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  Documentation/devicetree/bindings/media/i2c/adv7180.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/i2c/adv7180.yaml b/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
> index 4371a0ef2761..572f06d12a39 100644
> --- a/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
> @@ -49,6 +49,8 @@ properties:
>        Indicates that the output is a BT.656-4 compatible stream.
>      type: boolean
>  
> +  interrupts: true
> +

How many interrupts are there?

>    port:
>      $ref: /schemas/graph.yaml#/$defs/port-base
>      unevaluatedProperties: false
> -- 
> 2.34.1
>
Fabio Estevam Jan. 10, 2025, 6:51 p.m. UTC | #2 
On Fri, Jan 10, 2025 at 1:09 PM Rob Herring <robh@kernel.org> wrote:

> How many interrupts are there?

Only one.

I sent a v2 that makes this clear.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/i2c/adv7180.yaml b/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
index 4371a0ef2761..572f06d12a39 100644
--- a/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/adv7180.yaml
@@ -49,6 +49,8 @@  properties:
       Indicates that the output is a BT.656-4 compatible stream.
     type: boolean
 
+  interrupts: true
+
   port:
     $ref: /schemas/graph.yaml#/$defs/port-base
     unevaluatedProperties: false